-
Notifications
You must be signed in to change notification settings - Fork 13
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
FISH-196 Use JerseyClassAnalyzer for EjbComponent if CDI is not enabled #67
FISH-196 Use JerseyClassAnalyzer for EjbComponent if CDI is not enabled #67
Conversation
Fixes jersey/jersey#3355 Signed-off-by: Jonathan Coustick <[email protected]>
Signed-off-by: Jonathan Coustick <[email protected]>
Signed-off-by: Jan Supol <[email protected]>
* Fix Extended Wadl Example OSGi Test for JDK11 Signed-off-by: Jan Supol <[email protected]>
Signed-off-by: Jonathan Coustick <[email protected]>
Signed-off-by: Jonathan Coustick <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A commit to also update to p4-SNAPSHOT would be appreciated.
That Wadl test is failing for me, Windows 10 JDK 8. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Complaining about HTTPS still it seems
[ERROR] testWadlOptionsMethod(org.glassfish.jersey.examples.extendedwadl.ExtendedWadlWebappOsgiTest) Time elapsed: 4.538 s <<< ERROR!
java.io.IOException: Error resolving artifact org.ops4j.pax.url:pax-url-mvn:jar:1.3.7: Could not transfer artifact org.ops4j.pax.url:pax-url-mvn:jar:1.3.7 from/to central (http://repo1.maven.org/maven2/): Failed to transfer file: http://repo1.maven.org/maven2/org/ops4j/pax/url/pax-url-mvn/1.3.7/pax-url-mvn-1.3.7.jar. Return code is: 501 , ReasonPhrase:HTTPS Required.
Also includes a further fix for APPSERV-61 and some test fixes from upstream