-
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
Dropwizard Application doesn't start when used with Guice #43
Comments
@ganeshs could you include extra informations such as how guice is setup too? I have never used guice so it would be helpful. Actually, the best would be some kind of test case such as in the dropwizard-pac4j tests but with guice instead of hk2… :) |
In my scenario, guice was setup using the bundle https://github.com/gojek-engineering/gojek-commons/blob/master/gojek-commons-guice/src/main/java/com/gojek/guice/GuiceBundle.java I will try to put up a test case for guice integration. I have already come up with a fix for this issue in the pac4j jax-rs module. Will raise a PR for that in a while |
@victornoel Raised a PR for this in the jaxrs module. pac4j/jax-rs-pac4j#42 |
@ganeshs once the latest SNAPSHOT of jax-rs-pac4j is built, could you validate everything works as you expect it? |
@ganeshs did you get some time to confirm everything is ok? you should be able to use dropwizard-pac4j 3.0.0-SNAPSHOT for your tests :) |
Victor, I didn't find time for this yet. Will spend sometime early next
week. Thanks
…-Ganesh
On Sat, May 19, 2018 at 2:51 PM, Victor Noël ***@***.***> wrote:
@ganeshs <https://github.com/ganeshs> did you get some time to confirm
everything is ok? you should be able to use dropwizard-pac4j 3.0.0-SNAPSHOT
for your tests :)
I also backported your PR to 2.x btw.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#43 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABIWOjcfEXReDSI9GDd70ND1ftz9EFbtks5tz-QtgaJpZM4ThIWl>
.
|
@ganeshs have you had the time to validate everything is ok? We are going to release dropwizard-pac4j 3.0.0 soon btw (see pac4j/jax-rs-pac4j#44) |
I'm closing this, please open a new issue if this is still a problem :) |
Throws the following exception on startup. The
ServletJaxRsContextFactoryProvider
has a request scoped injection thats causing this.The text was updated successfully, but these errors were encountered: