Skip to content

Fix coursework document storage lifecycle - #25

Open
Linda-riziki wants to merge 1 commit into
mainfrom
document-storage-lifecycle
Open

Fix coursework document storage lifecycle#25
Linda-riziki wants to merge 1 commit into
mainfrom
document-storage-lifecycle

Conversation

@Linda-riziki

Copy link
Copy Markdown
Collaborator

Summary

This PR fixes the coursework document storage lifecycle by improving the security and consistency of uploaded files.

Changes

  • Made the coursework storage bucket private.
  • Replaced public file URLs with private storage paths.
  • Added secure access to coursework files using server-side/signed retrieval.
  • Updated document deletion to remove both the database record and the storage object.
  • Added retry handling for failed storage cleanup.
  • Added a reconciliation script to identify and clean up orphaned or missing storage records.
  • Added a coursework retention policy.
  • Added migration and storage lifecycle test scripts.

Testing

  • Verified the application builds and runs locally.
  • Reviewed the upload and delete flows to ensure they use private storage paths and clean up storage correctly.
  • End-to-end upload testing could not be completed because uploads require a paid credit.

@netlify

netlify Bot commented Jul 28, 2026

Copy link
Copy Markdown

Deploy Preview for skillsyncglobal ready!

Name Link
🔨 Latest commit 8e255a4
🔍 Latest deploy log https://app.netlify.com/projects/skillsyncglobal/deploys/6a68a34dcc4d9c0008c7b96b
😎 Deploy Preview https://deploy-preview-25--skillsyncglobal.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
skills-sync Ready Ready Preview, Comment Jul 28, 2026 12:40pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant