Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

etag should not be issued for mismatching content. #845

Closed
GoogleCodeExporter opened this issue Apr 6, 2015 · 2 comments
Closed

etag should not be issued for mismatching content. #845

GoogleCodeExporter opened this issue Apr 6, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

When PageSpeed responds to a request for a .pagespeed. URL, and the optimized 
resource is cached, it sends
   Etag: W/"0"
This makes sense because the URL already contains the content hash.  However, 
when the optimized resource is not cached, PageSpeed may send back unoptimized 
content instead to avoid blocking clients waiting for the resource.  In that 
case it uses Cache-Control:max-age=300,private, so that proxies & CDNs don't 
cache the unoptimized response and serve it to other clients.  However, we 
still send:
   Etag: W/"0"
This does not seem appropriate.  The language in 
http://tools.ietf.org/html/draft-ietf-httpbis-p6-cache-25 indicates that it 
won't make the resource public:
   The "private" response directive indicates that the response message
   is intended for a single user and MUST NOT be stored by a shared
   cache. 
But it still doesn't make sense to provide a trivial validation for what is not 
the fully optimized resource content.

Original issue reported on code.google.com by jmara...@google.com on 27 Nov 2013 at 4:22

@GoogleCodeExporter
Copy link
Author

This issue was closed by revision r3650.

Original comment by jmara...@google.com on 27 Nov 2013 at 10:52

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

Original comment by jmara...@google.com on 27 Nov 2013 at 10:53

  • Added labels: Milestone-v31, release-note

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant