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

Add cache busting parameter to media thumbnails and links in Media Library #15273

Closed
Piedone opened this issue Feb 7, 2024 · 0 comments · Fixed by #15276
Closed

Add cache busting parameter to media thumbnails and links in Media Library #15273

Piedone opened this issue Feb 7, 2024 · 0 comments · Fixed by #15276
Assignees

Comments

@Piedone
Copy link
Member

Piedone commented Feb 7, 2024

Is your feature request related to a problem? Please describe.

It's recommended to send long far-future client cache TTLs in HTTP response headers for static resources like images, documents, and CSS files, and this is what OC also does. However, in the Media Library admin this causes an issue if you change files, since thumbnails and "View" links will now load the old files. This is especially a problem with CDNs, since then it's not just your browser caching these files but the CDN too, which for an ordinary user is impossible to purge, and thus they won't see the updated files.

Describe the solution you'd like

Make Media Library admin thumbnails and "View' links use the usual cache-busting parameter mechanism.

Describe alternatives you've considered

The only alternative I can think of is using ETAGs but validating resources would still need an HTTP request by clients (though a small HEAD one).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant