Skip to content

BIP 86: Key Derivation for Single Key P2TR Outputs#1137

Merged
luke-jr merged 1 commit intobitcoin:masterfrom
achow101:taproot-bip44
Jul 2, 2021
Merged

BIP 86: Key Derivation for Single Key P2TR Outputs#1137
luke-jr merged 1 commit intobitcoin:masterfrom
achow101:taproot-bip44

Conversation

@achow101
Copy link
Member

This BIP specifies a suggested derivation path to use for single key P2TR outputs, following BIP 43/44.

Test vectors and the actual derivation path value for the purpose will be set after receiving a BIP number.

@dr-orlovsky
Copy link
Contributor

Have you seen by earlier ideas around more generic support for both single- and multisig wallets for P2TR in bitcoin-dev?

https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2021-February/018381.html

@achow101
Copy link
Member Author

Have you seen by earlier ideas around more generic support for both single- and multisig wallets for P2TR in bitcoin-dev?

Yes, but I think more complicated derivation schemes can come later. Using the system we already have is the path of least resistance and will allow for faster adoption of Taproot in current wallets.

@achow101 achow101 changed the title New BIP: Derivation scheme for Single Key P2TR based accounts New BIP: Key Derivation for Single Key P2TR Outputs Jun 27, 2021
@luke-jr luke-jr changed the title New BIP: Key Derivation for Single Key P2TR Outputs BIP 86: Key Derivation for Single Key P2TR Outputs Jul 2, 2021
@luke-jr luke-jr added the New BIP label Jul 2, 2021
@luke-jr
Copy link
Member

luke-jr commented Jul 2, 2021

Assigned BIP number 86

@achow101 achow101 force-pushed the taproot-bip44 branch 4 times, most recently from 4b08437 to 98f2048 Compare July 2, 2021 20:44
@luke-jr luke-jr merged commit 07228b6 into bitcoin:master Jul 2, 2021
@benthecarman
Copy link
Contributor

Is there a extended key version defined for BIP 86 like there was in BIP 84?

https://github.com/bitcoin/bips/blob/master/bip-0084.mediawiki#extended-key-version

@achow101
Copy link
Member Author

achow101 commented Jul 3, 2021

Is there a extended key version defined for BIP 86 like there was in BIP 84?

This intentionally does not define one.

@benthecarman
Copy link
Contributor

Is there a extended key version defined for BIP 86 like there was in BIP 84?

This intentionally does not define one.

I think it'd be worthwhile adding the reasoning to the BIP

@Rspigler
Copy link
Contributor

Rspigler commented Jul 8, 2021

I've been away, so apologies for the post-merge comments.

I agree with @achow101 's comments on the ML - I believe this is mostly unnecessary with the use of descriptors, but as long as this is for single-sig only, ACK, as it can help with adoption. Hopefully we will eventually move away from this however.

@benma
Copy link

benma commented Jul 10, 2021

Small post-merge comment: The test vectors contain addresses, so it would be great if the BIP mentioned that the addresses are bech32m (BIP-350).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants