Skip to content

fix(rpc): populate block_hash in eth_simulateV1 logs#21413

Merged
mattsse merged 1 commit intoparadigmxyz:mainfrom
mattsse:fix/eth-simulate-log-block-hash
Jan 26, 2026
Merged

fix(rpc): populate block_hash in eth_simulateV1 logs#21413
mattsse merged 1 commit intoparadigmxyz:mainfrom
mattsse:fix/eth-simulate-log-block-hash

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Jan 25, 2026

Summary

The logs in eth_simulateV1 response were missing the block_hash field, returning null instead of the simulated block's hash.

Test

Fixes hive test eth_simulateV1/ethSimulate-eth-send-should-produce-logs:

"blockHash": -- null ++ "0xd568b74dd9ae10031741a45a142f7dcb2625d2e31b552a6fef3a87542daef335"

The logs in eth_simulateV1 response were missing the block_hash field,
returning null instead of the simulated block's hash.

Amp-Thread-ID: https://ampcode.com/threads/T-019bf5ac-dd59-76c0-8ba8-8739e84b6a33
Co-authored-by: Amp <amp@ampcode.com>
@mattsse mattsse requested a review from Rjected as a code owner January 25, 2026 15:33
@github-project-automation github-project-automation bot moved this to Backlog in Reth Tracker Jan 25, 2026
@mattsse mattsse added the A-rpc Related to the RPC implementation label Jan 25, 2026
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.

oh nice

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Jan 26, 2026
@mattsse mattsse added this pull request to the merge queue Jan 26, 2026
Merged via the queue into paradigmxyz:main with commit 6cfd369 Jan 26, 2026
46 checks passed
@mattsse mattsse deleted the fix/eth-simulate-log-block-hash branch January 26, 2026 12:54
@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.

3 participants