Skip to content

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

Open
Kewe63 wants to merge 1 commit into
base:mainfrom
Kewe63:fix/sepolia-engine-rpc-protocol
Open

fix: align engine RPC protocols between mainnet and sepolia configs#1055
Kewe63 wants to merge 1 commit into
base:mainfrom
Kewe63:fix/sepolia-engine-rpc-protocol

Conversation

@Kewe63
Copy link
Copy Markdown

@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.

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented May 11, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

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