Page MenuHomePhabricator

[electrum] move ECKey.verify_message to ECPubkey.verify_message_for_address
ClosedPublic

Authored by PiRK on Jun 19 2024, 12:35.

Details

Summary

The ECKey class is about to be replaced. And we don't need a private key to verify a message anyway.

Depends on D16351

Test Plan

python test_runner.py

Diff Detail

Event Timeline

PiRK requested review of this revision.Jun 19 2024, 12:35
electrum/electrumabc/ecc.py
291 ↗(On Diff #48300)

similar change of behavior as D16351, we no longer raise errors from pubkey_from_signature on malformed signatures, we just return false. So sign_message below no longer needs to catch the error (and now actually checks the return value of verify_message)

This revision is now accepted and ready to land.Jun 20 2024, 07:58