Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Issue 543 excess presigned url checks #544

Closed
wants to merge 9 commits into from
Closed

Issue 543 excess presigned url checks #544

wants to merge 9 commits into from

Conversation

petermAprende
Copy link
Contributor

Fixes #543

andrewmadden and others added 9 commits January 17, 2023 02:07
@andrewmadden
Issue catalyst#519 Add performance check that validates if presigned …
ecf53d8 
…urls are set up correctly.
@andrewmadden
Issue catalyst#519 Replace existing presigned test page with new chec…
0b52885 
…k API
@andrewmadden
issue catalyst#535 Honor the key_prefix when making requests using cl…
d2633f5 
…oudfront.
@michael-milette
Added Aprende Github workflows.
359b2c3
@michael-milette
Merge branch 'issue519-MOODLE_310_STABLE_presigned_urls_check' of htt…
bdd6495 
…ps://github.com/catalyst/moodle-tool_objectfs into LMSDEV-4490
@davidherzlos
Merge pull request #1 from Aprende-com/LMSDEV-4490
9e83328 
LMSDEV-4490: Upgrade tool_objectfs plugin
@michael-milette
LMSDEV-4518: Revert "Added Aprende Github workflows."
0146643 
This reverts commit 359b2c3.
@michael-milette
Merge pull request catalyst#2 from Aprende-com/LMSDEV-4518
79b805a 
LMSDEV-4518: Revert "Added Aprende Github workflows."
@petermAprende
fix: only trigger pre-signed check if on tool_objectfs settings page
338a28f
@petermAprende petermAprende deleted the issue_543_excess_presigned_url_checks branch February 28, 2023 19:12
@petermAprende
Copy link
Contributor Author

Sorry this included a bunch of nonsense that I didn't mean to include, I'll try again without it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

The pre-signed URL check triggers on every admin page
4 participants
@petermAprende @andrewmadden @michael-milette @davidherzlos