Skip to content

[keys] limit version of zeroize to support rust 1.47+#400

Merged
afilini merged 1 commit intobitcoindevkit:masterfrom
notmandatory:fix/limit_zeroize_version
Jul 20, 2021
Merged

[keys] limit version of zeroize to support rust 1.47+#400
afilini merged 1 commit intobitcoindevkit:masterfrom
notmandatory:fix/limit_zeroize_version

Conversation

@notmandatory
Copy link
Copy Markdown
Member

Description

This will fix #399 by limiting the transitive dependency zeroize to < 1.4.0 since zeroize >= 1.4.0 has a MSRV of 1.51+.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

Bugfixes:

  • This pull request breaks the existing API
  • I've added tests to reproduce the issue which are now passing
  • I'm linking the issue being fixed by this PR

Copy link
Copy Markdown
Contributor

@rajarshimaitra rajarshimaitra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked build locally. Seems good.

One feature flag comment.

Copy link
Copy Markdown
Contributor

@rajarshimaitra rajarshimaitra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ReACK.

Copy link
Copy Markdown
Contributor

@sandipndev sandipndev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK, limiting zeroize to <1.4.0 should fix this.

@afilini afilini merged commit 474620e into bitcoindevkit:master Jul 20, 2021
@notmandatory notmandatory deleted the fix/limit_zeroize_version branch September 16, 2021 20:14
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.

Switch to a different BIP39 implementation

4 participants