Skip to content

Conversation

@Inbell1s
Copy link
Contributor

@Inbell1s Inbell1s commented Mar 3, 2025

The web3 namespace was not enabled by default in the http.api and ws.api options within the reth-entrypoint script. This caused compatibility issues with external services, such as Chainlink nodes, which rely on web3_clientVersion for health checks.

This commit adds web3 to the exposed namespaces, ensuring compatibility with standard EVM JSON-RPC expectations.

The web3 namespace was not enabled by default in the `http.api` and `ws.api` options within the reth-entrypoint script. This caused compatibility issues with external services, such as Chainlink nodes, which rely on `web3_clientVersion` for health checks.

This commit adds `web3` to the exposed namespaces, ensuring compatibility with standard EVM JSON-RPC expectations.
@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Mar 3, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/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 0
Sum 1

@bentrnr21
Copy link

Thank you

Copy link
Collaborator

@danyalprout danyalprout left a comment

Choose a reason for hiding this comment

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

lgtm thank you

@danyalprout danyalprout merged commit 0e60232 into base:main Oct 9, 2025
1 check passed
@Inbell1s Inbell1s deleted the patch-1 branch October 21, 2025 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants