-
Notifications
You must be signed in to change notification settings - Fork 520
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
Comments
It seems jetty 11 has reached its end of life. |
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. |
That's true. Especially for Jetty 12 they refactored a lot to drop dependency for JavaEE APIs. |
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? |
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. |
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"]
: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.The text was updated successfully, but these errors were encountered: