Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update nodes.md #249

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update nodes.md #249

wants to merge 1 commit into from

Conversation

avious00
Copy link
Contributor

hey @smrz2001 following up on the feedback from shacha on Gitcoin that it should generally be port 4001 instead of 4011, with an additional caveat around not being blocked by a firewall + consistency with the swarm address announced by IPFS

"API port on 5001 is fine, IPFS log says API server listening on /ip4/0.0.0.0/tcp/5001 and that should be internal-traffic only. Docs (https://developers.ceramic.network/run/nodes/nodes/#staying-connected-to-ipfs) says the swarm port is 4011 but it might be 4001 instead -- swarm port is the one that needs to be externally reachable"

hey @smrz2001 following up from the feedback from shacha on Gitcoin that it should generally be port 4001 instead of 4011, with an additional caveat around not being blocked by a firewall + consistency with the swarm address announced by IPFS

> "API port on 5001 is fine, IPFS log says API server listening on /ip4/0.0.0.0/tcp/5001 and that should be internal-traffic only. Docs (https://developers.ceramic.network/run/nodes/nodes/#staying-connected-to-ipfs) says the swarm port is 4011 but it might be 4001 instead -- swarm port is the one that needs to be externally reachable"
@avious00 avious00 requested a review from smrz2001 June 10, 2022 02:18
@avious00
Copy link
Contributor Author

avious00 commented Jun 10, 2022

@avious00 avious00 assigned avious00 and smrz2001 and unassigned avious00 Sep 27, 2022
Copy link
Contributor

@smrz2001 smrz2001 left a comment

Choose a reason for hiding this comment

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

LGTM apart from a minor change. I can merge it once finalized.

cc @v-stickykeys to double-check.

@@ -264,7 +264,7 @@ The Ceramic daemon serves an HTTP API that clients use to interact with your Cer

### **Staying connected to IPFS**

Ceramic nodes rely on IPFS for networking. IPFS nodes connect to each other using a Libp2p module called "switch" (aka "swarm"). This module operates over a websocket, on port `4011` by default. The websocket port must be accessible to the internet so your Ceramic node can be connected to the network.
Ceramic nodes rely on IPFS for networking. IPFS nodes connect to each other using a Libp2p module called "switch" (aka "swarm"). This module operates over a websocket, on port `4001` by default, as well as port `4011`. The websocket port must be accessible to the internet so your Ceramic node can be connected to the network, i.e. not blocked by a firewall. In addition, the port must match the swarm address announced by IPFS.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Ceramic nodes rely on IPFS for networking. IPFS nodes connect to each other using a Libp2p module called "switch" (aka "swarm"). This module operates over a websocket, on port `4001` by default, as well as port `4011`. The websocket port must be accessible to the internet so your Ceramic node can be connected to the network, i.e. not blocked by a firewall. In addition, the port must match the swarm address announced by IPFS.
Ceramic nodes rely on IPFS for networking. IPFS nodes connect to each other using a Libp2p module called "switch" (aka "swarm"). This module operates over a websocket, on port `4001` by default. The websocket port must be accessible to the internet so your Ceramic node can be connected to the network, i.e. not blocked by a firewall. In addition, the port must match the swarm address announced by IPFS.

Copy link
Contributor

Choose a reason for hiding this comment

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

Probably best to just mention the 4001 default port.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants