Skip to content

Conversation

mattsse
Copy link
Member

@mattsse mattsse commented Aug 19, 2025

closes #11353

revm has a new check for txtype before returning the blobhash

this forces the tx to 4844 when we set blobhashes

@mattsse mattsse changed the title Matt/add repro blobhashes fix: force 4844 txtype in blobhashes setter Aug 19, 2025
Copy link
Collaborator

@grandizzy grandizzy left a comment

Choose a reason for hiding this comment

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

lgtm!

@mattsse mattsse added T-bug Type: bug C-forge Command: forge labels Aug 19, 2025
@mattsse mattsse merged commit 303afb8 into master Aug 19, 2025
22 checks passed
@mattsse mattsse deleted the matt/add-repro-blobhashes branch August 19, 2025 19:27
@github-project-automation github-project-automation bot moved this to Done in Foundry Aug 19, 2025
grandizzy pushed a commit to grandizzy/foundry that referenced this pull request Aug 20, 2025
* test: add blobhashes repro

* fix: force 4844 tx type
@grandizzy grandizzy mentioned this pull request Aug 20, 2025
3 tasks
grandizzy added a commit that referenced this pull request Aug 20, 2025
* fix(lint): explicit message when lint on build failures (#11224)

* fix(lint): lint only files that we build (#11247)

* fix: force 4844 txtype in blobhashes setter (#11355)

* test: add blobhashes repro

* fix: force 4844 tx type

* fix(forge): handle error if etherscan identifier cannot resolve config (#11356)

* fix(forge): handle error if etherscan identifier cannot resolve config

* warn on config failures

* fix: disable tx gas limit cap (#11347)

* fix(forge): write ordered deps in foundry.lock (#11360)

* chore: fix clippy (#11361)

* chore: bump version 1.3.2 (#11363)

* chore: fix cargo deny - update slab to 0.4.11

---------

Co-authored-by: Matthias Seitz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-forge Command: forge T-bug Type: bug
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

bug(forge test): vm.blobhashes does not set blobhashes
3 participants