Skip to content

[fix][sec] Upgrade Jetty to 9.4.57.v20241219 to mitigate CVE-2024-6763 - #24232

Merged
lhotari merged 1 commit into
apache:masterfrom
lhotari:lh-address-CVE-2024-6763
Apr 30, 2025
Merged

[fix][sec] Upgrade Jetty to 9.4.57.v20241219 to mitigate CVE-2024-6763#24232
lhotari merged 1 commit into
apache:masterfrom
lhotari:lh-address-CVE-2024-6763

Conversation

@lhotari

@lhotari lhotari commented Apr 30, 2025

Copy link
Copy Markdown
Member

Fixes #24114

Motivation & Modifications

Upgrade Jetty to 9.4.57.v20241219 to address CVE-2024-6763
Jetty 9.4.57.v20241219 contains backported CVE-2024-6763 fix in jetty/jetty.project#12532 although it's not explicitly mentioned and most security scanners don't yet contain the information that it's been addressed in 9.4.57.
More details:

Note: The backport is a partial mitigation and Jetty 9.4.57 will continue to be marked as vulnerable. There's a discussion and explanation here: https://gitlab.eclipse.org/security/cve-assignement/-/issues/25#note_2968611

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@lhotari lhotari added this to the 4.1.0 milestone Apr 30, 2025
@lhotari lhotari self-assigned this Apr 30, 2025
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Apr 30, 2025
@lhotari lhotari changed the title [fix][sec] Upgrade Jetty to 9.4.57.v20241219 to address CVE-2024-6763 [fix][sec] Upgrade Jetty to 9.4.57.v20241219 to mitigate CVE-2024-6763 Apr 30, 2025
@wangchao316

Copy link
Copy Markdown
Member

hello , I am a new developer for pulsar, our product start to use pulsar, which replace kafka in serverless env. I can learn your pr. our other product has upgraded jetty to 9.4.57.v20241219. this pr is good.

@wangchao316 wangchao316 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov-commenter

codecov-commenter commented Apr 30, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.19%. Comparing base (bbc6224) to head (d12d6ee).
Report is 1057 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #24232      +/-   ##
============================================
+ Coverage     73.57%   74.19%   +0.62%     
+ Complexity    32624    32557      -67     
============================================
  Files          1877     1866      -11     
  Lines        139502   144900    +5398     
  Branches      15299    16549    +1250     
============================================
+ Hits         102638   107515    +4877     
+ Misses        28908    28871      -37     
- Partials       7956     8514     +558     
Flag Coverage Δ
inttests 26.86% <ø> (+2.28%) ⬆️
systests 23.30% <ø> (-1.03%) ⬇️
unittests 73.67% <ø> (+0.83%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1079 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lhotari
lhotari merged commit 611dc3f into apache:master Apr 30, 2025
lhotari added a commit that referenced this pull request Apr 30, 2025
lhotari added a commit that referenced this pull request Apr 30, 2025
lhotari added a commit that referenced this pull request Apr 30, 2025
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request May 2, 2025
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request May 6, 2025
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request May 7, 2025
manas-ctds pushed a commit to datastax/pulsar that referenced this pull request May 7, 2025
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request May 12, 2025
@lhotari

lhotari commented May 26, 2025

Copy link
Copy Markdown
Member Author

Addresses CVE-2024-13009 too.

nodece pushed a commit to ascentstream/pulsar that referenced this pull request May 28, 2025
@joakime

joakime commented May 28, 2025

Copy link
Copy Markdown

Jetty 9.4.57.v20241219 contains backported CVE-2024-6763 fix in jetty/jetty.project#12532 although it's not explicitly mentioned and most security scanners don't yet contain the information that it's been addressed in 9.4.57.

This is intentional, this is the "Unsupported when Assigned" behavior (a CVE thing), as Jetty 9 is EOL (End of Life)

If you still need javax.servlet namespace support, use the ee8 environment in Jetty 12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] The Jetty version in Pulsar contains CVE-2024-6763

5 participants