Page MenuHomePhabricator

Merge #11191: RPC: Improve help text and behavior of RPC-logging.

Authored by nakihito on Jul 19 2019, 23:01.



c60c49b Improve help text and behavior of RPC-logging (Akio Nakamura)

Pull request description:

  1. It is allowed libevent logging to be updated during runtime, but still described that restriction in the help text. So we delete these text.
  2. Add a descrption about the evaluation order of <include> and <exclude> to clarify how debug loggig categories to be set.
  3. Add a description about the available logging category "all" which is not explained.
  4. Add "optional" to the help text of <include> and <exclude>.
  5. Add missing new lines before "Argument:".
  6. "0","1" are allowed in both array of <include> and <exclude>. "0" is ignored and "1" is treated same as "all". It is confusing, so forbid them.
  7. It always returns all logging categories with status. Fix the help text to match this behavior.

Tree-SHA512: c2142da1a9bf714af8ebc38ac0d82394e2073fc0bd56f136372e3db7b2af3b6746f8d6b0241fe66c1698c208c124deb076be83f07dec0d0a180ad150593af415

Backport Core PR11191

Depends on D3684

Test Plan
make check
./bitcoin-cli help logging

Changes should be reflected in the help text

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Owners added a reviewer: Restricted Owners Package.Jul 19 2019, 23:01
nakihito edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Jul 23 2019, 08:50