Skip to content

Conversation

@tcharding
Copy link
Member

@tcharding tcharding commented Jun 19, 2023

Expose signature verification functionality for schnorr signatures on the XOnlyPublicKey type.

Idea from Kixunil: rust-bitcoin/rust-bitcoin#1744 (comment)

@apoelstra
Copy link
Member

FYI this does not compile.

Expose signature verification functionality for schnorr signatures on
the `XOnlyPublicKey` type.
@tcharding tcharding force-pushed the 06-20-verify-function branch from fa094d1 to d60b891 Compare June 20, 2023 23:49
@tcharding
Copy link
Member Author

FYI this does not compile.

oof

apoelstra added a commit to rust-bitcoin/rust-bitcoin that referenced this pull request Jun 23, 2023
e04ac1e Add a verify function to PublicKey (Tobin C. Harding)

Pull request description:

  Expose signature verification functionality for ECDSA signatures on the `PublicKey` type.

  We should have an identical function on `XOnlyPublicKey` but this will have to be done in `secp2561`: rust-bitcoin/rust-secp256k1#618

  Idea from Kixunil: #1744 (comment)

ACKs for top commit:
  apoelstra:
    ACK e04ac1e
  Kixunil:
    ACK e04ac1e

Tree-SHA512: f26c223a1e5cc89e5c5fc12b22e621b9e8c395b8f91d7a58c6c938d45bc531e6682b178990b5a049718dbea66fff6d19d6fbcf926f142c781ad5213708ee7afa
@tcharding
Copy link
Member Author

FYI this does not compile.

Merge this PR already so I don't have to read that review again :)

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK d60b891

@apoelstra apoelstra merged commit 408c124 into rust-bitcoin:master Jul 9, 2023
@tcharding tcharding deleted the 06-20-verify-function branch July 13, 2023 00:34
@tcharding tcharding changed the title Add a verify function to PublicKey Add a verify function to XOnlyPublicKey Aug 9, 2023
apoelstra added a commit that referenced this pull request Aug 11, 2023
b9cb37d Add a verify function to PublicKey (Tobin C. Harding)

Pull request description:

  To be uniform with `XOnlyPublicKey` add a `verify` function to the `PublicKey`.

  Should have been done when we did #618

ACKs for top commit:
  apoelstra:
    ACK b9cb37d

Tree-SHA512: e1d8127daafd18d3c9b5df6edc46a961ed49e87a44c650b92c695606002f1d4c1aee3e89822e188a65ba888abd50c5b6f247570d73fa8508d739efa8bc4df7f0
chain-forgexcr45 added a commit to chain-forgexcr45/rust-secp256k1 that referenced this pull request Sep 28, 2025
…cKey

d60b89112606df822f5bc3c2e00ff66fd2787ce0 Add a verify function to PublicKey (Tobin C. Harding)

Pull request description:

  Expose signature verification functionality for schnorr signatures on the `XOnlyPublicKey` type.

  Idea from Kixunil: rust-bitcoin/rust-bitcoin#1744 (comment)

ACKs for top commit:
  apoelstra:
    ACK d60b89112606df822f5bc3c2e00ff66fd2787ce0

Tree-SHA512: 2ffa3de528b857c5b0a402815b71b35da913c668bea53b63801705fa6a86eb6d44766aa2395c02f67a4712b451c77caf627af9450183ae70957abf246a63c279
william2332-limf added a commit to william2332-limf/rust-secp256k1 that referenced this pull request Oct 2, 2025
…cKey

d60b89112606df822f5bc3c2e00ff66fd2787ce0 Add a verify function to PublicKey (Tobin C. Harding)

Pull request description:

  Expose signature verification functionality for schnorr signatures on the `XOnlyPublicKey` type.

  Idea from Kixunil: rust-bitcoin/rust-bitcoin#1744 (comment)

ACKs for top commit:
  apoelstra:
    ACK d60b89112606df822f5bc3c2e00ff66fd2787ce0

Tree-SHA512: 2ffa3de528b857c5b0a402815b71b35da913c668bea53b63801705fa6a86eb6d44766aa2395c02f67a4712b451c77caf627af9450183ae70957abf246a63c279
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.

2 participants