Fix D2181 by including DumpRPC
ClosedPublic
Actions

Authored by deadalnix on May 14 2019, 21:42.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

Summary

As per title. The backport did not take into account modification that were done to our codebase.

Test Plan

make check
test/functional/test_runner.py

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Fabien requested changes to this revision.May 16 2019, 11:35

Fabien added a subscriber: Fabien.

Fabien added inline comments.

src/wallet/rpcdump.cpp
1473 ↗	(On Diff #8663)	My understanding is that `-disablewallet` can be set even if `ENABLE_WALLET` is set (and wallet is compiled). I would keep that one to avoid issues. Note: there seems to be a `DEFAULT_DISABLE_WALLET` which can be used instead of false (for another diff).

This revision now requires changes to proceed.May 16 2019, 11:35

deadalnix requested review of this revision.May 16 2019, 12:30

deadalnix added inline comments.

src/wallet/rpcdump.cpp
1473 ↗	(On Diff #8663)	The check is performed in `WalletInit::RegisterRPC` . This follow the pattern in D2181 . It's not the responsibility of the function that register command to check for user level configurations.