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

4.x: Upgrade Jersey to 3.1.1 #5821

Closed
Tracked by #5392
tomas-langer opened this issue Jan 10, 2023 · 4 comments
Closed
Tracked by #5392

4.x: Upgrade Jersey to 3.1.1 #5821

tomas-langer opened this issue Jan 10, 2023 · 4 comments
Assignees
Labels
4.x Version 4.x jax-rs JAX-RS and Jersey related issues MP task To do
Milestone

Comments

@tomas-langer
Copy link
Member

tomas-langer commented Jan 10, 2023

Upgrade Jersey:

  • Upgrade to a version that supports Jakarta 10 (requires 4.x: Upgrade Weld to 5.x #5815)
  • Add JAX-RS TCK to our set of TCK tests to make sure Helidon is compliant (if feasible)
@tomas-langer tomas-langer added task To do MP jax-rs JAX-RS and Jersey related issues 4.x Version 4.x labels Jan 10, 2023
@tomas-langer tomas-langer added this to the 4.0.0 milestone Jan 10, 2023
@barchetta barchetta modified the milestones: 4.0.0, 4.0.0-M1 Jan 26, 2023
@jbescos
Copy link
Member

jbescos commented Feb 6, 2023

TCKs are passing when I exclude the tests that deploy war files, and the signature test.

The problem is that these tcks are not published in maven. Jersey is taking them in this way:
wget -q https://download.eclipse.org/ee4j/jakartaee-tck/jakartaee10/staged/eftl/jakarta-restful-ws-tck-3.1.1.zip -O jakarta-restful-ws-tck-3.1.1.zip

@tomas-langer question: Do we make TCKs to be published in maven repo, or do I make similar approach than Jersey with a maven plugin?.

@tomas-langer tomas-langer changed the title 4.x: Upgrade Jersey to 3.1.0 4.x: Upgrade Jersey to 3.1.1 Feb 6, 2023
@tomas-langer
Copy link
Member Author

Let's wait for the TCK to be available in Maven central (unless they already are).
For certification purposes, we will add a separate job to use the proper TCK zip file.

@tomas-langer
Copy link
Member Author

@jbescos could you please upgrade Jersey to 3.1.1?

@jbescos
Copy link
Member

jbescos commented Feb 8, 2023

Let's wait for the TCK to be available in Maven central (unless they already are). For certification purposes, we will add a separate job to use the proper TCK zip file.

I have more information about this topic.

The jars cannot be published in Maven central because of a license issue, and this is not going to be fixed in EE10.

The bundle (the zip file containing the jars) can be published, and this is in progress. I guess this is the only option we have right now, unless we want to wait for EE11.

@jbescos jbescos closed this as completed Mar 14, 2023
@m0mus m0mus added this to Backlog Aug 12, 2024
@m0mus m0mus moved this to Closed in Backlog Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x Version 4.x jax-rs JAX-RS and Jersey related issues MP task To do
Projects
Archived in project
Development

No branches or pull requests

3 participants