Skip to content

fix: align engine RPC protocols between mainnet and sepolia configs#1

Merged
Kewe63 merged 1 commit into
mainfrom
fix/sepolia-engine-rpc-protocol
May 11, 2026
Merged

fix: align engine RPC protocols between mainnet and sepolia configs#1
Kewe63 merged 1 commit into
mainfrom
fix/sepolia-engine-rpc-protocol

Conversation

@Kewe63
Copy link
Copy Markdown
Owner

@Kewe63 Kewe63 commented May 11, 2026

Summary

Aligns the Sepolia engine RPC protocols with the Mainnet configuration for consistency.

Problem

The Sepolia configuration had OP_NODE_L2_ENGINE_RPC as ws:// and BASE_NODE_L2_ENGINE_RPC as http://, which was the inverse of the Mainnet setup. This inconsistency could lead to confusion and potential health-check derivation failures.

Solution

  • Swapped OP_NODE_L2_ENGINE_RPC to http://.
  • Swapped BASE_NODE_L2_ENGINE_RPC to ws://.

Verification

  • Verified that the protocol swap matches the documented Mainnet pattern.
  • Ensured health-check URL derivation works correctly.

Impact

Improves configuration consistency across environments.

@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Align engine RPC protocols between mainnet and sepolia configs

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Swap engine RPC protocols in sepolia config to match mainnet
• OP_NODE_L2_ENGINE_RPC changed from ws:// to http://
• BASE_NODE_L2_ENGINE_RPC changed from http:// to ws://
• Fixes health-check URL derivation in base-consensus-entrypoint
Diagram
flowchart LR
  A["Sepolia Config"] -->|"OP_NODE_L2_ENGINE_RPC"| B["http://execution:8551"]
  A -->|"BASE_NODE_L2_ENGINE_RPC"| C["ws://execution:8551"]
  D["Health-check URL"] -->|"requires ws://"| C
Loading

Grey Divider

File Changes

1. .env.sepolia 🐞 Bug fix +2/-2

Swap engine RPC protocols in sepolia environment config

• OP_NODE_L2_ENGINE_RPC protocol changed from ws:// to http://
• BASE_NODE_L2_ENGINE_RPC protocol changed from http:// to ws://
• Aligns sepolia configuration with mainnet engine RPC setup
• Enables proper health-check URL derivation via protocol substitution

.env.sepolia


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented May 11, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@Kewe63 Kewe63 merged commit 71d2f23 into main May 11, 2026
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.

1 participant