Skip to content
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

add brotli compression support #21

Closed
gruns opened this issue May 10, 2022 · 2 comments · Fixed by #35
Closed

add brotli compression support #21

gruns opened this issue May 10, 2022 · 2 comments · Fixed by #35
Assignees
Labels
enhancement New feature or request v1

Comments

@gruns
Copy link
Contributor

gruns commented May 10, 2022

brotli is widely supported in browsers (https://caniuse.com/brotli) and enough superior to gzip to use it over gzip when supported

see https://docs.nginx.com/nginx/admin-guide/dynamic-modules/brotli/ and http://wiki.centos-webpanel.com/enabling-brotli-compression-on-nginx

there has to be some way to add brotli support without nginx plus.. but if nginx plus keeps being an impediment to our various usage, ie death by a thousand cuts, perhaps its worth migrating to another http server in the future (😢)

@gruns gruns added the enhancement New feature or request label May 10, 2022
@gruns gruns added this to the v0 milestone May 10, 2022
@gruns
Copy link
Contributor Author

gruns commented May 11, 2022

see #22 for the greater context

@DiegoRBaquero DiegoRBaquero removed this from the v0 milestone Jun 6, 2022
@gruns
Copy link
Contributor Author

gruns commented Jul 28, 2022

🎉 @enolal826

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants