-
Notifications
You must be signed in to change notification settings - Fork 13
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
Enable gzip compression and browser caching of resources #890
Comments
@tsteiner what would it take to get these issues addressed? It seems fairly easy, but I'm not sure how easy it would be with the current architecture. |
Silly nginx: when you turn on gzip, it only compresses text/html content. I've told it to compress everything now.
Applied on http://unlcms-dr.autosplit.its-sys.unl.edu/. If everything looks ok, I'll do an RFC and get it on production. |
Sweet. Thank you @tsteiner Do these changes also include browser caching of resources, or just gzip? |
How long do you want wdn files cached? |
Would it be possible to coincide the cache expiration with a regular release timeline or RFC process? |
I like @saltybeagle's idea, but I don't know if it is practical in a server configuration. Optimally, |
The HTML5 Boilerplate comes with a wealth of good server configuration options. |
@kabel are you suggesting not caching until we resolve the issue or only caching for a short time (~1 day)? |
No, I would think that one month (our release cycle) is just fine. |
Yes, good enough. We've gotta start somewhere and I don't know enough about the varnish/nginx combo to know if we're shooting ourselves in the foot when it comes to cache busting. |
Config updated on http://unlcms-dr.autosplit.its-sys.unl.edu/ . Let me know if/when to set up an RFC to push to production. |
This appears to be working for me using curl, but not when changing my 1: using curl:
2: changing my This is what I added to my hosts file
|
I'm not sure. I made the same changes to my hosts file and everything worked as expected:
|
Works for me too, albeit, not with the IPs you had in your host file as those appear to be the production IPs. |
Looks like a DR server to me |
Ah, good point, read right over those extra letters. 👊 |
I blame my browser, it appears to be working now. However, I did notice that |
Varnish is now setup to compress any uncompressed content of MIME type text/* (on the DR servers) |
see https://developers.google.com/speed/pagespeed/insights/?url=http%3A%2F%2Fwdn.unl.edu%2F&tab=mobile
This is currently an issue for many framework provided assets and should be fixed.
The text was updated successfully, but these errors were encountered: