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

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.

   2. 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

   cc @achow101
4. fanquake added the label Wallet on Jan 31, 2020
5. 
   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.
6. 
   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.
7. MarcoFalke removed the label Bug on May 7, 2020
8. MarcoFalke added the label Docs on May 7, 2020
9. Sjors closed this on Jan 6, 2021

   ---

10. DrahtBot locked this on Aug 18, 2022

Contributors
Sjors achow101

Labels
Docs Wallet