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

k256: remove ecdsa::recoverable module; add replacement docs #675

Merged
merged 1 commit into from
Nov 6, 2022

Conversation

tarcieri
Copy link
Member

@tarcieri tarcieri commented Nov 6, 2022

Support for recovering VerifyingKeys from signatures has been upstreamed into the ecdsa crate in RustCrypto/signatures#576.

This commit removes the ecdsa::recoverable module and the custom impls of SigningKey/VerifyingKey needed to support it.

In its places are docs about how to use the generic APIs to accomplish equivalent functionality.

@tarcieri tarcieri force-pushed the k256/remove-ecdsa-recoverable-module-add-docs branch 6 times, most recently from e94e834 to 47d027f Compare November 6, 2022 15:09
Support for recovering `VerifyingKey`s from signatures has been
upstreamed into the `ecdsa` crate in RustCrypto/signatures#576.

This commit removes the `ecdsa::recoverable` module and the custom impls
of `SigningKey`/`VerifyingKey` needed to support it.

In its places are docs about how to use the generic APIs to accomplish
equivalent functionality.
@tarcieri tarcieri force-pushed the k256/remove-ecdsa-recoverable-module-add-docs branch from 47d027f to 6354d19 Compare November 6, 2022 16:20
@tarcieri tarcieri merged commit c87d391 into master Nov 6, 2022
@tarcieri tarcieri deleted the k256/remove-ecdsa-recoverable-module-add-docs branch November 6, 2022 16:37
@hexfusion
Copy link

Just curious the plans will this be part of the 1.12 minor bump?

@tarcieri
Copy link
Member Author

tarcieri commented Nov 7, 2022

Version bump of what?

The new API will ship in k256 v0.12 which will use ecdsa v0.15

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

2 participants