getpeginaddress does not respect -rpcwallet when multiple wallets are open

[gwillen]

When multiple wallets are open, getpeginaddress does not specify the wallet when making a (hacky) recursive rpc call to getnewaddress.

The getnewaddress functionality should be refactored out into a helper function, or at least getpeginaddress should correctly set up the RPC request structure.

[instagibbs]

Whoops probably not the only place. I'll take a look.

…

On Tue, Apr 30, 2019, 8:29 PM gwillen ***@***.***> wrote: When multiple wallets are open, getpeginaddress does not specify the wallet when making a (hacky) recursive rpc call to getnewaddress. The getnewaddress functionality should be refactored out into a helper function, or at least getpeginaddress should correctly set up the RPC request structure. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#599>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABMAFU3PMQMHJNTC3ZKDPIDPTDP67ANCNFSM4HJRDV3Q> .

[instagibbs]

same hack for claimpegin as well, though we properly pass in the arguments for this one