-
Notifications
You must be signed in to change notification settings - Fork 36
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
Evaluate and add brotli #145
Comments
In my own research, I can setup our build process to also produce |
this pretty low priority, especially if you aren't confident in it. |
@navdeepsinghkhalsa What can we do to make this happen? We can produce brotli JS assets at build step (webpack plugin) to begin with. |
@bogas04 That's great that you are ready for this at the front-end. Just need to evaluate if it's something we want to do. I will take a look after this sprint closes. |
20% better compression at no cost. Just that we need to pre-build the files. |
@navdeepsinghkhalsa do we still want to do this? please re-assign as needed. |
This ticket still valid @ManjotS |
@tsingh777 please let us know |
ping @tsingh777 |
Would like to see if expressjs/compression#172 gets accepted. keep it open at low priority. |
This needs more discussion/research and might be even avoidable if we opt for next.js or any other framework. |
https://certsimple.com/blog/nginx-brotli
The text was updated successfully, but these errors were encountered: