feat: support for brotli
The changes from #172 are brought in, with the exception of using the accept negotiation logic.
It should use accepts, so this PR is not ready yet, and I don't have the option to turn it into a draft. I will soon open a PR in accepts to be able to use this logic
@bjohansebas @blakeembrey Is there a tentative date for merging this?
@sardeeplakhera I hope it can be merged soon, although there’s still some time needed to ensure that everything works correctly.
Thanks, @bjohansebas! Do you think first week of December could be a realistic target for the release? Or is there anything else that might push it back further?
@sardeeplakhera i think a new patch version for this package could be ready by December. Currently, this work is more like a proposal for version 1.8.0 (see #189)
Thank you the maintenance team! Glad to see it finally gets some progress.
hi @wesleytodd @blakeembrey, I've updated the options to avoid using object-assign, would love to hear your feedback
I really don't think there are any more changes to be made here.
~~Now the tests are going to fail due to an issue with decoding. I’m going to investigate why it’s not decoding properly, this problem was probably always there.~~
~~I think it's an issue with supertest and the way it decodes.~~
It works great in browsers.
@wesleytodd any chance you have the time to review it once again so it could be merged?
@bjohansebas @blakeembrey any idea when this PR will be merged and new version will be released?
Thanks to everyone who made this possible
@nicksrandall @danielgindi @bjohansebas This feature will save a huge chunk of the overall internet data transfers thanks to the widespread use of Express and 20 million weekly downloads of this package. Kudos for the efforts you've put in to enable this 🎉