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

jersey2-guice is defunct #114

Open
josephks opened this issue Jul 12, 2019 · 1 comment
Open

jersey2-guice is defunct #114

josephks opened this issue Jul 12, 2019 · 1 comment

Comments

@josephks
Copy link

dropwizard-guice depends on jersey2-guice, but jersey2-guice is defunct and bugs there won't be fixed.

@jhaber
Copy link
Member

jhaber commented Jul 14, 2019

It appears that the jersey2-guice approach of making HK2 + Guice play nicely is also no longer necessary, as newer Jersey versions allow HK2 to be replaced entirely (via custom InjectionManagerFactory). I think the best approach going forward is to write a GuiceInjectionManagerFactory and use that to ditch HK2 and jersey2-guice.

This should probably live outside of this repo so that hopefully all the various dropwizard<->Guice integrations could leverage it. I've been planning to give it a try but haven't found the time yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants