-
Notifications
You must be signed in to change notification settings - Fork 5.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BIP373: Clarify where keys in MuSig fields may appear in the Taproot output #1695
Conversation
cc @bigspider |
I think you meant |
ACK, only left a small nit to make it more explicit. |
351aad6
to
16be6ec
Compare
Oops yes, fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, and it matches the way I implemented it as mentioned in #29675
- The aggregate pubkey in `PSBT_{IN,OUT}_MUSIG2_PARTICIPANT_PUBKEYS` does not have to appear anywhere in the Taproot output. - The plain pubkeys in `PSBT_IN_MUSIG2_PUB_NONCE` and `PSBT_IN_MUSIG2_PARTIAL_SIG` must be either the output pubkey, or appears in a script, and not the internal key.
16be6ec
to
0ff32bd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 0ff32bd
PSBT_{IN,OUT}_MUSIG2_PARTICIPANT_PUBKEYS
does not have to appear anywhere in the Taproot output.PSBT_IN_MUSIG2_PUB_NONCE
andPSBT_IN_MUSIG2_PARTIAL_SIG
must be either the output pubkey, or appears in a script, and not the internal key.