-
Notifications
You must be signed in to change notification settings - Fork 98
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 commons-compress to 1.21, to fix security issues, and related commons-* dependencies to align with docker-java:3.4.0 #1340
Conversation
223991c
to
44da509
Compare
OK, please ignore above comment. I clicked the It looks that the update of the pull request does not trigger the action, maybe add |
Hi @gaol - and thanks for your feedback. It seems to me the error is actually related to the fact that |
….0, as required by commons-compress 1.21 - everything as required by docker-java 3.4.0 - to try and fix security issues https://github.com/arquillian/arquillian-cube/security/dependabot?q=package%3Aorg.apache.commons%3Acommons-compress+manifest%3Acore%2Fpom.xml+has%3Apatch
44da509
to
b9860e9
Compare
Hi @gaol - it seems this is the less invasive solution, i.e. to align with the set of |
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 to me.
Upgrading commons-compress to 1.21 can get 4 out 5 security alerts out in that component.
And I saw your pr to upgrade commons in docker-java component: docker-java/docker-java#2378, we can upgrade the docker-java once a new release is out.
Duplicates #1325 but using 1.27.1, since the one proposed by dependantbot is throwing a NoClassFound exception.
Short description of what this resolves:
See #1325
Changes proposed in this pull request:
Fixes https://github.com/arquillian/arquillian-cube/security/dependabot?q=package%3Aorg.apache.commons%3Acommons-compress+manifest%3Acore%2Fpom.xml+has%3Apatch