Skip to content

refactor: unify transaction validation logic between pool and evm#3463

Open
klkvr wants to merge 4 commits intoklkvr/refactor-load-fee-fieldsfrom
klkvr/unify-pool-evm
Open

refactor: unify transaction validation logic between pool and evm#3463
klkvr wants to merge 4 commits intoklkvr/refactor-load-fee-fieldsfrom
klkvr/unify-pool-evm

Conversation

@klkvr
Copy link
Copy Markdown
Member

@klkvr klkvr commented Apr 7, 2026

Based on #3461

Changes TempoTransactionValidator to fully delegate validation of transactions to TempoEvm.

This removes a lot of tests (mostly the ones that were covering validate_against_keychain). My assumption is that validator would now fully inherit EVM test coverage and if some scenarios are not covered there we should just migrate the tests

@klkvr klkvr changed the title Klkvr/unify pool evm refactor: unify transaction validation logic between pool and evm Apr 7, 2026
@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedtokio@​1.50.0 ⏵ 1.51.05910093100100
Updatedprometheus-client@​0.24.0 ⏵ 0.24.18210093100100
Updatedinsta@​1.47.1 ⏵ 1.47.29310093100100
Updatedalloy-rlp@​0.3.13 ⏵ 0.3.1510010093100100
Updatedindexmap@​2.13.0 ⏵ 2.13.110010093100100
Updatedsemver@​1.0.27 ⏵ 1.0.2810010093100100

View full report

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.

1 participant