-
Notifications
You must be signed in to change notification settings - Fork 21
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
Test for the immutable attribute #70
Comments
That would be great! The tricky thing is to put the browsers into a state where they're reloading, to trigger the semantics of Happy to help / review / etc. as you like. |
I have some tests and discussion at https://bayden.com/test/cache/refresh.aspx. One challenge is that there are so many different sorts of "reload" in different browsers. I think moving to |
Hm. A quick test shows that none of the major 3 browser engines seems to honour My reading of the fetch spec's definition of those semantics (scroll down) is that they arguably should, or at least that it would make sense to; i.e., that |
See discussion in whatwg/fetch#308. |
(I would love it if we could get to a point where new "protocol" features have some kind of end-to-end integration performed so that everyone is aligned on what should happen everywhere in the pipeline before we declare it ready. That might help reduce the amount of time we spent on fixing things many years into the future.) |
Do they all behave the same way in terms of request headers? E.g. do the requests send the same |
Looks like they are ( |
AFAICT we're testing this, it's just that the browsers aren't following the spec. Please add new information if you find it; closing for now. |
We are considering sending
Cache-Control: immutable...
for files on cdnjs.com (cdnjs/cdnjs#8655). However, the browser support is not ideal and it's hard to understand what's going to happen.It would be very helpful to us to test the
immutable
attribute as part of the cache tests and I would be happy to invest the time. What do you think?The text was updated successfully, but these errors were encountered: