Skip to content

feat(legacy): remove eth_transactionPreExec#124

Merged
louisliu2048 merged 2 commits intomainfrom
vui-chee/no-more-legacy-pre-exec
Jan 27, 2026
Merged

feat(legacy): remove eth_transactionPreExec#124
louisliu2048 merged 2 commits intomainfrom
vui-chee/no-more-legacy-pre-exec

Conversation

@Vui-Chee
Copy link
Contributor

Since pre exec function is transferred to node group, we don't need to filter these rpc anymore.

if filter.sub_tx_filter.subscribe_addresses.len() > max_subscribed_addresses {
return Err(invalid_params_rpc_err("too many subscribe addresses"));
}
if let FlashblockParams::FlashblocksFilter(filter) = self
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Clippy - ignore this.

Copy link
Contributor

@JimmyShi22 JimmyShi22 left a comment

Choose a reason for hiding this comment

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

LGTM

@louisliu2048 louisliu2048 merged commit 1128506 into main Jan 27, 2026
@Vui-Chee Vui-Chee deleted the vui-chee/no-more-legacy-pre-exec branch January 27, 2026 11:45
Vui-Chee added a commit that referenced this pull request Jan 30, 2026
* dev:
  fix: upgrade reth v1.10.2 version with bug fixes (#134)
  Update dependencies, use reth v1.10.2 and builder v0.3.0 (#133)
  chore: fix rust formatter (#132)
  chore: better rename, fix unused variable warning (#99) (#131)
  Fix comments (#127)
  Add trace log on tx cache hits (#126)
  replace magic error code number to INTERNAL_ERROR_CODE (#125)
  feat(legacy): remove eth_transactionPreExec (#124)
  fix tools readme (#123)
  fix(rpc): improve parameter validation in get_logs and service methods (#121)
  remove output file on error (#119)
  feat(legacy): support legacy routing for batch requests (#117)
  fix(legacy): avoid possible panic on malformed block params (#111)
  fallback to legacy if res is empty (#118)
  refactor(rpc): remove transaction_pre_exec API (#120)
  use XLayerPayloadServiceBuilder to simplify main() codes, merge to main branch (#112)
  fix(tools): check end block >= start block (#116)
  forward req to legacy on err (#110)
  fix: resolve audit issues related to flashblocks subscription (#113)
  fix(rpc): add trace_address for delegatecall and fix address format c… (#104)
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