-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Introduce CompressionHandler to support compression from gzip, brotli, and zstandard #8769
Comments
This issue has been automatically marked as stale because it has been a |
When we do this we need to keep in mind the effort for Compression Dictionary Transport. |
@joakime lets start working on this in the 12.1.x branch |
I'm working on this now. |
I've got the basics of the Compression API done. What's left ...
|
@gregw lets have a chat about the remaining tasks. |
Merged PR #12075 |
Jetty version(s)
Jetty 12
Enhancement Description
The current
GzipHandler
does a good job at handlingContent-Encoding: gzip
(andAccept-Encoding: gzip
), but we shouldn't hardcode only gzip support.Can we refactor the existing
GzipHandler
to be more generic, something likeDynamicCompressionHandler
and have the Gzip support (courtesy of the existingGzipRequest
object) just be loaded via a ServiceLoader?This way we can encourage other compression algorithms under the same
Accept-Encoding
/Content-Encoding
/ already compressed rules that the existing codebase handles for us?Other compression algorithms that have been requested in our issue tracker.
application/zstd
Media Type(FYI,
curl
supports both of these compression types now)The text was updated successfully, but these errors were encountered: