RPC sendmany first (dummy, empty string) argument is not optional #33182

1. 
   pinheadmz commented at 11:20 am on August 13, 2025: member

   I need a sanity check on this since #25093 was merged over three years ago and no one seems to have complained yet.

   0-->  bitcoin-cli -regtest sendmany ''  '{"bcrt1qzsmq9xxmt85uf0nu2qjrptvqdgak38umnj0p4h": 10}'
   19a43e18d3e5fcfb36dd6d4832b6ff374f877ac5dd83056dddf5d60147e082a51
   2
   3-->  bitcoin-cli -regtest sendmany '{"bcrt1qzsmq9xxmt85uf0nu2qjrptvqdgak38umnj0p4h": 10}'
   4error code: -1
   5error message:
   6sendmany ( "" ) {"address":amount,...} ( minconf "comment" ["address",...] replaceable conf_target "estimate_mode" fee_rate verbose )
   

   Certainly looks like the first argument is required:

   https://github.com/bitcoin/bitcoin/blob/2581258ec200efb173ea6449ad09b2e7f1cc02e0/src/wallet/rpc/spend.cpp#L391-L396

   …how do you make a first argument optional, anyway?

2. fanquake added the label Wallet on Aug 13, 2025
3. fanquake added the label RPC/REST/ZMQ on Aug 13, 2025

Contributors
pinheadmz

Labels
Wallet RPC/REST/ZMQ