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.

Sign up for GitHub

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

Jump to bottom

Integrate Jersey 3.1.0-M3 #23845

Merged
merged 4 commits into from
Mar 6, 2022
Merged

Integrate Jersey 3.1.0-M3 #23845

merged 4 commits into from
Mar 6, 2022

Conversation

arjantijms
Copy link
Contributor

Signed-off-by: Arjan Tijms [email protected]

@arjantijms arjantijms added component upgrade A component dependency has been upgraded ee10-component Jakarta EE 10 component labels Mar 5, 2022
@arjantijms arjantijms added this to the 7.0.0 milestone Mar 5, 2022
@arjantijms arjantijms requested a review from dmatej March 5, 2022 23:13
@arjantijms arjantijms self-assigned this Mar 5, 2022
@arjantijms arjantijms requested review from hs536 and jGauravGupta March 6, 2022 12:54
@hs536 hs536 merged commit bacef5b into eclipse-ee4j:master Mar 6, 2022
@hs536
Copy link
Contributor

hs536 commented Mar 6, 2022

a little bit concerned about Jersey still depends on JAXB3:
https://github.com/eclipse-ee4j/jersey/blob/3.1.0-M3/pom.xml#L2196-L2197

@arjantijms
Copy link
Contributor Author

https://github.com/eclipse-ee4j/jersey/blob/3.1.0-M3/pom.xml#L2196-L2197

Good catch, @jansupol any idea why that is?

@arjantijms arjantijms deleted the jersey310M3 branch March 7, 2022 10:42
@jansupol
Copy link
Contributor

@arjantijms
We are too slow, but working on that. One issue is with the Jersey-Jackson module, Jackson has a dependency on JAX-B/3 and possibly the module won't start in the OSGi env with JAX-B/4.

@dmatej
Copy link
Contributor

dmatej commented Mar 17, 2022

Just FYI: eclipse-ee4j/jersey#5006
So if you would be able to work on jackson, we could probably meet on some stable point of compatibility of 4 modules: jersey, grizzly, metro-wsit and jackson ... and finally GlassFish, which is waiting for that here #23843 - tests detected incompatibility :-)

@jansupol
Copy link
Contributor

JAX-B version update is merged. For Jackson, we need to have both JAX-B/3 and JAX-B/4 on classpath. Looks like OSGi is ok with that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hs536 hs536 hs536 approved these changes

@dmatej dmatej Awaiting requested review from dmatej

@jGauravGupta jGauravGupta Awaiting requested review from jGauravGupta

Assignees

@arjantijms arjantijms

Labels
component upgrade A component dependency has been upgraded ee10-component Jakarta EE 10 component
Projects
None yet
Milestone
7.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@arjantijms @hs536 @jansupol @dmatej