Skip to content

Conversation

@instagibbs
Copy link
Member

Introduced in #15483

@instagibbs instagibbs force-pushed the getrpcinfo_logpath_help branch from 5be19f9 to a30bd09 Compare July 3, 2019 13:36
" },...\n"
" ]\n"
" ],\n"
" \"logpath\": \"xxx\" (string) The complete file path to the debug log\n"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/complete/absolute?

@fanquake fanquake changed the title Add logpath description for getrpcinfo rpc: Add logpath description for getrpcinfo Jul 4, 2019
Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK a30bd09

master (2b46519):

$ src/bitcoin-cli help getrpcinfo
getrpcinfo

Returns details of the RPC server.

Result:
{
 "active_commands" (array) All active commands
  [
   {               (object) Information about an active command
    "method"       (string)  The name of the RPC command 
    "duration"     (numeric)  The running time in microseconds
   },...
  ]
}

Examples:
> bitcoin-cli getrpcinfo 
> curl --user myusername --data-binary '{"jsonrpc": "1.0", "id":"curltest", "method": "getrpcinfo", "params": [] }' -H 'content-type: text/plain;' http://127.0.0.1:8332/

This PR (a30bd09):

src/bitcoin-cli help getrpcinfo
getrpcinfo

Returns details of the RPC server.

Result:
{
 "active_commands" (array) All active commands
  [
   {               (object) Information about an active command
    "method"       (string)  The name of the RPC command 
    "duration"     (numeric)  The running time in microseconds
   },...
  ],
 "logpath": "xxx" (string) The complete file path to the debug log
}

Examples:
> bitcoin-cli getrpcinfo 
> curl --user myusername --data-binary '{"jsonrpc": "1.0", "id":"curltest", "method": "getrpcinfo", "params": [] }' -H 'content-type: text/plain;' http://127.0.0.1:8332/

@laanwj laanwj merged commit a30bd09 into bitcoin:master Jul 8, 2019
laanwj added a commit that referenced this pull request Jul 8, 2019
a30bd09 Add logpath description for getrpcinfo (Gregory Sanders)

Pull request description:

  Introduced in #15483

ACKs for top commit:
  fanquake:
    ACK a30bd09

Tree-SHA512: f561af675d1184412b9e426debab6269f80a65098fc7226ee93581f4075dfc93846dd4b226bd4842eb43e1649d3291c7d18558bfeb851970728b64b8a0e6df0f
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jul 9, 2019
a30bd09 Add logpath description for getrpcinfo (Gregory Sanders)

Pull request description:

  Introduced in bitcoin#15483

ACKs for top commit:
  fanquake:
    ACK a30bd09

Tree-SHA512: f561af675d1184412b9e426debab6269f80a65098fc7226ee93581f4075dfc93846dd4b226bd4842eb43e1649d3291c7d18558bfeb851970728b64b8a0e6df0f
jasonbcox pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Oct 8, 2020
Summary:
Backport of Core [[bitcoin/bitcoin#16332 | PR16332]]
Depends on D7820

Test Plan: Build, run `src/bitcoind` and `src/bitcoin-cli help getrpcinfo`

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D7821
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants