eclipse-ee4j / eclipse-ee4j/jersey

Jersey missing explicit dependency on javax.annotation

Open
#3,767 5 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
730
Forks
382
PR merge metrics
No merged PRs in 30d

Description

When using Jersey client under Java 9 I get:

```
java.lang.NoClassDefFoundError: javax/annotation/Priority
at org.glassfish.jersey.model.internal.RankedProvider.computeRank(RankedProvider.java:111)
at org.glassfish.jersey.model.internal.RankedProvider.(RankedProvider.java:72)
at org.glassfish.jersey.internal.inject.Injections.lookupService(Injections.java:113)
at org.glassfish.jersey.internal.inject.Injections.lookupInjectionManagerFactory(Injections.java:97)
at org.glassfish.jersey.internal.inject.Injections.createInjectionManager(Injections.java:68)
at org.glassfish.jersey.client.ClientConfig$State.initRuntime(ClientConfig.java:432)
at org.glassfish.jersey.internal.util.collection.Values$LazyValueImpl.get(Values.java:341)
at org.glassfish.jersey.client.ClientConfig.getRuntime(ClientConfig.java:826)
at org.glassfish.jersey.client.ClientRequest.getConfiguration(ClientRequest.java:285)
at org.glassfish.jersey.client.JerseyInvocation.validateHttpMethodAndEntity(JerseyInvocation.java:143)
at org.glassfish.jersey.client.JerseyInvocation.(JerseyInvocation.java:112)
at org.glassfish.jersey.client.JerseyInvocation.(JerseyInvocation.java:108)
at org.glassfish.jersey.client.JerseyInvocation.(JerseyInvocation.java:99)
at org.glassfish.jersey.client.JerseyInvocation$Builder.method(JerseyInvocation.java:445)
at org.glassfish.jersey.client.JerseyInvocation$Builder.post(JerseyInvocation.java:351)
```

My dependencies are:

```

org.glassfish.jersey.core
jersey-client
2.26


org.glassfish.jersey.inject
jersey-hk2
2.26


org.glassfish.jersey.media
jersey-media-json-jackson
2.26


org.glassfish.jersey.connectors
jersey-jetty-connector
2.26

```

Jersey should depend on `javax.annotation` instead of assuming it will be provided.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.