Skip to content

Fix preview app worker proxy access issue - #87

Merged
richardthe3rd merged 1 commit into
mainfrom
claude/fix-preview-worker-proxy-01M2QfvgqgzavYUxKRMYzMy8
Dec 2, 2025
Merged

Fix preview app worker proxy access issue#87
richardthe3rd merged 1 commit into
mainfrom
claude/fix-preview-worker-proxy-01M2QfvgqgzavYUxKRMYzMy8

Conversation

@richardthe3rd

Copy link
Copy Markdown
Owner

Changed Access-Control-Max-Age from 86400s (24 hours) to 300s (5 minutes). This allows faster recovery when CORS issues occur with preview deployments, while still providing caching benefits to reduce OPTIONS requests.

Fixes issue where preview deployments couldn't access the worker proxy unless in a private session due to cached CORS failures.

Changed Access-Control-Max-Age from 86400s (24 hours) to 300s (5 minutes).
This allows faster recovery when CORS issues occur with preview deployments,
while still providing caching benefits to reduce OPTIONS requests.

Fixes issue where preview deployments couldn't access the worker proxy
unless in a private session due to cached CORS failures.
Copilot AI review requested due to automatic review settings December 2, 2025 20:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a CORS caching issue that prevented preview deployments from accessing the worker proxy. The change reduces the preflight cache duration from 24 hours to 5 minutes, allowing faster recovery from cached CORS failures while still providing benefits to reduce OPTIONS requests.

Key Changes:

  • Modified Access-Control-Max-Age header value from 86400 to 300 seconds (5 minutes)

@richardthe3rd
richardthe3rd merged commit cdc27d5 into main Dec 2, 2025
19 checks passed
@richardthe3rd
richardthe3rd deleted the claude/fix-preview-worker-proxy-01M2QfvgqgzavYUxKRMYzMy8 branch December 2, 2025 20:38
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.

3 participants