infallible conversions from hashes to secp256k1 message

[kafaichoi]

Close 824

Updated
The CI will be fixed after updating to new secp256k1 version including this change rust-bitcoin/rust-secp256k1#410

[apoelstra]

There are two issues here:

1. Our CI is broken because of some honggfuzz issue. This should be fixed now as I just merged Tempararily pin fuzzer on rust 1.58 #848
2. There is some issue with the bitcoin_hashes feature, which I haven't understood yet. But we should fix this, there's no reason rust-bitcoin shouldn't have this feature turned on.

[kafaichoi]

There are two issues here:

1. Our CI is broken because of some honggfuzz issue. This should be fixed now as I just merged [Tempararily pin fuzzer on rust 1.58 #848](https://github.com/rust-bitcoin/rust-bitcoin/pull/848)

2. There is some issue with the `bitcoin_hashes` feature, which I haven't understood yet. But we should fix this, there's no reason rust-bitcoin shouldn't have this feature turned on.

As u said, first issue is resolved after rebasing latest master. For the issue 2, it looks like it's related to no std. I am looking into this🙏🏽(Changed PR status to draft)

[tcharding]

For your reference, the CI failure has an open issue.

[kafaichoi]

For your reference, the CI failure has an open issue.

Yes I think the issue should be related to bitcoin_cash default-features is enabled in rust-secp256k1. I get the ci pipeline working by adding default-features = false in forked rust-secp

Related commits in my forked ones

rust-bitcoin kafaichoi@19d1bfa
related github action https://github.com/bruteforcecat/rust-bitcoin/actions/runs/1908337297
rust-secp256k1 kafaichoi/rust-secp256k1@e3af344

[apoelstra]

Looks like this was fixed by #1118