WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
REOPENED
170714
HTTP cache does not honor a number of cache-control request directives
https://bugs.webkit.org/show_bug.cgi?id=170714
Summary
HTTP cache does not honor a number of cache-control request directives
Mark Nottingham
Reported
2017-04-10 21:48:48 PDT
Safari Tech Preview's HTTP cache does not honour a number of cache-control request directives: * If a request contains Cache-Control: max-age with a value other than "0", Safari TP will reuse a stored response, even when the stored response is older * If a request contains Cache-Control: max-age and the response contains an Age header that is greater than the requested value, Safari TP will reuse the cached response * If a request contains Cache-Control: min-fresh, Safari TP will not honour its value * If a request contains Cache-Control: no-store, Safari TP will still use a stored response * If a request contains Cache-Control: only-if-cached, Safari TP will not generate a 504 response if it is not stored. For tests, see:
http://w3c-test.org/fetch/http-cache/cc-request.html
Probably the most concerning issue here is the lack of support for Cache-Control: no-store.
Attachments
Add attachment
proposed patch, testcase, etc.
Sam Sneddon [:gsnedders]
Comment 1
2022-05-19 10:13:04 PDT
(In reply to Mark Nottingham from
comment #0
)
> For tests, see: >
http://w3c-test.org/fetch/http-cache/cc-request.html
Now
http://wpt.live/fetch/http-cache/cc-request.any.html
Radar WebKit Bug Importer
Comment 2
2022-05-19 10:13:13 PDT
<
rdar://problem/93585414
>
Sam Sneddon [:gsnedders]
Comment 3
2022-05-19 10:17:49 PDT
-> into Radar, given cache behaviour of Safari is owned by CFNetwork, and not WebKit.
Chris Dumez
Comment 4
2022-05-19 10:19:13 PDT
(In reply to Sam Sneddon [:gsnedders] from
comment #3
)
> -> into Radar, given cache behaviour of Safari is owned by CFNetwork, and > not WebKit.
That's not true. WebKit doesn't use CFNetwork for HTTP caching.
Mark Nottingham
Comment 5
2022-05-19 15:26:55 PDT
In the meantime - we've relaxed the requirement for caches to honour these:
https://httpwg.org/http-core/draft-ietf-httpbis-cache-latest.html#cache-request-directive
(that will be published as an RFC any day now) See tests also at:
https://cache-tests.fyi/#cc-request
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug