Skip to content

Map EE9 Jetty security and WebSocket artifacts to published Jetty 12 modules - #1311

Merged
timtebeek merged 2 commits into
mainfrom
jetty-ee10-coordinates-that-exist
Oct 19, 2026
Merged

timtebeek merged 2 commits into
mainfrom
jetty-ee10-coordinates-that-exist

Conversation

@timtebeek

Copy link
Copy Markdown
Member

Summary

  • JettyUpgradeEE10 mapped three EE9 artifacts to EE10 coordinates that have no Jetty 12 release on Maven Central. Every project using them ended up with an unresolvable POM.
  • Map them to the Jetty 12 core modules that do exist, and keep newVersion: 12.0.x.

Problem

EE9 artifact Previous target Published versions of the previous target
org.eclipse.jetty.ee9:jetty-ee9-security org.eclipse.jetty.ee10:jetty-ee10-security never published
org.eclipse.jetty.ee9.websocket:jetty-ee9-websocket-jetty-api org.eclipse.jetty.ee10.websocket:jetty-ee10-websocket-jetty-api 12.0.0.alpha0..12.0.0.beta0 only
org.eclipse.jetty.ee9.websocket:jetty-ee9-websocket-jetty-client org.eclipse.jetty.ee10.websocket:jetty-ee10-websocket-jetty-client 12.0.0.alpha0..12.0.0.beta2 only

Before this change, the new test produced:

  • jetty-ee10-security failed. Unable to download metadata ... HTTP 404, with jetty-ee9-security left in place.
  • Unable to download POM: ...jetty-ee10-websocket-jetty-api:12.0.x (and the same for -client). Both dependencies were rewritten with the literal 12.0.x, because no matching release exists.
  • Error markers that kept accumulating, so the recipe never settled after one cycle.

Solution

  • jetty-ee9-security → org.eclipse.jetty:jetty-security
  • jetty-ee9-websocket-jetty-api → org.eclipse.jetty.websocket:jetty-websocket-jetty-api
  • jetty-ee9-websocket-jetty-client → org.eclipse.jetty.websocket:jetty-websocket-jetty-client

These were found in a catalog-wide check of dependency-swap steps against Maven Central metadata.

Not addressed here: JettyUpgradeEE10 changes dependencies but not the matching org.eclipse.jetty.ee9.* → org.eclipse.jetty.ee10.* Java packages, apart from one ChangeType.

Test plan

  • New JettyUpgradeEE10Test fails before the change and passes after
  • JettyUpgradeEE9Test and JakartaEE10Test pass

@timtebeek
timtebeek merged commit fd54e3c into main Oct 19, 2026
2 checks passed
@timtebeek
timtebeek deleted the jetty-ee10-coordinates-that-exist branch October 19, 2026 12:35
@github-project-automation github-project-automation Bot moved this from In Progress to Done in OpenRewrite Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant