Skip to content

Add setting to disallow presigned urls based on cache headers #469 - #763

Open
bwalkerl wants to merge 1 commit into
MOODLE_404_STABLEfrom
469-presigned-cache
Open

Add setting to disallow presigned urls based on cache headers #469#763
bwalkerl wants to merge 1 commit into
MOODLE_404_STABLEfrom
469-presigned-cache

Conversation

@bwalkerl

@bwalkerl bwalkerl commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Test instructions:

  1. Setup objectfs and enable Pre-Signed URLs.
  2. Upload a file with https://github.com/catalyst/moodle-tool_publicfiles or find any other immutable public files.
  3. Start with the setting disabled and confirm Pre-Signing is used on the file
  4. Enable the new setting
  5. Open dev tools and disable caching as the Pre-Signed URL will still likely be cached.
  6. With dev tools still open load the file and confirm the file isn't redirected to the Pre-Signed URL.

Closes #469

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.

Have an option to not sign cache control immutable / public files

1 participant