-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Implement brotli everywhere #2518
Comments
btw I am not sure if |
Is it a good idea to brotli-compress dynamic content in runtime? |
Maybe, but if should be off by default and controlled by user. Like we already do for gzip. |
From what I found it should faster and smaller than gzip on 4-5 compression levels |
Done |
Should this be closed? The linked PR was never merged and I don't see any documentation regarding the server supporting Brotli. |
Thanks. Can the old PR be revived? If not, an incremental change to at least serve |
It was closed by the author, you are welcome to fork their branch and open a new one. |
Now brotli is supported on client side for reading brotli compressed responses.
We need to support it on other sides: sending client request, reading request on server and sending server response, multipart maybe.
See the difference between
brotly
andzlib
usage.Ideally
brotly
should be supported everywhere we do supportgzip
byzlib
library usage.The text was updated successfully, but these errors were encountered: