rpc: getrpcinfo docs #15754

pull benthecarman wants to merge 1 commits into bitcoin:master from benthecarman:rpc_get_info_doc changing 1 files +14 −2
  1. benthecarman commented at 5:52 am on April 5, 2019: contributor
    Fixes #15731
  2. fanquake added the label Docs on Apr 5, 2019
  3. fanquake added the label RPC/REST/ZMQ on Apr 5, 2019
  4. in src/rpc/server.cpp:302 in 54f7fdf7cb outdated
    296@@ -297,8 +297,17 @@ static UniValue getrpcinfo(const JSONRPCRequest& request)
    297             RPCHelpMan{"getrpcinfo",
    298                 "\nReturns details of the RPC server.\n",
    299                 {},
    300-                RPCResults{},
    301-                RPCExamples{""},
    302+                RPCResult{
    303+            "{ (json object with active commands)\n"
    304+            "  { (json object with information about an active command)\n"
    


    promag commented at 6:32 am on April 5, 2019:
    Should have the key "active_commands": saying it’s a json array.

    benthecarman commented at 6:38 am on April 5, 2019:
    fixed
  5. benthecarman force-pushed on Apr 5, 2019
  6. promag commented at 6:53 am on April 5, 2019: member

    The output should be something like

    0Result:
    1{
    2 "active_commands" (array) All active commands
    3   [                 
    4     {             (object) Information about an active command
    5       "method"    (string) The name of the RPC command
    6       "duration"  (numeric) The running time in microseconds
    7     },...
    8   ]
    9}
    
  7. rpc: getrpcinfo docs f4b7a2f205
  8. benthecarman force-pushed on Apr 5, 2019
  9. promag commented at 3:23 pm on April 5, 2019: member
    LGTM.
  10. MarcoFalke added this to the milestone 0.18.0 on Apr 8, 2019
  11. MarcoFalke added the label Needs backport on Apr 8, 2019
  12. MarcoFalke removed the label Needs backport on Apr 8, 2019
  13. MarcoFalke removed this from the milestone 0.18.0 on Apr 8, 2019
  14. MarcoFalke added this to the milestone 0.18.1 on Apr 8, 2019
  15. MarcoFalke added the label Needs backport on Apr 8, 2019
  16. in src/rpc/server.cpp:301 in f4b7a2f205
    296@@ -297,8 +297,20 @@ static UniValue getrpcinfo(const JSONRPCRequest& request)
    297             RPCHelpMan{"getrpcinfo",
    298                 "\nReturns details of the RPC server.\n",
    299                 {},
    300-                RPCResults{},
    301-                RPCExamples{""},
    302+                RPCResult{
    303+            "{\n"
    


    laanwj commented at 9:01 am on April 10, 2019:
    Maybe indent this line and the strings below it four to the right?
  17. laanwj commented at 9:01 am on April 10, 2019: member
    utACK f4b7a2f2055fa52d5e02c37bed7b46599b960e2e
  18. MarcoFalke removed this from the milestone 0.18.1 on Apr 10, 2019
  19. MarcoFalke added this to the milestone 0.18.0 on Apr 10, 2019
  20. MarcoFalke merged this on Apr 10, 2019
  21. MarcoFalke closed this on Apr 10, 2019

  22. MarcoFalke referenced this in commit e82f6ad6f2 on Apr 10, 2019
  23. benthecarman referenced this in commit 60d4621975 on Apr 11, 2019
  24. laanwj referenced this in commit f42ed1442e on Apr 15, 2019
  25. fanquake commented at 12:44 pm on April 15, 2019: member
    Bckported in #15790.
  26. fanquake removed the label Needs backport on Apr 15, 2019
  27. benthecarman deleted the branch on Apr 16, 2019
  28. HashUnlimited referenced this in commit 0c6e9e634b on Apr 19, 2019
  29. jasonbcox referenced this in commit 6e2b7528f8 on Oct 5, 2020
  30. DrahtBot locked this on Dec 16, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-23 12:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me