compression
compression copied to clipboard
feat: support brotli
@dougwilson Thank you so much for the quick and detailed review.
For the options
, which solution do you think is better ?
- Try to match current options into brotli options for keeping consistency, this may be harder and risky since the zlip api are very different between zlib-based and brotli based.
- Add a new brotli options field(maybe
options.brotli
) in options of compression and add doc (the same way as node is doing now)
For the test, since brotli is only supported in node 11.7, can we add it in travis ?
Friendly reminder @dougwilson ☺️
Hi there 👋
I would really like to serve my static files with brotli. So are there any plans when this might be merged or can I help out somewhere?
Greetings Sven
Any progress?
Any progress?
This PR is long due. If some one can review it then that would be great.
Unsubscribed, looks like this will be never merged.
Workaround:
Set up nginx proxy in front of expressjs server, as bonus you can also do TLS termination and round-robin balancing.