doc: clarify cookie generation in JSON-RPC-interface.md #28283

1. 
   iamcarlos94 commented at 12:48 PM on August 17, 2023: contributor

   clarifying when the .cookie file is generated

2. Update JSON-RPC-interface.md

   clarifying when the .cookie file is generated

   bdb2e8d4ae
3. 
   DrahtBot commented at 12:48 PM on August 17, 2023: contributor

   <!--e57a25ab6845829454e8d69fc972939a-->

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   <!--006a51241073e994b41acfe9ec718e94-->

   Code Coverage

   For detailed information about the code coverage, see the test coverage report.

   <!--021abf342d371248e50ceaed478a90ca-->

   Reviews

   See the guideline for information on the review process.

   Type	Reviewers
   ACK	maflcko, achow101
   Concept ACK	jonatack, ErikDeSmedt

   If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

4. 
   maflcko commented at 12:55 PM on August 17, 2023: member

   lgtm, but missing the doc: prefix in the Github title?

5. fanquake renamed this:
   Update JSON-RPC-interface.md
   doc: clarify cookie generation in JSON-RPC-interface.md
   on Aug 17, 2023
6. DrahtBot added the label Docs on Aug 17, 2023
7. in doc/JSON-RPC-interface.md:131 in bdb2e8d4ae

    127 | @@ -128,7 +128,7 @@ RPC interface will be abused.
    128 |      Instead, expose it only on the host system's localhost, for example:
    129 |      `-p 127.0.0.1:8332:8332`
    130 |  
    131 | -- **Secure authentication:** By default, Bitcoin Core generates unique
    132 | +- **Secure authentication:** By default, when no `rpcpassword` is specified, Bitcoin Core generates unique
   

   ---

   ---

   

   jonatack commented at 4:38 PM on August 17, 2023:

   - **Secure authentication:** By default, when no `-rpcpassword` configuration option is specified, Bitcoin Core generates unique
   

   ---

   Unrelated, is this still planned?

   src/httprpc.cpp:251: LogPrintf("Config options rpcuser and rpcpassword will soon be deprecated. Locally-run instances may remove rpcuser to use cookie-based auth, or may be replaced with rpcauth. Please see share/rpcauth for rpcauth auth generation.\n");

8. 
   jonatack commented at 4:39 PM on August 17, 2023: member

   Concept ACK

   Edit: can optionally rebase to current master for the CI to be green.

9. DrahtBot added the label CI failed on Aug 17, 2023
10. DrahtBot removed the label CI failed on Aug 18, 2023
11. 
    ErikDeSmedt commented at 11:44 AM on August 21, 2023: none

    ACK

12. DrahtBot added the label CI failed on Sep 4, 2023
13. DrahtBot removed the label CI failed on Sep 5, 2023
14. DrahtBot added the label CI failed on Sep 28, 2023
15. DrahtBot removed the label CI failed on Sep 28, 2023
16. 
    maflcko commented at 9:22 AM on September 28, 2023: member

    lgtm ACK bdb2e8d4aef927efd4e74d31b5d4dffe73470d1f

17. DrahtBot requested review from jonatack on Oct 20, 2023
18. DrahtBot added the label CI failed on Oct 25, 2023
19. 
    achow101 commented at 6:37 PM on October 26, 2023: member

    ACK bdb2e8d4aef927efd4e74d31b5d4dffe73470d1f

20. achow101 merged this on Oct 26, 2023
21. achow101 closed this on Oct 26, 2023

    ---

22. bitcoin locked this on Oct 25, 2024

Contributors

iamcarlos94DrahtBotmaflckojonatackErikDeSmedtachow101

Review Requested
jonatack