Skip to content
This repository has been archived by the owner on May 10, 2019. It is now read-only.

gzip compress include.js #1115

Closed
lookfirst opened this issue Feb 14, 2012 · 5 comments
Closed

gzip compress include.js #1115

lookfirst opened this issue Feb 14, 2012 · 5 comments

Comments

@lookfirst
Copy link

This is a re-opening of #207 as it seems to have regressed. I'd expect a Content-Encoding header in there.

foo

PageSpeed is reporting:

bar

Load of the FB js in the same session (showing correct headers):

ack

@ghost ghost assigned fetep Feb 14, 2012
@lloyd
Copy link
Contributor

lloyd commented Feb 14, 2012

nice find, @lookfirst.

@fetep, handle this at the nginx level? something like https://gist.github.com/1830700

@lloyd
Copy link
Contributor

lloyd commented Apr 6, 2012

bump. we should fix this, related to issue #1351

@lloyd
Copy link
Contributor

lloyd commented Apr 30, 2012

this has not been forgotten.

this fix should roll to production with train-2012.04.27 - on or around may 9th. see https://bugzilla.mozilla.org/749756

@lloyd
Copy link
Contributor

lloyd commented Jul 7, 2012

verified. done a while ago.

@lloyd lloyd closed this as completed Jul 7, 2012
@jbonacci
Copy link
Contributor

Yep. Catching up on older bugs.

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

4 participants