This repository has been archived by the owner on May 5, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 31
Define preload cache and request matching algorithm #30
Comments
From: https://groups.google.com/a/chromium.org/d/msg/blink-dev/_nu6HlbNQfo/OAzGqQa8BgAJ
I think these will be answered once we resolve this bug, which is about defining the matching algorithm within Fetch. A really high-level overview of what I'm thinking:
/cc @annevk @yoavweiss |
That sounds sane-ish. H2 server push is already kept separate until it's actually referenced from content. /cc @mcmanus |
igrigorik
added a commit
that referenced
this issue
Nov 23, 2016
Preload response's are one of several responses that interact with the (yet to be formally defined) response cache. As such, the logic for both populating and querying said cache should live in Fetch. - Updated issue text as a warning, with some context and pointer to the discussion in Fetch. - Removed "match a preloaded response" definition: this should be handled transparently by Fetch, as one of the first steps in its main fetch algorithm. Fetch issue where we should continue this discussion: whatwg/fetch#354 Closes #30.
igrigorik
added a commit
that referenced
this issue
Nov 30, 2016
* response cache and matching lives in Fetch Preload response's are one of several responses that interact with the (yet to be formally defined) response cache. As such, the logic for both populating and querying said cache should live in Fetch. - Updated issue text as a warning, with some context and pointer to the discussion in Fetch. - Removed "match a preloaded response" definition: this should be handled transparently by Fetch, as one of the first steps in its main fetch algorithm. Fetch issue where we should continue this discussion: whatwg/fetch#354 Closes #30.
I think it would be good to actually leave this open until it is properly defined. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Relevant sections:
The text was updated successfully, but these errors were encountered: