-
Notifications
You must be signed in to change notification settings - Fork 178
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
docs: Update L1 and L2 URL examples for op-node setup #781
Conversation
The docs already specify that this is HTTP, but the example does not include the HTTP scheme
WalkthroughThis update modifies the address format for L1 beacon nodes from an IP address alone to include the HTTP protocol, ensuring clarity and consistency in endpoint specifications. Additionally, explanations are enhanced for environment variables in both mainnet and testnet tutorials, specifically addressing the typical URLs used for local nodes. Changes
Tip AI model upgrade
|
✅ Deploy Preview for docs-optimism ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Description
The tutorial for setting up
op-node
does not give URL and port examples for its environment variables, which can make it difficult to figure out which URLs should be used. I gave the default ports used by Geth/Reth and Prysm local nodes respectively.In addition, the
op-node
documentation does not state that Geth/Reth use different terminology fromop-node
when referring to the Engine API. I specified that when Geth/Reth refer to theauthrpc
parameter, they are referring to the same value used by the L2 Engine JSON-RPC endpoint.Tests
Confirmed that all changes format well visually and pass all lint checks
Additional context
N/A
Metadata
N/A