Skip to content

Implement P2PKH support and legacy signing method - #66

Merged
raphjaph merged 6 commits into
rust-bitcoin:masterfrom
aagbotemi:feat/p2pkh-support
Jul 30, 2026
Merged

Implement P2PKH support and legacy signing method#66
raphjaph merged 6 commits into
rust-bitcoin:masterfrom
aagbotemi:feat/p2pkh-support

Conversation

@aagbotemi

Copy link
Copy Markdown
Contributor

Summary

Add BIP-137 legacy message signing/verification and BIP-322 P2PKH support.
Rebased on #65.

Changes

  • Add sign_legacy_encoded and verify_legacy_encoded for the BIP-137 legacy format
  • Add create_message_signature_p2pkh / verify_full_p2pkh for BIP-322 P2PKH signing and verification
  • Add InvalidRecoveryFlag and LegacyRecover error variants

Note

Legacy is restricted to P2PKH, both sign and verify reject non-P2PKH addresses.

Closes #35
Closes #36

@aagbotemi
aagbotemi force-pushed the feat/p2pkh-support branch from 1f6cbd1 to 5869267 Compare June 29, 2026 09:58
@aagbotemi
aagbotemi force-pushed the feat/p2pkh-support branch from 5869267 to 0b7c58e Compare July 7, 2026 12:42
@aagbotemi
aagbotemi force-pushed the feat/p2pkh-support branch 2 times, most recently from 0636912 to da9355f Compare July 17, 2026 21:43
@aagbotemi
aagbotemi force-pushed the feat/p2pkh-support branch from da9355f to f7749b2 Compare July 18, 2026 01:23
@aagbotemi
aagbotemi force-pushed the feat/p2pkh-support branch from f7749b2 to 27bfbf2 Compare July 27, 2026 06:38
@aagbotemi
aagbotemi force-pushed the feat/p2pkh-support branch from 27bfbf2 to 7cb460f Compare July 29, 2026 10:43
@aagbotemi
aagbotemi force-pushed the feat/p2pkh-support branch from 7cb460f to c384f38 Compare July 29, 2026 10:47

@raphjaph raphjaph left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@raphjaph
raphjaph merged commit 972b12f into rust-bitcoin:master Jul 30, 2026
13 checks passed
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.

Implement P2PKH (Legacy) Any plans to add ECDSA support in this?

2 participants