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

CAMEL-19481: Upgrade to Jetty 10 #10467

Closed
wants to merge 1 commit into from

Conversation

essobedo
Copy link
Contributor

@essobedo essobedo commented Jun 21, 2023

Fix for https://issues.apache.org/jira/browse/CAMEL-19481

Motivation

Jetty 9 has reached its EOL see jetty/jetty.project#7958, thus we need to upgrade to at least Jetty 10

Modifications

  • Upgrades camel-cometd, camel-jetty, camel-resteasy, camel-salesforce, camel-undertow and camel-websocket to Jetty 10 but keep Jetty 9 for camel-atmosphere-websocket that is not compatible
  • Adapts the camel-test-infra-jetty to support Jetty 9 and 10

@github-actions
Copy link
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟

⚠️ Please note that the changes on this PR may be tested automatically.

If necessary Apache Camel Committers may access logs and test results in the job summaries!

@github-actions
Copy link
Contributor

Components test results:

Total Tested Failed ❌ Passed ✅
9 9 2 7

@github-actions
Copy link
Contributor

🚫 There are (likely) no changes in core core to be tested in this PR

@github-actions
Copy link
Contributor

Components test results:

Total Tested Failed ❌ Passed ✅
9 9 2 7

@github-actions
Copy link
Contributor

🚫 There are (likely) no changes in core core to be tested in this PR

@essobedo essobedo force-pushed the CAMEL-19481/upgrade-jetty-10 branch from 74ded78 to 2f507b8 Compare June 22, 2023 08:47
@github-actions
Copy link
Contributor

Components test results:

Total Tested Failed ❌ Passed ✅
9 9 0 9

@github-actions
Copy link
Contributor

🚫 There are (likely) no changes in core core to be tested in this PR

@davsclaus
Copy link
Contributor

Is there anything from the Karaf community whether they will do a Karaf release that supports Jetty 10 ?

@essobedo
Copy link
Contributor Author

essobedo commented Jun 22, 2023

It is still under discussion but it is doable even in the next Karaf 4.4.x which will be released very soon but not before 3.21. So the question is do we make users wait 6 months to get this upgrade to have it included into 3.22 or can we include it directly in 3.21 with a note indicating the limitation that will be removed once the next Karaf version will be released in a couple of weeks?

@davsclaus
Copy link
Contributor

We keep 3.21 stable as-is - people dont casually upgrade karaf.
We are to build the release tomorrow.

@davsclaus
Copy link
Contributor

And we can do a 3.22.0 release sooner if we think its ready and community users would benefit from Karaf 4.5.x and Jetty 10 support

@essobedo
Copy link
Contributor Author

Ok let's merge it once a Karaf version supporting Jetty 10 is released

@github-actions
Copy link
Contributor

Components test results:

Total Tested Failed ❌ Passed ✅
9 9 1 8

@github-actions
Copy link
Contributor

🚫 There are (likely) no changes in core core to be tested in this PR

@davsclaus
Copy link
Contributor

camel 3.22 is being released now as-is, and there is no new 3.x releases planned

@davsclaus davsclaus closed this Dec 20, 2023
@davsclaus davsclaus deleted the CAMEL-19481/upgrade-jetty-10 branch July 7, 2024 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants