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

dependencies: update zeroize to 1.2 #52

Merged
merged 1 commit into from Oct 12, 2021
Merged

Conversation

FintanH
Copy link
Contributor

@FintanH FintanH commented Sep 30, 2021

There was a vulnerability reported[0] in zeroize which is patches with
versions >=1.2.0. This patch sets the bound to "1.2" in the Cargo.toml.

Signed-off-by: Fintan Halpenny fintan.halpenny@gmail.com

There was a vulnerability reported[0] in zeroize which is patches with
versions >=1.2.0. This patch sets the bound to "1.2" in the Cargo.toml.

[0]: https://rustsec.org/advisories/RUSTSEC-2021-0115

Signed-off-by: Fintan Halpenny <fintan.halpenny@gmail.com>
@teor2345
Copy link
Contributor

teor2345 commented Oct 1, 2021

Thanks for this PR, we were just about to do a release anyway!

@FintanH
Copy link
Contributor Author

FintanH commented Oct 1, 2021

Awesome! :) Just to clarify, shall I close this?

@teor2345
Copy link
Contributor

teor2345 commented Oct 5, 2021

Awesome! :) Just to clarify, shall I close this?

@dconnolly is handling the release, so I'll let them decide.

@FintanH
Copy link
Contributor Author

FintanH commented Oct 6, 2021

@dconnolly is handling the release, so I'll let them decide.

Sweet, I'll wait for an update then 😊

@dconnolly
Copy link
Contributor

Tested locally, merging. Thank you!

@dconnolly dconnolly merged commit 5382ead into ZcashFoundation:main Oct 12, 2021
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

3 participants