[LIVY-1077] Bump jetty.version to latest 9.4.x (9.4.58.v20250814) - #551
Merged
Merged
Conversation
pom.xml currently pins jetty.version to 9.4.56.v20240826, which is affected by CVE-2026-2332 / GHSA-355h-qmc2-wpwf (HTTP/1.1 request smuggling via malformed chunk-extension quoted-string parsing in Jetty's chunked transfer-encoding parser, CVSS 7.4 High, CWE-444). Per OSV/GHSA-355h-qmc2-wpwf and Maven Central directly: the actual fix for this CVE on the Jetty 9.4.x branch is version 9.4.60, but 9.4.59/9.4.60 have never been published to Maven Central -- the upstream Jetty advisory itself still reads "No patches yet" for that branch. The only published fixes are on Jetty 10.x/11.x/12.x, which require Java 11+ (12.x requires Java 17+ and moves javax.servlet -> jakarta.servlet), a real migration rather than a version-property bump. Bumps jetty.version to 9.4.58.v20250814, the latest version actually published on the 9.4.x branch, as defense-in-depth (picks up every other 9.4.x security fix since 9.4.56). CVE-2026-2332 itself remains open pending an upstream Jetty 9.4.x patch release, or a separately-scoped move to Jetty 10+/Java 11+.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #551 +/- ##
============================================
+ Coverage 68.68% 68.83% +0.14%
- Complexity 1218 1219 +1
============================================
Files 106 107 +1
Lines 6815 6873 +58
Branches 836 839 +3
============================================
+ Hits 4681 4731 +50
- Misses 1666 1674 +8
Partials 468 468 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Contributor
|
Thank you for your PR @soumyadeeplogin ! As mentioned in #480 , Jetty 9.4 is the last version compatible with Java 8, thus we are stuck with it as long as we want to retain Java 8 / Spark 3.5 support. It is still good to have the latest publicly available version though, so once again thanks for opening a PR for it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this pull request?
Bumps
jetty.versioninpom.xmlfrom9.4.56.v20240826to9.4.58.v20250814-- the latest version actually published on the Jetty 9.4.x branch.The current
jetty-http/jetty-server/jetty-securityversion is affected by CVE-2026-2332 / GHSA-355h-qmc2-wpwf -- an HTTP/1.1 request smuggling issue via malformed chunk-extension quoted-string parsing in Jetty's chunked transfer-encoding parser (CVSS 7.4 High, CWE-444).Checked against the advisory (OSV) and Maven Central directly: the actual fix for this CVE on the 9.4.x branch is version
9.4.60, but9.4.59/9.4.60have never been published to Maven Central -- the upstream advisory itself still reads "No patches yet" for that branch as of this writing. The only published fixes are on Jetty 10.x/11.x/12.x, all of which require Java 11+ (12.x requires Java 17+ and movesjavax.servlet->jakarta.servlet) -- a real migration, not a version-property bump, and out of scope for this PR.This bumps to the latest available 9.4.x release as defense-in-depth, picking up every other 9.4.x security fix since 9.4.56. CVE-2026-2332 itself remains open against
jetty-httppending an upstream Jetty 9.4.x patch release, or a separately-scoped move to Jetty 10+/Java 11+.How was this patch tested?
mvn dependency:treeresolves cleanly to9.4.58.v20250814across every module that pulls injetty.version(server,assembly,client-http,thriftserver), with no dependency conflicts.mvn compileon those modules succeeds with no errors. This is a version-property change only, no code paths are touched, so no new unit tests are needed.Was this patch authored or co-authored using generative AI tooling?
Yes. Generated-by: Claude Code (Sonnet 5, Anthropic), used to research the CVE against Maven Central/OSV/GHSA data, draft this change, and file the companion JIRA ticket, under human review. Please refer to the ASF Generative Tooling Guidance for details.
Note on JIRA
Filed as LIVY-1077.