Skip to content

Conversation

sashaodessa
Copy link
Contributor

Summary

Enforce the transaction gas limit cap only when explicitly enabled.

What changed

  • Replaced is_none() with is_some() in:
    • crates/anvil/src/eth/backend/mem/mod.rs
    • crates/anvil/src/eth/backend/executor.rs

Why

  • The previous logic inverted the check, causing the cap to not be enforced when configured.
  • Aligns validation/execution with Osaka (EIP-7825) expectations and prevents oversized txs slipping through.

Copy link
Contributor

@onbjerg onbjerg left a comment

Choose a reason for hiding this comment

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

this is incorrect.

@onbjerg onbjerg closed this Sep 28, 2025
@github-project-automation github-project-automation bot moved this to Done in Foundry Sep 28, 2025
@sashaodessa
Copy link
Contributor Author

@onbjerg but you approved this #11783

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants