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

new gwizard module: servlets #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fzakaria
Copy link

@fzakaria fzakaria commented Nov 1, 2015

Hey!

I've Guice-ified (GWizardified) the dropwizard's metrics-servlets package.

This brings in that nice AdminServlet functionality of being able to access your metrics, thread dumps, ping & health checks all within GWizard/Guice's warm embrace.

This is a sample PR. It is missing tests, however if you guys like the PR I can move over a lot of the tests from the metrics-servlets package. I saw in the google-group that at the moment, GWizard has been handed-off, so I am seeing if contributions are still being taken before finishing the PR.

Anyways please give me feedback.
Cheers.

Farid Zakaria added 2 commits October 31, 2015 19:08
Guice-ified the dropwizard-servlets package.
Added some helper servlet modules to get them started.
Bumped the javacTask to 1.8 for source/target.
Moved gwizard-servlet classes under org.gwizard.servlets
Added gwizard-servlet tests
@fzakaria
Copy link
Author

fzakaria commented Nov 1, 2015

I've added tests.

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

Successfully merging this pull request may close these issues.

None yet

1 participant