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

Change tracker.js Cache-Control header to no-store #293

Merged
merged 1 commit into from
Feb 14, 2020
Merged

Change tracker.js Cache-Control header to no-store #293

merged 1 commit into from
Feb 14, 2020

Conversation

JacobEvelyn
Copy link
Contributor

According to all documentation I can find, no-store is the strongest "do not cache" Cache-Control header there is, and no-cache and must-revalidate are strictly weaker and will have no effect when no-store is present. Additionally, I cannot find any evidence that these other values are useful for backwards compatibility or older browsers, and no-store support appears to be universal. Lastly, webhint.io is specifically telling me that I should not use must-revalidate.

According to all documentation I can find, `no-store` is the strongest "do not cache" `Cache-Control` header there is, and `no-cache` and `must-revalidate` are strictly weaker and will have no effect when `no-store` is present. Additionally, I cannot find any evidence that these other values are useful for backwards compatibility or older browsers, and `no-store` support appears to be universal. Lastly, webhint.io is specifically telling me that I should not use `must-revalidate`.
@JackEllis
Copy link
Collaborator

Thanks @JacobEvelyn (and @m1guelpf for getting this to the top). We'll merge it in and then recompile in the future when we do our new release

@JackEllis JackEllis closed this Feb 14, 2020
@JackEllis JackEllis reopened this Feb 14, 2020
@JackEllis JackEllis merged commit 0dd6d41 into usefathom:master Feb 14, 2020
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