Skip to content

Conversation

@reta
Copy link
Contributor

@reta reta commented Oct 7, 2025

Fix JDK ServiceLoader manifests due to package changes.

Caused by: java.util.ServiceConfigurationError: jakarta.ws.rs.ext.MessageBodyWriter: Provider com.fasterxml.jackson.jakarta.rs.json.JacksonJsonProvider not found
        at java.base/java.util.ServiceLoader.fail(ServiceLoader.java:559)
        at java.base/java.util.ServiceLoader$LazyClassPathLookupIterator.nextProviderClass(ServiceLoader.java:1090)
        at java.base/java.util.ServiceLoader$LazyClassPathLookupIterator.hasNextService(ServiceLoader.java:1099)
        at java.base/java.util.ServiceLoader$LazyClassPathLookupIterator.hasNext(ServiceLoader.java:1142)
        at java.base/java.util.ServiceLoader$1.hasNext(ServiceLoader.java:1164)
        at java.base/java.util.ServiceLoader$2.hasNext(ServiceLoader.java:1246)
        at org.apache.cxf.cdi.JAXRSCdiResourceExtension.loadExternalProviders(JAXRSCdiResourceE

Version 3.0.0 (applicable to 3.x release line only)

@cowtowncoder
Copy link
Member

Thank you -- looks legit.

Sidenote: probably also needed for https://github.com/FasterXML/jackson-jaxrs-providers.

@cowtowncoder
Copy link
Member

Quick note: before approving and merging -- thank you for contributing this PR! -- there's one piece of process: we need a CLA. It's from here

https://github.com/FasterXML/jackson/blob/main/contributor-agreement.pdf

and the usual way is to print, fill & sign, scan/photo, email to cla at fasterxml dot com.
Once we get that I can proceed.
CLA only needs to be sent once per contributor and one is good for all future contributions.

@reta
Copy link
Contributor Author

reta commented Oct 8, 2025

Sidenote: probably also needed for https://github.com/FasterXML/jackson-jaxrs-providers.

Thank you @cowtowncoder , will do that

-- there's one piece of process: we need a CLA. It's from here

CLA has been signed and sent out, thank you!

@cowtowncoder
Copy link
Member

Great, got CLA, can merge.

It'd be great to have test for SPI, if you (or anyone else) has time & interest.

I'll file an issue as reminder.

@cowtowncoder cowtowncoder merged commit def4b80 into FasterXML:3.x Oct 8, 2025
3 checks passed
@reta
Copy link
Contributor Author

reta commented Oct 8, 2025

It'd be great to have test for SPI, if you (or anyone else) has time & interest.

This is great idea actually, happy to take #58, thank you @cowtowncoder

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants