ecdsa::hazmat::DigestPrimitive is not implemented for Secp256k1 about elliptic-curves HOT 14 CLOSED

As noted in #269, I think you want k256::ecdsa::recoverable::Signature.

from elliptic-curves.

You need to enable the sha256 feature.

(I see this is not documented at the moment)

from elliptic-curves.

I might do a follow-up release that enables sha256 feature as part of the ecdsa feature.

It was removed as some users may only care about other digest functions (e.g. keccak256) but eliminating the superfluous dependency for those users probably isn't worth the confusion it causes.

from elliptic-curves.

Thanks!

Not enabling it as part of the ecdsa feature is fine. People may also want to use a different sha256 implementation.

What is actually confusing is the fact that the sign() and verify() functions are available when the sha256 is not enabled, even though they cannot be used.

from elliptic-curves.

sign_with_rng() also seems to be gone (not sign_digest_with_rng()).

Is that intentional?

from elliptic-curves.

I can gate those on the sha256 feature too. That's probably not even a SemVer breaking change, since they're useless without it.

from elliptic-curves.

Yep, gating them would be useful, especially since the Rust error message doesn't really help understand the root cause.

from elliptic-curves.

sign_with_rng() also seems to be gone... Is that intentional?

That looks like an oversight

from elliptic-curves.

Here's a PR to gate the Signer and Verifier impls on sha256: #192

from elliptic-curves.

#193 adds a RandomizedSigner impl.

@jedisct1 can you try the latest master and see if it resolves your issues?

I'll also add some more documentation around the sha256 feature.

from elliptic-curves.

Looking good, thank you!

from elliptic-curves.

Will cut another release with these changes soon.

from elliptic-curves.

When I try using the keccak256 feature instead of sha256, I get this same error. Is this intended?

Basically I want to generate an Ethereum-friendly signature, which is keccak256. Any advice?

from elliptic-curves.

I resolved the issue by adding the following to k256/src/ecdsa.rs

#[cfg(all(feature = "ecdsa", feature = "keccak256"))]
impl ecdsa_core::hazmat::DigestPrimitive for Secp256k1 {
    type Digest = sha3::Keccak256;
}

from elliptic-curves.