Skip to content

fix: always update auth txenv var #10707

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 4, 2025
Merged

fix: always update auth txenv var #10707

merged 1 commit into from
Jun 4, 2025

Conversation

mattsse
Copy link
Member

@mattsse mattsse commented Jun 4, 2025

closes #10706

this is a legacy revm style bug where the txenv isn't fully updated

we should consider adapting more alloy-evm style oneshot tx -> txenv conversion patterns for this

@mattsse mattsse added T-bug Type: bug C-cast Command: cast labels Jun 4, 2025
@mattsse mattsse self-assigned this Jun 4, 2025
@mattsse mattsse enabled auto-merge (squash) June 4, 2025 20:21
@mattsse mattsse merged commit dc43e2c into master Jun 4, 2025
22 checks passed
@mattsse mattsse deleted the matt/fix-reinit-txenv branch June 4, 2025 20:31
@github-project-automation github-project-automation bot moved this to Done in Foundry Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cast Command: cast T-bug Type: bug
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

cast run fails with transaction validation error: call gas cost exceeds the gas limit
2 participants