Skip to content
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

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

achow101
Copy link
Member

@achow101 achow101 commented Nov 6, 2024

  • 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.

@achow101
Copy link
Member Author

achow101 commented Nov 6, 2024

cc @bigspider

@jonatack jonatack added Proposed BIP modification Pending acceptance This BIP modification requires sign-off by the champion of the BIP being modified and removed Pending acceptance This BIP modification requires sign-off by the champion of the BIP being modified labels Nov 6, 2024
@bigspider
Copy link
Contributor

  • The aggregate pubkey in PSBT_IN_MUSIG2_PUB_NONCE does not have to appear anywhere in the Taproot output.

I think you meant PSBT_{IN,OUT}_MUSIG2_PARTICIPANT_PUBKEYS (clarifying for any other reader passing by)

@bigspider
Copy link
Contributor

ACK, only left a small nit to make it more explicit.

@achow101
Copy link
Member Author

achow101 commented Nov 7, 2024

I think you meant PSBT_{IN,OUT}_MUSIG2_PARTICIPANT_PUBKEYS (clarifying for any other reader passing by)

Oops yes, fixed.

bip-0373.mediawiki Outdated Show resolved Hide resolved
Copy link
Contributor

@bigspider bigspider left a 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.
Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

ACK 0ff32bd

@jonatack jonatack changed the title 373: Clarify where keys in MuSig fields may appear in the Taproot output BIP373: Clarify where keys in MuSig fields may appear in the Taproot output Nov 8, 2024
@jonatack jonatack merged commit bffde65 into bitcoin:master Nov 8, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants