-
Notifications
You must be signed in to change notification settings - Fork 29
update dropwizard to 4.0.11 #5804
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
Conversation
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #5804 +/- ##
=============================================
- Coverage 74.53% 74.53% -0.01%
+ Complexity 5535 5533 -2
=============================================
Files 381 380 -1
Lines 19922 19918 -4
Branches 2057 2057
=============================================
- Hits 14849 14845 -4
Misses 4085 4085
Partials 988 988
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
|
Note to self: updated to 4.0.7, but need to test without forced jersey override |
4.0.10 avoids this |
|
Issue looks a lot like docker-java/docker-java#2363 |
|
Description
update dropwizard and it's supporting components to dropwizard 4.0.11
should resolve dependabot mdoerate severity PRs such as #6017 #6016 #6015
and unblock us from future dropwizard updates
Downside is I had to remove the restarting elastic search method from integration tests.
Can follow-up in https://ucsc-cgl.atlassian.net/browse/SEAB-6834 and re-add by calling out to command-line for example
Review Instructions
Builds should pass
Issue
https://ucsc-cgl.atlassian.net/browse/SEAB-6239
Security and Privacy
No real change aside from transitive improvements
Please make sure that you've checked the following before submitting your pull request. Thanks!
mvn clean install@RolesAllowedannotation