-
Notifications
You must be signed in to change notification settings - Fork 14
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
Output descriptor with both receiving and change paths #31
Comments
Oh okay, it is not directly in the specs but seems like an extension. Would be cool to have this anyway though! |
I believe this format is specific to Ledger. It's a more compact way to represent internal/external keys, making them easier to display on the device. @bigspider may have more insight into this, especially if it has gained wider adoption. By the way, this library supports Ledger HWW. You can use standard |
The multipath syntax is not (yet?) a standard (albeit already quite widely adopted by many software wallets, I think). There's a PR to add it to descriptors: bitcoin/bitcoin#22838 Ledger's script wallet accounts are based on the wallet policies BIP proposal, also adopted in BitBox02 and Jade, but not yet standardized either. |
Yeah, I asked this because I was using an output descriptor from Sparrow for my tests and it is using this multipath syntax. @landabaso Do you think it would be appropriate to support this syntax in the library, considering its growing adoption? |
I'm currently out of office for a couple of weeks... I'll definitely catch up with this in August. Thanks for the notice @louneskmt . |
Hey! I am trying to use an output descriptor with both the receiving (0) and change (1) paths, as follow:
wpkh([284bbfa8/84h/1h/0h]tpubDCe8CRwGAiQ2r3rdChcB3Fp8v4GVjYJqoyKKwBczxGVNnawaK21p7sWh8oiMRo26wmSEhQqHGPVofX7t9BnGaSZjbED9WwRxMt8UeTjF64S/<0;1>/*)
. It seems that this library does not support this expression<0;1>
? I get aCould not parse descriptor
error while it seems to work well with0
instead of<0;1>
. Please let me know if I can be of any help implementing this, or the rationale behind not supporting this.The text was updated successfully, but these errors were encountered: