-
Notifications
You must be signed in to change notification settings - Fork 128
Unoptimized gzip and cache settings #3202
Comments
@relud is turning on gzip a switch you can flip? |
yeah, there's an nginx setting for it. what content-types should we compress? I have to specify each of them, except for the settings look like this: https://github.com/mozilla-services/cloudops-deployment/blob/master/projects/normandy/puppet/modules/normandy/files/http_normandy_root_location.conf#L21-L23 |
this should enable it, on the types i listed above: https://github.com/mozilla-services/cloudops-deployment/pull/1166 |
We shouldn't have compression on |
okay, i updated the list to |
okay, mozilla-services/cloudops-deployment#1166 is merged, so the next to deploys to dev/stage will pick up this change. |
This is relatively safe since we use commit-hash-based cache busters on URLs. This fixes one of the last issues from #3202
This is relatively safe since we use commit-hash-based cache busters on URLs. This fixes one of the last issues from #3202
More details on https://www.webpagetest.org/result/170724_VE_23RP/1/performance_optimization/
Missing gzip: https://screenshots.firefox.com/static/img/spinner.svg?rev=8.1.0-465-gcce97ba
Caching can be extended to multiple days for most static resources, given that the
rev
is in the query (to some, why not to all?):The text was updated successfully, but these errors were encountered: