-
Notifications
You must be signed in to change notification settings - Fork 633
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 test
#2815
Conversation
33ae09e
to
79aa324
Compare
reactor-netty-http/src/main/java/reactor/netty/http/server/SimpleCompressionHandler.java
Outdated
Show resolved
Hide resolved
0f5cb25
to
2eb7cf2
Compare
fyi @hyperxpro |
@violetagg Thanks for the feedback! I incorporated your suggestions and I squashed the commits. |
2eb7cf2
to
24aa10e
Compare
b03c1ca
to
83585a2
Compare
The failure on Windows OS is a flaky test |
@reactor/netty-team PTAL |
Please hold off on this PR until I release 1.12.0 in the next 2 days. |
sure |
83585a2
to
3e900d7
Compare
fyi, I upgraded this PR to brotli4j 1.12.0 |
LGTM :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds good to me, thanks for the PR @sullis.
The change is back ported to |
Note: Netty 4.x supports Brotli compression. Brotli compression is available if and only if the Brotli4j library is on the runtime classpath.