wallet: walletprocesspsbt does not add input bip32_derivs when signing #18039

issue Sjors opened this issue on January 31, 2020
  1. Sjors commented at 9:36 AM on January 31, 2020: member

    Steps to reproduce:

    1. Create PSBT without BIP32 derivation info: bitcoin-cli -testnet walletcreatefundedpsbt '[]' '[{"mkHS9ne12qx9pS9VojpwU5xtRd4T7X7ZUt": 0.001}]' 0 '{"feeRate": 0.0001}' false

    2. Process, but don't sign: walletprocesspsbt THE_PSBT false ALL true

    Notice that this PSBT has bip32_derivs set for all inputs.

    1. Process and sign: walletprocesspsbt THE_PSBT true ALL true

    Notice that this PSBT has bip32_derivs set for the change output, but not for the inputs.

  2. Sjors added the label Bug on Jan 31, 2020
  3. Sjors commented at 9:36 AM on January 31, 2020: member
  4. fanquake added the label Wallet on Jan 31, 2020
  5. Sjors cross-referenced this on Jan 31, 2020 from issue rpc: set default bip32derivs to true for psbt methods by Sjors
  6. achow101 commented at 6:29 PM on January 31, 2020: member

    Unable to replicate. The bip32_derivs for the inputs in my test tx are filled.

  7. Sjors commented at 8:17 PM on January 31, 2020: member

    TIL walletprocesspsbt is also a Finalizer. It's probably enough to just document the bip32_derivs argument to mention that it won't add that for complete inputs.

  8. Sjors cross-referenced this on Feb 12, 2020 from issue "PSBT Operations" dialog by gwillen
  9. MarcoFalke removed the label Bug on May 7, 2020
  10. MarcoFalke added the label Docs on May 7, 2020
  11. Sjors closed this on Jan 6, 2021

  12. bitcoin locked this on Aug 18, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-20 06:54 UTC