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

jrestless incompatible with jersey 2.26 #41

Closed
boardthatpowder opened this issue Oct 20, 2017 · 3 comments
Closed

jrestless incompatible with jersey 2.26 #41

boardthatpowder opened this issue Oct 20, 2017 · 3 comments
Assignees
@bbilger
Labels
enhancement
Milestone
0.6.0

Comments

@boardthatpowder
Copy link

Jrestless is not compatible with Jersey 2.26 due to removing ServiceLocator and replacing InjectionManager:

http://javaarm.com/file/glassfish/jersey/doc/userguide/Jersey-2.26-User-Guide.htm#mig-2.26-injection-manager
@bbilger
Copy link
Owner

bbilger commented Oct 20, 2017

Oh, thanks for pointing this out.
I'll take a look as soon as possible. Since I am pretty busy right now, it'll take some time, unfortunately.

bbilger added a commit that referenced this issue Oct 25, 2017
@bbilger
update/migrate to Jersey 2.26; #41
94aee46 
Jersey 2.26 got rid of the (direct) HK2 dependency, making it necessary
for us to break with previous versions of Jersey. These breaking changes
should not be very relevant for users in most cases. Users are,
however, required to add an additional dependency to
"org.glassfish.jersey.inject:jersey-hk2"
@bbilger bbilger self-assigned this Oct 25, 2017
@bbilger bbilger added this to the 0.6.0 milestone Oct 25, 2017
@bbilger
Copy link
Owner

bbilger commented Oct 25, 2017

I updated to Jersey 2.26 on master.
It will, however, take some more time before this can be released since I need to update the examples in bbilger/jrestless-examples and run regressions.

@bbilger
Copy link
Owner

bbilger commented Nov 20, 2017

The fix is out with JRestless 0.6.0 which was released last weekend.

Sorry it took me a little longer but I got fooled by the new alternative injection manager: jersey-cdi2-se.
I thought there were some issues in JRestless but it turns out it's actually an issue in jersey-cdi2-se that affects other containers (e.g. Grizzly), as well: https://github.com/jersey/jersey/issues/3621. (I overlooked that Jersey's release notes mention that jersey-cdi2-se is not yet ready for production.)

Note that CDI is still working using jersey-hk2 together with jersey-weld2-se: https://github.com/bbilger/jrestless-examples/tree/master/aws/gateway/aws-gateway-cdi

@bbilger bbilger closed this as completed Nov 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bbilger bbilger

Labels
enhancement
Projects
None yet
Milestone
0.6.0
Development

No branches or pull requests
2 participants
@bbilger @boardthatpowder