Skip to content

[comp] Production Deploy#2224

Merged
tofikwest merged 2 commits intoreleasefrom
main
Mar 5, 2026
Merged

[comp] Production Deploy#2224
tofikwest merged 2 commits intoreleasefrom
main

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Mar 5, 2026

This is an automated pull request to release the candidate branch into production, which will trigger a deployment.
It was created by the [Production PR] action.

github-actions bot and others added 2 commits March 5, 2026 17:25
#2223)

Presigned S3 download URLs (15-min TTL) were stored in the database at
upload time. When users viewed submissions later, clicking attachments
returned "AccessDenied - Request has expired". Now regenerate fresh
presigned URLs when returning submissions via getFormWithSubmissions
and getSubmission, following the same pattern already used in exportCsv.
@cursor
Copy link

cursor bot commented Mar 5, 2026

PR Summary

Medium Risk
Changes the evidence-submission read paths to regenerate presigned S3 download URLs at request time, which can affect response shape and adds per-submission S3 signing calls (latency/throughput risk). Scope is limited to evidence-forms retrieval endpoints.

Overview
Ensures evidence form submissions returned by the API include fresh presigned download links for any file fields.

Adds refreshFileUrls() to rewrite { fileKey, downloadUrl, ... } objects with a newly generated downloadUrl, and wires it into both getFormWithSubmissions (list/paginated results) and getSubmission (single submission) before normalizeSubmissionFormType.

Written by Cursor Bugbot for commit 783db83. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Mar 5, 2026

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

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
app (staging) Skipped Skipped Mar 5, 2026 8:37pm
portal (staging) Skipped Skipped Mar 5, 2026 8:37pm

Request Review

@tofikwest tofikwest merged commit ce3ed46 into release Mar 5, 2026
11 of 13 checks passed
@claudfuen
Copy link
Contributor

🎉 This PR is included in version 1.88.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants