From ab357151da881f0e7dffd30b639c9dcdcdd4cc10 Mon Sep 17 00:00:00 2001 From: felipe Date: Tue, 17 Mar 2026 09:07:28 -0600 Subject: [PATCH] cmd/evm: don't strip prefixes on requests over t8n (#33997) Found this bug while implementing the Amsterdam changes t8n changes for benchmark test filling in EELS. Prefixes were incorrectly being stripped on requests over t8n and this was leading to `fill` correctly catching hash mismatches on the EELS side for some BAL tests. Though this was caught there, I think this change might as well be cherry-picked there instead and merged to `master`. This PR brings this behavior to parity with EELS for Osaka filling. There are still some quirks with regards to invalid block tests but I did not investigate this further. --- cmd/evm/internal/t8ntool/execution.go | 4 ---- 1 file changed, 4 deletions(-) diff --git a/cmd/evm/internal/t8ntool/execution.go b/cmd/evm/internal/t8ntool/execution.go index b3fb79bc4a..efe22d36f5 100644 --- a/cmd/evm/internal/t8ntool/execution.go +++ b/cmd/evm/internal/t8ntool/execution.go @@ -365,10 +365,6 @@ func (pre *Prestate) Apply(vmConfig vm.Config, chainConfig *params.ChainConfig, // Set requestsHash on block. h := types.CalcRequestsHash(requests) execRs.RequestsHash = &h - for i := range requests { - // remove prefix - requests[i] = requests[i][1:] - } execRs.Requests = requests }