-
Notifications
You must be signed in to change notification settings - Fork 16
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
We should enable Brotli! #24
Comments
We might get it with Akamai without doing any work. |
We used Debian, right? https://blobfolio.com/2017/06/nginx-brotli-on-debian-stretch-without-any-source-monkeying/ has some instructions. Seems it requires installing an external module. Maybe it's better to wait for native Nginx support? |
That involves replacing the default nginx-full package with a package from another maintainer, and it's likely that security updates aren't quite as speedy. So, I'd rather wait until support is enabled by default in Debian, but won't stand in the way of anyone else who wants to try. |
FWIW @domenic @foolip @annevk have used it thoroughly the past year. Currently using in production. Cute content negotiation trick we do with the lib (besides typical Encoding negotiation) is using brötli That being said, below are links to a brief synopsis of what's been worked on with @tmornini @brandondees. and a link to the curation of most of what we need to know (Down to nginx packages I believe). I know my way around an nginx.conf file but understand the brötli much better. However only use it as a build step since the core lib is a tiny static asset. Haven't had experience (yet) with "real time". @tmornini who is superior to me on NGINX may have some thoughts? Am I wildly oversimplifying or something reasonable to take a stab at... Please advise. P.S. 🙏 @annevk thanks for that blobfolio link. I don't believe i have that on the Relative links:
Don't tip the scaleWe actually have a "weight check" to ensure the platform lib never exceeds 1 MTU ethernet frame. (1500OCTETS). 😂 |
This seems more mainstream now; at least, https://docs.nginx.com/nginx/admin-guide/dynamic-modules/brotli/ does not mention installing any non-default package sources. |
We are still on Debian 9 and nginx 1.10.3. I've tried creating a new droplet with Debian 10, but it also doesn't have a package called nginx-plus-module-brotli. I think we'd need to add some additional deb repository for this, or build from source. |
Somehow.
Via https://twitter.com/DasSurma/status/924346557864054785
The text was updated successfully, but these errors were encountered: