-
-
Notifications
You must be signed in to change notification settings - Fork 41
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 Brotli compression #89
Comments
Should be handled upstream first like it was done for gzip matomo-org/matomo#13633. Suggest to open an issue there. |
I meant to enable it in nginx, similar to gzip is done right now (
|
We would need to compile the module as it's not available in its current state. Also I don't think it brings any real benefit against gzip (at most 17% from what I see from different benchmark). |
Oh my bad it's in the alpine: https://git.alpinelinux.org/aports/tree/main/nginx/APKBUILD?h=3.15-stable#n150 |
Let me know if you want to open a PR and make some tests with this sample configuration. |
I think the images need to be built with brotli module, https://github.com/nginxinc/docker-nginx/tree/master/modules (I don't know how to do that though). Regarding the savings, the matomo js file is ~30% smaller when compressed with Brotli. |
I don't know how to enable in your dockerfile, but I have tried the original matomo docker image and it works there without problems (at least I couldn't find any). |
Right now only gzip compression is enabled, perhaps brotli can also be enabled? This significantly reduces matomo.js file size.
The text was updated successfully, but these errors were encountered: