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
Update: Add snappy support on HttpContentDecoder #13312
Update: Add snappy support on HttpContentDecoder #13312
Conversation
Motivation: I want to allow logstash http input to be able to read snappy compressed data. Modifications: To allow the HttpContentDecoder to support snappy it created a new control flow to test for snappy content encoding. If it is present, we are using a specific snappy decompressor to read the data. In order to make sure it was working I also included a e2e test on the behaior of HttpContentDecoder. Result: We should be able to send snappy traffic to a server using HttpContentDocder and be ale to uncompresse the traffic.
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.
Fix checkstyle.
codec-http/src/main/java/io/netty/handler/codec/http/HttpContentDecompressor.java
Outdated
Show resolved
Hide resolved
codec-http/src/main/java/io/netty/handler/codec/http/HttpHeaderValues.java
Outdated
Show resolved
Hide resolved
codec-http/src/test/java/io/netty/handler/codec/http/HttpContentDecoderTest.java
Outdated
Show resolved
Hide resolved
codec-http/src/test/java/io/netty/handler/codec/http/HttpContentDecoderTest.java
Show resolved
Hide resolved
codec-http/src/test/java/io/netty/handler/codec/http/HttpContentDecoderTest.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Aayush Atharva <hyperx.pro@outlook.com>
codec-http/src/main/java/io/netty/handler/codec/http/HttpContentDecompressor.java
Outdated
Show resolved
Hide resolved
…ntDecompressor.java
@redcinelli can you please sign our ICLA: https://netty.io/s/icla ? |
We also need this for Http2. |
Hi @normanmaurer, I believe I had done it already. I have re-done it anyway just to make sure. 👌 |
Motivation: I want to allow logstash http input to be able to read snappy compressed data. Modifications: To allow the HttpContentDecoder to support snappy it created a new control flow to test for snappy content encoding. If it is present, we are using a specific snappy decompressor to read the data. In order to make sure it was working I also included a e2e test on the behaior of HttpContentDecoder. Result: We should be able to send snappy traffic to a server using HttpContentDocder and be ale to uncompresse the traffic. Co-authored-by: Aayush Atharva <hyperx.pro@outlook.com> Co-authored-by: Norman Maurer <norman_maurer@apple.com>
Motivation:
I want to allow logstash http input to be able to read snappy compressed data.
Modifications:
To allow the HttpContentDecoder to support snappy it created a new control flow to test for snappy content encoding.
If it is present, we are using a specific snappy decompressor to read the data.
In order to make sure it was working I also included a e2e test on the behaviour of HttpContentDecoder.
Result:
We should be able to send snappy traffic to a server using HttpContentDocder and be ale to uncompress the traffic.
Fixes #13307