Skip to content

Revert "ecdsa: remove RecoverableSignPrimitive"#174

Merged
tarcieri merged 1 commit into
masterfrom
ecdsa/revive-recoverable-sign-primitive
Sep 27, 2020
Merged

Revert "ecdsa: remove RecoverableSignPrimitive"#174
tarcieri merged 1 commit into
masterfrom
ecdsa/revive-recoverable-sign-primitive

Conversation

@tarcieri

Copy link
Copy Markdown
Member

This reverts commit a03cda2 (#146)

Revives RecoverableSignPrimitive since there have been some requests/use cases for it.

First, there was interest in supporting such a primitive for p256:

RustCrypto/elliptic-curves#184

Second, it would be helpful for Ethereum, since it seems implementations often use (HMAC-DRBG-)SHA-256 for RFC6979, but Keccak256 as the message digest function. Exposing this interface would allow for such combinations which aren't possible otherwise.

This reverts commit a03cda2 (#146)

Revives `RecoverableSignPrimitive` since there have been some
requests/use cases for it.

First, there was interest in supporting such a primitive for `p256`:

RustCrypto/elliptic-curves#184

Second, it would be helpful for Ethereum, since it seems implementations
often use (HMAC-DRBG-)SHA-256 for RFC6979, but Keccak256 as the message
digest function. Exposing this interface would allow for such
combinations which aren't possible otherwise.
@tarcieri tarcieri merged commit 0993b4b into master Sep 27, 2020
@tarcieri tarcieri deleted the ecdsa/revive-recoverable-sign-primitive branch September 27, 2020 15:52
@tarcieri tarcieri mentioned this pull request Sep 27, 2020
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