docs: use 127.0.0.1 for localhost on consensus node page#1956
Conversation
The config has 127.0.0.1, so this PR updates documentation to use the same so it is clear not to change it to localhost.
WalkthroughThis pull request updates the Celestia app guide by changing the IP addresses for ports 2121 and 26657 from Changes
Suggested reviewers
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Pull Request Overview
This PR updates the documentation to specify the use of "127.0.0.1" instead of "localhost" in the Celestia app configuration for consistency with the config file.
- Updated the P2P port address from "localhost" to "127.0.0.1".
- Updated the RPC port address from "localhost" to "127.0.0.1".
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
how-to-guides/celestia-app.md (1)
132-132: Align descriptive text with explicit IP address for port 26657.Although the IP address has been updated to
127.0.0.1, the description still refers to "localhost." For consistency and clarity, consider updating the descriptive text (e.g., "false (only open to 127.0.0.1)") so that it fully reflects the explicit address used.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
how-to-guides/celestia-app.md(1 hunks)
🔇 Additional comments (1)
how-to-guides/celestia-app.md (1)
130-130: Clarify explicit IP usage for port 2121.The IP address for port 2121 has been changed from
localhostto127.0.0.1, which improves clarity by explicitly showing the intended binding address. This update aligns the documentation with the configuration settings.
The config has 127.0.0.1, so this PR updates documentation to use the same so it is clear not to change it to localhost.
Overview
Summary by CodeRabbit