Skip to content

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Apr 30, 2025

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
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.

Copy link
Member

@wangchao316 wangchao316 left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link

codecov-commenter commented Apr 30, 2025

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
57 checks passed
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
Copy link
Member Author

lhotari commented May 26, 2025

Addresses CVE-2024-13009 too.

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

joakime commented May 28, 2025

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