-
Notifications
You must be signed in to change notification settings - Fork 337
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
Cache state: partial content #38
Comments
Not sure what is the question here. Is the "cache mode" from this bug title the cache mode on fetch? |
Sorry, I messed up the title, this is about a related feature: https://fetch.spec.whatwg.org/#concept-response-cache-state And also how we should deal with partial content cache entries overall within Fetch. |
Exactly. We always hand out a 200 (on a new 200 server response -> cache state "none", or a 206 response from the server that we, as you say, combine -> cache state "partial" or could well be "validated" since 206 is mostly the same as 304. But depends. I kinda don't see a use case right now for the "partial" state, tho)
No, this can happen for anything, large HTML, CSS whatever. The response just needs to fulfill some conditions to be "resumable". I don't think you need to be concerned about it tho. |
Some IRC discussion of what we do in gecko's http cache: 12:13 PM [valentin] mayhemer: bkelly asked: do you know if our http cache does anything special for range requests? are they automatically combined together or anything? |
Closing this in favor of #144 which is a bit more clear. |
@mayhemer it's not entirely clear to me how to deal with partial content. Does us handling that mean we also pay special attention to a 206 response? What's the logic here, if we notice we have a partial cache entry, we set
If-Range
, then if there's a 206 response, we combine some stuff and hand out the combined response as a 200 to content? Does this only happen for media elements basically?The text was updated successfully, but these errors were encountered: