Skip to content

fix(engine): only warn for critical capability mismatches#21398

Merged
yongkangc merged 1 commit intomainfrom
mattsse/fix-capability-warning
Jan 26, 2026
Merged

fix(engine): only warn for critical capability mismatches#21398
yongkangc merged 1 commit intomainfrom
mattsse/fix-capability-warning

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Jan 24, 2026

Limit capability mismatch warnings to critical Engine API methods (engine_forkchoiceUpdated, engine_getPayload, engine_newPayload) that are essential for block production and chain synchronization.

Non-critical methods like engine_getBlobs are not warned about since not all clients support them and they're not required for operation.

This prevents spurious warnings when CL clients don't support optional methods like engine_getBlobsV3.

@mattsse mattsse added the A-rpc Related to the RPC implementation label Jan 24, 2026
@mattsse mattsse requested a review from Rjected as a code owner January 24, 2026 13:40
@github-project-automation github-project-automation bot moved this to Backlog in Reth Tracker Jan 24, 2026
Limit capability mismatch warnings to critical Engine API methods
(forkchoiceUpdated, getPayload, newPayload) that are essential for
block production and chain synchronization.

Non-critical methods like engine_getBlobs are not warned about since
not all clients support them and they're not required for operation.

Amp-Thread-ID: https://ampcode.com/threads/T-019bf030-638d-712d-88a5-281bf095778c
Co-authored-by: Amp <amp@ampcode.com>
@mattsse mattsse force-pushed the mattsse/fix-capability-warning branch from b8e8c67 to 3f0afdf Compare January 24, 2026 14:18
Copy link
Member

@yongkangc yongkangc left a comment

Choose a reason for hiding this comment

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

this makes sense

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Jan 26, 2026
@yongkangc yongkangc added this pull request to the merge queue Jan 26, 2026
Merged via the queue into main with commit 6d19c0e Jan 26, 2026
45 of 47 checks passed
@yongkangc yongkangc deleted the mattsse/fix-capability-warning branch January 26, 2026 12:45
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Jan 26, 2026
rakita pushed a commit that referenced this pull request Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-rpc Related to the RPC implementation

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants