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.

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

Fixing the cache max age during the deployment to an S3 #396

Merged
merged 1 commit into from
May 14, 2024

Conversation

amarouane-ABDELHAK
Copy link
Contributor

The PR contains setting the max-age=30, must-revalidate, s-maxage=60 parameters to force the reloading of asset files when necessary.
This line ensures that the resource is cached for up to 30 seconds for individual clients, but shared caches must revalidate it after 60 seconds to ensure it remains fresh. The must-revalidate directive enforces revalidation once the resource becomes stale, regardless of whether it's in an individual or shared cache.

Copy link

netlify bot commented May 14, 2024

Deploy Preview for visex ready!

Name Link
🔨 Latest commit ead1f68
🔍 Latest deploy log https://app.netlify.com/sites/visex/deploys/664371308ea8e50008aa3d92
😎 Deploy Preview https://deploy-preview-396--visex.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@hanbyul-here hanbyul-here left a comment

Choose a reason for hiding this comment

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

Thank you for working on this @amarouane-ABDELHAK

@amarouane-ABDELHAK amarouane-ABDELHAK merged commit 6b1a70f into develop May 14, 2024
6 checks passed
@amarouane-ABDELHAK amarouane-ABDELHAK deleted the fix/cache_max_age branch May 14, 2024 19: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.

2 participants