Skip to content

PM-5831: Fix My Submissions download redirect - #7261

Merged
jmgasper merged 1 commit into
developfrom
PM-5831
Aug 11, 2026
Merged

PM-5831: Fix My Submissions download redirect#7261
jmgasper merged 1 commit into
developfrom
PM-5831

Conversation

@jmgasper

Copy link
Copy Markdown
Collaborator

What was broken

Downloading a submission from the My Submissions page failed after Review API began returning a signed storage redirect.

Root cause

Community App fetched the redirecting download endpoint as a blob. Firefox followed that authenticated request to S3, which triggered a rejected CORS preflight instead of downloading the file.

What was changed

  • Request the browser-safe /submissions/{submissionId}/download-url endpoint with the member token.
  • Validate the signed URL returned by Review API.
  • Start the existing browser-managed download directly from the signed URL, avoiding the cross-origin blob request.
  • Preserve the existing submission filename and leave artifact downloads unchanged.

Any added/updated tests

  • Added service tests for the authenticated, URL-encoded download-url request.
  • Added coverage for signed URL trimming and missing URL rejection.
  • Verified the focused tests, full test suite, lint, and production build all pass.

What was broken

Downloading a submission from My Submissions failed after Review API began returning a signed storage redirect.

Root cause

Community App fetched the redirecting endpoint as a blob, so Firefox followed the authenticated request to S3 and triggered a rejected CORS preflight.

What was changed

Request the browser-safe /download-url endpoint with the member token, validate the signed URL, and start the existing browser-managed download directly from that URL.

Any added/updated tests

Added service coverage for the authenticated, URL-encoded download-url request, signed URL parsing, and missing URL rejection.
@jmgasper
jmgasper requested a review from kkartunov as a code owner August 11, 2026 04:10
@jmgasper
jmgasper merged commit 1c4b64a into develop Aug 11, 2026
7 checks passed
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