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
Bump versions, fixes, doc updates. #620
Conversation
There were a number of vulnerabilities reported for the previous Jetty version as per the OWASP Dependency Check tool (https://www.owasp.org/index.php/OWASP_Dependency_Check).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Feel free to bump other deps if you want to just change the PR to be a deps refresh across the board
@ohpauleez good idea. I was planning on cutting a release after these changes landed as well. |
I would like to see this issue fixed before the release: #615 In summary, the |
👍 |
I've bumped deps but still need to run some tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic changeset! Thanks for all the hard work!
version as per the OWASP Dependency Check tool
(https://www.owasp.org/index.php/OWASP_Dependency_Check). Changes were benchmarked
locally using
lein benchmark-service
+service/bin/perfit.sh
and showed no significantdifference.
:container-options
.Resolves #619, #615.