Given a (blank) wallet with an unused(KEY) descriptor, derivehdkey "m/87h/0h/0h" gets the xpub or xpriv at any given path.
This is particularly useful for multisig setup where it’s not desirable to use our default derivations (e.g. 44h).
I updated the multisig tutorial and the functional test.
Based on:
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32784.
See the guideline for information on the review process.
| Type | Reviewers |
|---|---|
| Concept ACK | rkrux |
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
Reviewers, this pull request conflicts with the following ones:
exportwatchonlywallet RPC to export a watchonly version of a wallet by achow101)If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
🚧 At least one of the CI tasks failed.
Task lint: https://github.com/bitcoin/bitcoin/runs/44475268764
LLM reason (✨ experimental): The CI failure is caused by errors from the lint check ‘py_lint’, specifically due to unused imports flagged by ruff.
Try to run the tests locally, according to the documentation. However, a CI failure may still happen due to a number of reasons, for example:
Possibly due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.
A sanitizer issue, which can only be found by compiling with the sanitizer and running the affected test.
An intermittent issue.
Leave a comment here, if you need help tracking down a confusing failure.
151@@ -152,6 +152,14 @@ std::pair<int64_t, int64_t> ParseDescriptorRange(const UniValue& value);
152 /** Evaluate a descriptor given as a string, or as a {"desc":...,"range":...} object, with default range of 1000. */
153 std::vector<CScript> EvalDescriptorStringOrObject(const UniValue& scanobject, FlatSigningProvider& provider, const bool expand_priv = false);
154
155+//! Get extended key and origin info for a given path
156+//! @params[in] path The BIP 32 path
In util.h: “@params[in] path” → “@param[in] path” [Doxygen tag typo]
137
138-./build/bin/bitcoin-cli -signet -rpcwallet="multisig_wallet_01" getwalletinfo
139+./build/bin/bitcoin rpc -signet -rpcwallet="multisig_wallet_01" listdescriptors
140 ```
141
142+The `<0;1>` notation in `desc` caused the creation of two descriptors. One uses the chain 0 for external addressesd the other the chain 1 for internal ones (change).
addressesd -> addresses [extra “d” makes “addresses” misspelled]
unused() descriptors do not have scriptPubKeys. Instead, the wallet uses
them to store keys without having any scripts to watch for.
Use derivehdkey instead of extracting each participant xpub (and derivation info) from the listdescriptors output.
Additionally use the new <0;1> descriptor syntax.
Finally this commits adds a few debug log lines, and expand the explanation for why we use m/44h/1h/0h.
Use derivehdkey instead of extracting each participant xpub
from the listdescriptors output.
Additionally use the new <0;1> descriptor syntax.
Also use bitcoin rpc instead of bitcoin-cli.
Labels
Wallet
Needs rebase