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

Don't cache aggregaated files for 2 weeks. #185

Open
simesy opened this issue Jul 6, 2023 · 0 comments
Open

Don't cache aggregaated files for 2 weeks. #185

simesy opened this issue Jul 6, 2023 · 0 comments

Comments

@simesy
Copy link

simesy commented Jul 6, 2023

When i deploy into a site which has aggregation turned on, the CSS/JS aggregation files may be re-created with new content, but the file name remains the same. This makes it difficult to cache bust these files. For this reason, I think this 2 week caching is problematic and would cause issues for people who don't understand the mechanisms and how to address them.

drupal9/.platform.app.yaml

Lines 142 to 145 in ae188ba

rules:
# Provide a longer TTL (2 weeks) for aggregated CSS and JS files.
'^/sites/default/files/(css|js)':
expires: 2w

More information on this ticket https://www.drupal.org/project/drupal/issues/3370828#comment-15140268

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

No branches or pull requests

1 participant