HomePhabricator

Fix misleading "Method not found" multiwallet errors

Description

Fix misleading "Method not found" multiwallet errors

Summary:
Raise RPC_WALLET_NOT_SPECIFIED instead of RPC_METHOD_NOT_FOUND when a required
wallet filename was not specified in an RPC call.

Also raise more specific RPC_WALLET_NOT_FOUND error instead of
RPC_INVALID_PARAMETER in case an invalid wallet was specified, for consistency.

Change wallet method disabled error text

Not strictly backwards compatible because the error is not new in this release.

Fixes T185

Test Plan: make check

Reviewers: deadalnix, schancel, #bitcoin_abc

Reviewed By: deadalnix, #bitcoin_abc

Subscribers: teamcity

Maniphest Tasks: T185

Differential Revision: https://reviews.bitcoinabc.org/D1024

Details

Provenance
Russell Yanofsky <russ@yanofsky.org>Authored on Jul 17 2017, 09:42
matiuCommitted on Jan 24 2018, 01:03
matiuPushed on Jan 24 2018, 20:14
Reviewer
Restricted Project
Differential Revision
D1024: Fix misleading "Method not found" multiwallet errors
Parents
rSTAGINGe70a31b663f8: [cmake] Add support for wallet tests
Branches
Unknown
Tags
Unknown
Tasks
T185: Backport Core PR10931: Fix misleading "Method not found" multiwallet errors