-
Notifications
You must be signed in to change notification settings - Fork 600
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
Provide Brotli compression #62
Comments
We were recently addressing an issue with gzip on the CDN and discovered that gzip performs better than brotli for most jQuery files. Perhaps this issue can be closed? |
This was the default Brotli that Fastly provided IIRC? Perhaps it’s just using poor settings. Proper Brotli would be way smaller than that. It makes sense to do this IMO but we’d have to provide the Brotli versions by ourselves. |
It doesn't say what compression level is used, but Brotli does require more resources so I could see Fastly keeping the level low to avoid that. I'd rather not get into providing Brotli versions ourselves, but perhaps we could play with nginx settings so brotli is supported at the origin. But, it could have the same issues as we had with gzip where the origin is not always consulted, in which case it may end up using Fastly's gzip version anyway. |
Split from #38.
The text was updated successfully, but these errors were encountered: