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

Rio JSONLD module should not register GlassFish JsonProviderImpl #5233

Open
jetztgradnet opened this issue Jan 15, 2025 · 0 comments
Open

Rio JSONLD module should not register GlassFish JsonProviderImpl #5233

jetztgradnet opened this issue Jan 15, 2025 · 0 comments
Labels
🐞 bug issue is a bug

Comments

@jetztgradnet
Copy link

jetztgradnet commented Jan 15, 2025

Current Behavior

the rdf4j-rio-jsonld module registers org.glassfish.json.JsonProviderImpl in its META-INF/services/jakarta.json.spi.JsonProvider (see source file).

As this module does not actually provide that implementation, this is bad and unexpected behavior, especially as GlassFish JsonProviderImpl is the legacy implementation and has been superseded by Eclipse Parrson, which has a different classname org.eclipse.parsson.JsonProviderImpl (see Parrson's version of META-INF/services/jakarta.json.spi.JsonProvider).

As a result, I need to continue using the obsolete dependency of implementation org.glassfish:jakarta.json:2.0.1 as only that actually provides the implementation class org.glassfish.json.JsonProviderImpl.

As a consequence, I get exceptions which are caused by this:

Caused by: java.util.ServiceConfigurationError: jakarta.json.spi.JsonProvider: Provider org.glassfish.json.JsonProviderImpl not found

Please note that the jsonld-legacy module does not expose the same behavior, there is no registration for META-INF/services/jakarta.json.spi.JsonProvider.

Apparently this behavior was added as part of GH-5041.

Expected Behavior

RIO JSONLD should not pre-register any JSON provider, leaving me free to choose any implementation.

Steps To Reproduce

No response

Version

5.1.0

Are you interested in contributing a solution yourself?

None

Anything else?

No response

@jetztgradnet jetztgradnet added the 🐞 bug issue is a bug label Jan 15, 2025
@jetztgradnet jetztgradnet changed the title RIO jsonld module registers GlassFish JsonProviderImpl Rio JSONLD module should not register GlassFish JsonProviderImpl Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug issue is a bug
Projects
None yet
Development

No branches or pull requests

1 participant