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

Respect Vary headers for resources #241

Closed
GoogleCodeExporter opened this Issue Apr 6, 2015 · 3 comments

Comments

Projects
None yet
1 participant
@GoogleCodeExporter

GoogleCodeExporter commented Apr 6, 2015

Currently mod_pagespeed will rewrite & cache resources with Vary:User-Agent and 
Vary:Referer headers.  It probably shouldn't.

An alternative strategy would be to put the user-agent into the cache-key, but 
I'm afraid this will add a lot of cache pressure, in terms of file-system use 
and capacity.

Original issue reported on code.google.com by jmara...@google.com on 14 Mar 2011 at 9:33

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 6, 2015

Original comment by nfor...@google.com on 24 May 2011 at 5:49

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 6, 2015

I believe this has been fixed: nforman, can you verify and mark closed if so?

Original comment by jmaes...@google.com on 15 Jul 2011 at 1:56

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 6, 2015

If RespectVary is on, then yes, it's fixed.

Original comment by nfor...@google.com on 15 Jul 2011 at 2:12

  • Changed state: Fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment