search

lambdaclass / ethereum_rust

Lambda Ethereum Rust Execution client
Apache License 2.0
113 stars 7 forks source link

Revisit `base_fee_per_gas` optional field #396

Open avilagaston9 opened 5 days ago

avilagaston9 commented 5 days ago

We set base_fee_per_gas header field as optional solely to run the eth capability suite of tests of the devp2p Hive simulator (#394). This adds pre-merge logic and we want to support only post-merge networks. We should revisit this change in the future and think if we can revert these changes.

mpaulucci commented 2 days ago

One solution could be to update the Hive simulation to provide a post-merge genesis file, see: https://github.com/ethereum/hive/tree/master/simulators/devp2p

Edit: It seems it is already post merge: https://github.com/ethereum/go-ethereum/pull/28340