-
Notifications
You must be signed in to change notification settings - Fork 283
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
fix(security): vulnerabilities found in quorum-multi-party-all-in-one #2239
fix(security): vulnerabilities found in quorum-multi-party-all-in-one #2239
Conversation
Hello @petermetz , Most of the vulnerabilities are now fixed in quorum-multi-party-all-in-one but there are still some that are not because most of the remaining vulnerabilities are still new and is still waiting for the new changes to be pulled in and released on their respective package versions. And we tried to ask on the quorum discussions on github as you can see here. (Consensys/quorum#1513). Here are the remaining vulnerabilities for quorum-multi-party-all-in-one: |
efcf2e0
to
dc1e391
Compare
26f10ae
to
1f71bab
Compare
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.
@aldousalvarez LGTM, thank you!
1f71bab
to
3770c1b
Compare
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.
LGTM, thankyou!
Removing my approval so that we don't accidentally merge before someone else also approves. |
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.
LGTM
3770c1b
to
f164817
Compare
7801199
to
aac8628
Compare
Fixes hyperledger-cacti#2060 Signed-off-by: aldousalvarez <[email protected]> Signed-off-by: Peter Somogyvari <[email protected]>
aac8628
to
59cc831
Compare
Fixes #2060
Signed-off-by: aldousalvarez [email protected]