Implement validate and intrinsic_gas for unsigned typed transactions
fselmo opened this issue · 0 comments
fselmo commented
What is wrong?
- The properties
validate
andintrinsic_gas
for a transaction are used inchain.estimate_gas()
. This is currently not implemented forUnsignedAccessListTransaction
andUnsignedDynamicFeeTransaction
, only for the signedAccessListTransaction
andDynamicFeeTransaction
.
This causes some issues when trying to send a transaction without gas and having it be calculated in eth-tester
for example. This would also provide some more consistency across the code base as unsigned legacy transactions have these properties.
How can it be fixed
- Add
intrinsic_gas
andvalidate
(with the appropriate unsigned fields) properties toUnsignedAccessListTransaction
andUnsignedDynamicFeeTransaction
.