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

Handle wallet output descriptor when derivation path is not provided #1580

Closed
macaronish opened this issue Jan 5, 2025 · 2 comments
Closed

Comments

@macaronish
Copy link

When the derivation path is not provided in the output descriptor for one of the signers (see example provided by @jlopp for Bluewallet), Sparrow should assume path 'm/0/...', matching the path length of the other signers ('m/0/0/0' for the example above). Path length matching seems to matter for Coldcard because it can't sign PSBTs created by multisig wallets with mismatched derivation path lengths.

I would say this is low priority since this seems to be an issue with only Casa at the moment and they could just easily add the full derivation path for all signers when exporting the output descriptor.

@craigraw
Copy link
Collaborator

craigraw commented Jan 7, 2025

Sparrow does "handle" the wallet output descriptor when derivation path is not provided - it simply assumes the xpub is derived directly from the xprv, which contains the least assumptions. Matching the length of other derivation paths in a multisig seems to contain more assumptions to me. If there is some kind of industry standard around this with a good reasoning I'd be happy to consider it, but I'm not aware of such.

As you suggest, it's probably better for Casa to add some kind of derivation path (even if it's not the actual derivation path) to their exports.

@craigraw
Copy link
Collaborator

craigraw commented Feb 7, 2025

Closing this off.

@craigraw craigraw closed this as completed Feb 7, 2025
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

No branches or pull requests

3 participants
@craigraw @macaronish and others