do we need "blocknumber" : "1", in rlp_decoded section of invalid block?

[winsvega]

this field seems to be excessive or unused. because the block number is already in the header. and we now require invalid blocks to have valid RLP structure. so blockheader could be read.

[spencer-tb]

The only reason I can see it being "useful" is when we have valid blocks before invalid blocks like in this case: https://github.com/ethereum/execution-spec-tests/blob/main/src/ethereum_test_tools/tests/test_filling/fixtures/blockchain_london_invalid_filled.json

Then again I don't its needed (unless evmone prefer having it)!
What do you think?

[winsvega]

I see it comes from copy paste of the structures data