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

Bump Jetty deps from 11.x.x to 12.x.x #512

Open
metametadata opened this issue Oct 24, 2024 · 5 comments
Open

Bump Jetty deps from 11.x.x to 12.x.x #512

metametadata opened this issue Oct 24, 2024 · 5 comments

Comments

@metametadata
Copy link

Would be great to bump Jetty so that no vulns are reported. Specifically, I currently get this report for [ring/ring-jetty-adapter "1.13.0"]:

NAME        INSTALLED  FIXED-IN  TYPE          VULNERABILITY        SEVERITY
jetty-http  11.0.24    12.0.12   java-archive  GHSA-qh8g-58pp-2wxh  Low

I could not quickly override the Ring's Jetty deps in my projects as the structure of Jetty repos changed in 12.0.0. See org.eclipse.jetty.ee{8,9,10} new coordinates in https://jetty.org/docs/jetty/12/programming-guide/migration/11-to-12.html.

@sunng87
Copy link

sunng87 commented Oct 24, 2024

It seems jetty 11 has reached its end of life.
You can switch to rj9a for now for latest jetty https://github.com/sunng87/ring-jetty9-adapter .

@weavejester
Copy link
Member

Jetty tends to change around its API every version, so unfortunately we can't just update the artifacts and expect Jetty to still work. It'll likely require a fair bit of effort, as I'm sure @sunng87 can attest.

@sunng87
Copy link

sunng87 commented Oct 24, 2024

That's true. Especially for Jetty 12 they refactored a lot to drop dependency for JavaEE APIs.

@jumarko
Copy link

jumarko commented Nov 27, 2024

I got here because of a docker image scan failure reported to me, specifically for jetty-http: https://scout.docker.com/vulnerabilities/id/CVE-2024-6763?s=github&n=jetty-http&ns=org.eclipse.jetty&t=maven&vr=%3E%3D7.0.0%2C%3C%3D12.0.11

Is there a plan to switch to jetty 12.x anytime soon?

@weavejester
Copy link
Member

It'll be updated as soon as I, or anyone else, gets the time to do so. That probably means sometime in December.

As far as I'm aware, the reported vulnerabilities in Jetty 11 do not affect Ring.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants