Skip to content

Conversation

@WiNloSt WiNloSt marked this pull request as draft November 11, 2025 13:03
vercel.json Outdated
{
"key": "CDN-Cache-Control",
"value": "public, max-age=2592000"
"value": "max-age=0, s-maxage=2592000"
Copy link
Member Author

Choose a reason for hiding this comment

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

Use the recommended header instead.

@github-actions
Copy link

Preview Deployment Ready!
🔗 View Preview Deployment

@github-actions
Copy link

Preview Deployment Ready!
🔗 View Preview Deployment

@WiNloSt
Copy link
Member Author

WiNloSt commented Nov 25, 2025

The below deployment is the one with routes.transform to attempt to remove set-cookie header from the embedding-sdk.js response which prevent CDN from caching it on the first request to Metabase server.

@github-actions

This comment was marked as outdated.

@WiNloSt WiNloSt force-pushed the troubleshoot-cdn-not-caching branch from 6813352 to 69a0c2e Compare November 25, 2025 13:16
@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as off-topic.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions
Copy link

Preview Deployment Ready!
🔗 View Preview Deployment

@WiNloSt
Copy link
Member Author

WiNloSt commented Nov 25, 2025

Note: using vercel function. I couldn't get cache to work at all.

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.

2 participants