search

ruimarinho / bitcoin-core

A modern Bitcoin Core REST and RPC client.
482 stars 188 forks source link

Multiwallet Issue #83

Open juscamarena opened 5 years ago

juscamarena commented 5 years ago

Multiwallet support does not support using commands through .command for new rpcs.

For example this command below will fail even with a wallet set.

Checking debug logs shows that in cases with manual commands the wallet path is not added to the url.

client.command('walletprocesspsbt', rawPbstTxn)

pedrobranco commented 5 years ago

By the actual implementation, to use that wallet path it is required to be defined in methods.js file, and enable that feature.

Don't know if it is worth to have a walletCommand(wallet, ...args) to allow those calls.

juscamarena commented 5 years ago

Yeah, I manually added them to the methods.js as a workaround in a separate branch I kept, I think we should have an walletCommand(wallet, ...args) to allow those calls otherwise this lib needs to be constantly updated ever release to use any new multiwallet commands.