diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2021-04-07 10:53:20 +0200 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2021-04-07 10:53:26 +0200 |
| commit | aa69471ecd553dbcd7dd6d1b2e59dfb69d6a0cf3 (patch) | |
| tree | 165abd8f06510604d8dd9d133d041081db2290a5 /REVIEWERS | |
| parent | 2b3e5bf4c0dced72a93dd46da2799b86d04383cb (diff) | |
| parent | 937fd4a66f048780bffc5e714d0c800de987ce93 (diff) | |
Merge #21572: Fix wrong wallet RPC context set after #21366
937fd4a66f048780bffc5e714d0c800de987ce93 Fix wrong wallet RPC context set after #21366 (Russell Yanofsky)
Pull request description:
This bug doesn't have any effects currently because it only affects
external signer RPCs which aren't currently using the wallet context,
but it does cause an appveyor failure in a upcoming PR:
https://ci.appveyor.com/project/DrahtBot/bitcoin/builds/38512882
This bug is subtle and could have been avoided if JSONRPCRequest didn't
have constructors that were so loose with type checking. Suggested
change
https://github.com/bitcoin/bitcoin/pull/21366#issuecomment-792044351
eliminates these and would be a good followup for a future PR.
This PR just implements the simplest possible fix.
ACKs for top commit:
theStack:
Code-review ACK 937fd4a66f048780bffc5e714d0c800de987ce93
meshcollider:
Code review ACK 937fd4a66f048780bffc5e714d0c800de987ce93
Tree-SHA512: 53e6265ed6c7abb47d2b3e77d1604edfeb993c3a2440f0c19679cfeb23516965e6707ff486196a0acfbeff21c79a9a08b5cd33bae9a232d33d0134bca1bd0ff3
Diffstat (limited to 'REVIEWERS')
0 files changed, 0 insertions, 0 deletions