Skip to content
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

Broken http response when using http trailers with gzip compression for chunked response. #12904

Closed
Zizou84 opened this issue Mar 13, 2025 · 4 comments
Labels
Bug For general bugs on Jetty side Unsupported Release For releases that are no longer supported

Comments

@Zizou84
Copy link

Zizou84 commented Mar 13, 2025

Jetty version(s)
10.0.15
Jetty Environment

Java version/vendor (use: java -version)

OS type/version

Description
Broken http response when using http trailers with gzip compression for chunked response.

I have build simple maven project using maven-jetty-plugin to reporoduce the issue.
Have cerate a REST endpoint which return a Stream
Added MessageBodyWriter for handling Stream and inejcting HttpservletResponse for adding http trailers to be able to report to my client any issue that happen in the middle of processing

When I'm setting I'm adding a tarilers and enabling gzip compression, my client is crashing with error "CRLF expected at end of the chunk"
I have even tested using curl and getting same issue(curl: (18) transfer closed with outstanding read data remaining).

When I don't send trailers or I disable compression(by using Accept-Encoding in client) every thing work correctly.

@Zizou84 Zizou84 added the Bug For general bugs on Jetty side label Mar 13, 2025
@joakime
Copy link
Contributor

joakime commented Mar 13, 2025

Jetty 10 is at End of Community Support.

Upgrade to a supported version of Jetty and try again.
At this point in time, Jetty 12.0.x is the supported version of Jetty.

@joakime joakime added the Unsupported Release For releases that are no longer supported label Mar 13, 2025
@gregw
Copy link
Contributor

gregw commented Mar 14, 2025

@Zizou84 I saw your comment in email: "I have tested with jetty 12 (ee8) and getting same issue", but it is not here?

If you have reproduced on jetty-12, can you give us access to the maven project you have that reproduces the issue?

@Zizou84
Copy link
Author

Zizou84 commented Mar 20, 2025

@gregw Sorry I thought I have already posted my reply.
It was a wrong test, compression is working fine with Jetty 12

@sbordet
Copy link
Contributor

sbordet commented Mar 20, 2025

Closing as not an issue.

@sbordet sbordet closed this as not planned Won't fix, can't repro, duplicate, stale Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For general bugs on Jetty side Unsupported Release For releases that are no longer supported
Projects
None yet
Development

No branches or pull requests

4 participants