Add concrete steps in doc/cjdns.md to easily find a friend #24710

pull jonatack wants to merge 2 commits into bitcoin:master from jonatack:doc-cjdns-how-to-find-a-friend changing 1 files +28 −7
  1. jonatack commented at 7:20 PM on March 29, 2022: member

    and improve the informational links. CJDNS functions with a friend-of-a-friend topology and a key hurdle to getting started is to find a public peer and set up an outbound connection to it. This update makes doing it much easier for people getting started.

    Credit to Vasil Dimov for an IRC suggestion in October 2021 and to stickies-v for IRC discussions this week and the testing guide that led me to redo these steps, provide feedback at #24706 and refine the added documentation here.

  2. DrahtBot added the label Docs on Mar 29, 2022
  3. in doc/cjdns.md:48 in 0a7bc97da0 outdated
      49 | +git clone https://github.com/hyperboria/peers hyperboria-peers
      50 | +cd hyperboria-peers
      51 | +./testAvailable.py
      52 | +```
      53 | +
      54 | +and to choose a friend from among the peers returned in the
    


    stickies-v commented at 10:04 AM on March 30, 2022:

    I think it would be prudent to recommend adding a few, say, 3-5 friends instead of just one?


    jonatack commented at 12:11 PM on March 30, 2022:

    Done

  4. in doc/cjdns.md:52 in 0a7bc97da0 outdated
      53 | +
      54 | +and to choose a friend from among the peers returned in the
      55 | +`hyperboria-peers/<continent/<country>/<city>` directories.
      56 | +
      57 | +You can then copy/paste its credentials into the relevant IPv4 or IPv6
      58 | +"connectTo" JSON object in the `cjdroute.conf` file in your CJDNS directory and
    


    stickies-v commented at 10:11 AM on March 30, 2022:

    Maybe this is platform specific, but I'm not sure there always is a "CJDNS directory"? Perhaps you could phrase it as "in the cjdroute.conf file you created in the setup phase"?


    jonatack commented at 12:10 PM on March 30, 2022:

    Done

  5. in doc/cjdns.md:49 in 0a7bc97da0 outdated
      50 | +cd hyperboria-peers
      51 | +./testAvailable.py
      52 | +```
      53 | +
      54 | +and to choose a friend from among the peers returned in the
      55 | +`hyperboria-peers/<continent/<country>/<city>` directories.
    


    stickies-v commented at 10:16 AM on March 30, 2022:

    Phrasing is slightly ambiguous I think. Maybe you could rephrase that the script prints the location of the credentials and the pinging result, and then user can go fetch the credentials from that location and copy/paste into config? Just highlights a bit more the user action required to actually go into the folders based on script output.


    jonatack commented at 12:10 PM on March 30, 2022:

    Done

  6. stickies-v commented at 10:22 AM on March 30, 2022: member

    Looks good, I like the additional info. Even though there's some information duplication, including the friend finding process should make things a lot easier for new users.

    Left a few comments, but beyond that pretty much good to go for me.

  7. jonatack force-pushed on Mar 30, 2022
  8. jonatack commented at 12:13 PM on March 30, 2022: member

    Thank you for the feedback, @stickies-v, updated per git diff 0a7bc97 0156505

  9. in doc/cjdns.md:53 in 0156505088 outdated
      54 | +For each peer, the `./testAvailable.py` script prints the filename of the peer's
      55 | +credentials followed by the ping result.
      56 | +
      57 | +Choose one or several peers, copy their credentials from their respective files,
      58 | +paste them into the relevant IPv4 or IPv6 "connectTo" JSON object in the
      59 | +`cjdroute.conf` you created in step ["1. Generate a new configuration
    


    stickies-v commented at 1:15 PM on March 30, 2022:

    nit: missing word "file" after cjdroute.conf


    jonatack commented at 2:47 PM on March 30, 2022:

    thanks, added

  10. jonatack force-pushed on Mar 30, 2022
  11. stickies-v approved
  12. stickies-v commented at 3:11 PM on March 30, 2022: member

    ACK 3c2f7d8

  13. brunoerg approved
  14. brunoerg commented at 11:05 PM on March 30, 2022: member

    ACK 3c2f7d883531a7d21e53446b25e53909226f67a4

    I followed it when testing v23 and the steps added here worked fine and was helpful.

  15. fanquake referenced this in commit c243e08351 on Mar 31, 2022
  16. in doc/cjdns.md:27 in 3c2f7d8835 outdated
      23 | @@ -23,17 +24,37 @@ somewhat centralized. I2P connections have a source address and I2P is slow.
      24 |  CJDNS is fast but does not hide the sender and the recipient from intermediate
      25 |  routers.
      26 |  
      27 | -## Installing CJDNS and connecting to the network
      28 | +## Installing CJDNS and finding a friend to connect to the network
    


    laanwj commented at 11:35 AM on April 1, 2022:

    Should it still be called a friend if it's simply a random active public peer selected from a list? :smile:


    jonatack commented at 11:49 AM on April 1, 2022:

    Kind of like a dating app? :smile:


    jonatack commented at 2:12 PM on April 1, 2022:

    You're right, though, "peer" would be better here, done.

  17. laanwj commented at 1:05 PM on April 1, 2022: member

    ACK 3c2f7d883531a7d21e53446b25e53909226f67a4

  18. Add concrete steps in doc/cjdns.md to easily find a friend 19538dd41e
  19. Add and improve informational links in doc/cjdns.md 6a02355ae9
  20. jonatack force-pushed on Apr 1, 2022
  21. jonatack commented at 2:14 PM on April 1, 2022: member

    Thanks for the reviews and feedback; updated with the following change.

    diff --git a/doc/cjdns.md b/doc/cjdns.md
    index e0e31ef130..b69564729f 100644
    --- a/doc/cjdns.md
    +++ b/doc/cjdns.md
    @@ -24,7 +24,7 @@ somewhat centralized. I2P connections have a source address and I2P is slow.
     CJDNS is fast but does not hide the sender and the recipient from intermediate
     routers.
     
    -## Installing CJDNS and finding a friend to connect to the network
    +## Installing CJDNS and finding a peer to connect to the network
     
     To install and set up CJDNS, follow the instructions at
     https://github.com/cjdelisle/cjdns#how-to-install-cjdns.
    
  22. stickies-v approved
  23. stickies-v commented at 2:48 PM on April 1, 2022: member

    re-ACK 6a02355 even though I wasn't opposed to the "friend" terminology since it's the language CJDNS seems to use to denominate the peers you connect to directly in general. Not worth bikeshedding over though.

  24. dunxen commented at 3:07 PM on April 1, 2022: contributor

    ACK 6a02355

  25. dunxen approved
  26. lsilva01 approved
  27. lsilva01 commented at 11:07 PM on April 1, 2022: contributor

    Strong ACK 6a02355

    I skipped this step when setting up the CJDNS for the first time. So I think it's good to reinforce this in the documentation.

  28. laanwj merged this on Apr 5, 2022
  29. laanwj closed this on Apr 5, 2022

  30. jonatack deleted the branch on Apr 5, 2022
  31. sidhujag referenced this in commit d5fe786c60 on Apr 5, 2022
  32. DrahtBot locked this on Apr 5, 2023

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: 2026-04-19 06:13 UTC

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