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

Add Vary: Accept-Encoding header to .css/.js #68

Closed
jdorfman opened this issue Nov 30, 2012 · 4 comments
Closed

Add Vary: Accept-Encoding header to .css/.js #68

jdorfman opened this issue Nov 30, 2012 · 4 comments
Assignees
@jdorfman
Labels
enhancement

Comments

@jdorfman
Copy link
Member

Reported by: DLongley - https://plus.google.com/105970376423479621367/posts

<IfModule mod_headers.c>
  <FilesMatch "\.(js|css|xml|gz)$">
    Header append Vary: Accept-Encoding
  </FilesMatch>
</IfModule>
@ghost ghost assigned jdorfman Nov 30, 2012
@jdorfman
Copy link
Member Author

http://stackoverflow.com/questions/14540490/is-vary-accept-encoding-overkill

jdorfman added a commit that referenced this issue Jan 28, 2013
@jdorfman
vary important addition to .htaccess for issue #68
fd43ff8
@jdorfman
Copy link
Member Author

all done, blog post to follow. Will deploy asap.

jdorfman added a commit that referenced this issue Jan 28, 2013
@jdorfman
Merge pull request #94 from netdna/2.2.x
a5a8d18 
vary important addition to .htaccess for issue #68
@jdorfman
Copy link
Member Author 

curl -sI "netdna.bootstrapcdn.com/twitter-bootstrap/2.2.2/css/bootstrap-combined.min.css" |grep Vary
Vary: Accept-Encoding

curl -sI "netdna.bootstrapcdn.com/twitter-bootstrap/2.2.2/js/bootstrap.min.js" |grep Vary
Vary: Accept-Encoding

@jdorfman
Copy link
Member Author

jdorfman commented Feb 5, 2013

http://blog.netdna.com/opensource/bootstrapcdn/accept-encoding-its-vary-important/

jdorfman added a commit that referenced this issue Feb 24, 2015
@jdorfman
vary important addition to .htaccess for issue #68
5d24cf6
jdorfman added a commit that referenced this issue Feb 24, 2015
@jdorfman
Merge pull request #94 from netdna/2.2.x
536006e 
vary important addition to .htaccess for issue #68
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdorfman jdorfman

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jdorfman