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.

Sign up for GitHub

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

Jump to bottom

Http cache #496

Merged
merged 7 commits into from Mar 23, 2017
Merged

Http cache #496

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
b322a93
Rewrite HTTP cache logic
mnot Feb 27, 2017
0b52aa9
minor editorial nits and indentation
annevk Mar 6, 2017
4aa2432
Don't use stale cached response on network err
mnot Mar 8, 2017
2ae9df0
Take out partial content caching.
mnot Mar 14, 2017
7bcec8f
remove credentials from the cache key (for now)
mnot Mar 16, 2017
51179a3
set `response` to `storedResponse`
mnot Mar 23, 2017
edef79f
final nits
annevk Mar 23, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
10 changes: 0 additions & 10 deletions fetch.bs
View file
Open in desktop
Expand Up @@ -3361,11 +3361,6 @@ Range Requests</cite>. [[HTTP-RANGE]] However, this is not widely supported by b
"<a href=https://tools.ietf.org/html/rfc7234#section-4>Constructing Responses from Caches</a>"
chapter of <cite>HTTP Caching</cite> [[!HTTP-CACHING]], if any.

<p>The cache key used to locate a stored response must include the <var>credentials flag</var>.

<p class=note>This ensures that a stored response to a request with credentials is not used to
satisfy a request without credentials.

<li><p>If <var>storedResponse</var> is null, then abort these substeps.

<!-- cache hit -->
Expand Down Expand Up @@ -3446,11 +3441,6 @@ Range Requests</cite>. [[HTTP-RANGE]] However, this is not widely supported by b
"<a href=https://tools.ietf.org/html/rfc7234#section-3>Storing Responses in Caches</a>"
chapter of <cite>HTTP Caching</cite>. [[!HTTP-CACHING]]

<p>The cache key used to store the response must include the <var>credentials flag</var>.

<p class=note>This ensures that a stored response to a request with credentials is not used to
satisfy a request without credentials.

<p class=note>If <var>forwardResponse</var> is a <a>network error</a>, this effectively caches
the network error, which is sometimes known as "negative caching".
</ol>
Expand Down