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

signature: remove RNG generic parameter from RandomizedSigner #231

Merged
merged 1 commit into from
Jul 29, 2020

Conversation

tarcieri
Copy link
Member

@tarcieri tarcieri commented Jul 29, 2020

...replacing it with impl CryptoRng + RngCore parameter.

This is a SemVer breaking change, however per our policy around *-preview features, it will receive a minor version bump as these features are not part of the SemVer guarantees of this crate.

The rationale for this change is the exact RNG method doesn't matter, especially to the point it needs to be a generic parameter of the trait signature.

...replacing it with `impl CryptoRng + RngCore`.

This is a SemVer breaking change, however per our policy around
`*-preview` features, it will receive a minor version bump as these
features are not part of the SemVer guarantees of this crate.

The rationale for this change is the exact RNG method doesn't matter,
especially to the point it needs to be a generic parameter of the trait
signature.
@tarcieri tarcieri merged commit 0aea71b into master Jul 29, 2020
@tarcieri tarcieri deleted the signature/randomized-signer-revamp branch July 29, 2020 18:27
This was referenced Jul 29, 2020
dns2utf8 pushed a commit to dns2utf8/traits that referenced this pull request Jan 24, 2023
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.

1 participant