Add Cancun evm support web3 #9756
Conversation
Signed-off-by: Kristiyan Selveliev <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9756 +/- ##
============================================
- Coverage 92.23% 92.21% -0.02%
- Complexity 7628 7631 +3
============================================
Files 937 939 +2
Lines 32119 32130 +11
Branches 4071 4071
============================================
+ Hits 29624 29629 +5
- Misses 1542 1545 +3
- Partials 953 956 +3 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Kristiyan Selveliev <[email protected]>
Signed-off-by: Kristiyan Selveliev <[email protected]>
Signed-off-by: Kristiyan Selveliev <[email protected]>
| @@ -104,6 +104,7 @@ dependencies { | |||
| api("software.amazon.awssdk:bom:2.29.9") | |||
| api("uk.org.webcompere:system-stubs-jupiter:2.1.7") | |||
| api("org.web3j:core:4.12.2") | |||
| api("tech.pegasys:jc-kzg-4844:0.8.0") | |||
There was a problem hiding this comment.
Needed for KZGPointEvalPrecompiledContract.init(); method during evm initialization.
| @@ -54,7 +54,7 @@ public MirrorEvmMessageCallProcessor( | |||
| final EVM evm, | |||
| final PrecompileContractRegistry precompiles, | |||
| final PrecompiledContractProvider precompilesHolder, | |||
| final GasCalculatorHederaV22 gasCalculator, | |||
| final GasCalculator gasCalculator, | |||
There was a problem hiding this comment.
Shouldn't we use our own GasCalculator, instead of built-in from besu?
There was a problem hiding this comment.
Under the hood it will pick our GasCalculatorHederaV22 bean since it's our only bean implementing the GasCalculator interface. Just checked it locally with instance off and it picks GasCalculatorHederaV22 under the hood.
kselveliev
requested review from
bilyana-gospodinova,
nickeynikolovv,
steven-sheehy and
sdimitrov9
Signed-off-by: Kristiyan Selveliev <[email protected]>
Signed-off-by: Kristiyan Selveliev <[email protected]>
Just added integration tests exactly like the ones in |
|
Description:
This PR adds cancun evm version to the mirror node web3
This PR modifies:
EvmConfiguration.java- Added evm050 - bean configAdd
HederaSelfDestructOperationV050- added flag - useEIP6780Semantics - set to true only for V050Add
MirrorEvmMessageCallProcessorV50- using populateForCancun methodMirrorNodeEvmProperties- change evmSpecVersion to CANCUN and add block number for EVM_VERSION_0_50 for mainnet block configurations.Added unit and modified integration tests.
Related issue(s):
Fixes #8058
Notes for reviewer:
Checklist