Skip to content

Validate max block range eth_getLogs RPC#1794

Merged
RomarQ merged 5 commits intopolkadot-evm:masterfrom
moonbeam-foundation:artur/eth-getlogs-max-block-range
Jan 7, 2026
Merged

Validate max block range eth_getLogs RPC#1794
RomarQ merged 5 commits intopolkadot-evm:masterfrom
moonbeam-foundation:artur/eth-getlogs-max-block-range

Conversation

@arturgontijo
Copy link
Copy Markdown
Collaborator

Adds a cli parameter named --max-block-range <RANGE> for customising the block range limit when querying eth_getLogs RPC, the default value is 1024 blocks. Trying to query eth_getLogs for a block range higher than 1024 will result in the following error message: block range is too wide (maximum 1024).

ref: moonbeam-foundation/frontier/pull/245

@arturgontijo arturgontijo requested a review from sorpaas as a code owner January 7, 2026 19:55
@RomarQ RomarQ self-requested a review January 7, 2026 20:06
@RomarQ RomarQ enabled auto-merge January 7, 2026 20:06
Comment thread client/rpc/src/eth/filter.rs Outdated
Comment thread client/rpc/src/eth/filter.rs Outdated
Comment thread client/rpc/src/eth/filter.rs Outdated
Comment thread client/rpc/src/eth/filter.rs Outdated
@RomarQ RomarQ added this pull request to the merge queue Jan 7, 2026
Merged via the queue into polkadot-evm:master with commit 267a4e5 Jan 7, 2026
4 checks passed
@RomarQ RomarQ deleted the artur/eth-getlogs-max-block-range branch January 7, 2026 21:18
alstjd0921 added a commit to bifrost-platform/bifrost-frontier that referenced this pull request Feb 2, 2026
Merge upstream frontier master (polkadot-sdk stable2512 compatible).
This brings 43 commits including:
- Update to polkadot-sdk stable2512 (polkadot-evm#1796)
- Add support for EIP-7825, EIP-7823, EIP-7883, EIP-7939 (Osaka fork)
- Validate max block range eth_getLogs RPC (polkadot-evm#1794)
- Extend EthereumBlockNotification with reorg info (polkadot-evm#1787)

Bifrost customizations preserved:
- view_call: Read-only EVM calls without nonce modification
- call_as_internal_call: Internal EVM calls for fee payment operations
- FeelessCallFilter: Gas-free EVM call support
- ERC20 fee payment (NODE-194)
- Debug/trace RPC extensions (NODE-190, NODE-193)
- Custom web3_clientVersion format (NODE-84)
- logs_request_timeout parameter

Conflicts resolved:
- frame/evm/src/runner/mod.rs: Added both create_force_address and Bifrost methods
- frame/evm/src/runner/stack.rs: Added both create_force_address and Bifrost methods
- client/rpc/src/eth/filter.rs: Kept both logs_request_timeout and max_block_range

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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.

3 participants