libp2p bootstrap#187
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR introduces libp2p integration to bootstrap network connectivity, updating the self-test script, main node logic, CLI defaults, and Cargo dependencies to support the new libp2p-based connectivity. Key changes include:
- Updating selftest.sh to use new cargo run parameters and adjust the process termination command.
- Replacing TCP-based connection handling in node/src/main.rs with libp2p swarm-based dialing and event handling.
- Modifying CLI documentation and default bind address, and updating Cargo.toml to add the libp2p dependency.
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
There was a problem hiding this comment.
Ensure that using 'killall node' will not unintentionally terminate other unrelated processes named 'node'. Consider using a more specific process identifier or pattern to target only the intended process.
There was a problem hiding this comment.
We need to rename this to braidpoold or braidpool-node because node is nodejs on many systems. We'll do this in a future PR.
ABD-AZE
force-pushed
the
libp2p_bootstrap_new
branch
from
da3914b to
962c7e7
Compare
ABD-AZE
force-pushed
the
libp2p_bootstrap_new
branch
from
962c7e7 to
5eb6879
Compare
#180