Fix eth call #3618
Merged
Fix eth call #3618
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Maybe let's port all changes from ethereum/go-ethereum#23424 ? |
|
But I like |
AlexeyAkhunov
pushed a commit
that referenced
this pull request
Mar 15, 2022
AlexeyAkhunov
added a commit
that referenced
this pull request
Mar 15, 2022
* State override support (#3628) * added stateOverride type * solved import cycle * refactoring * imported wrong package * fixed Call arguments * typo * override for traceCall * Fix eth call (#3618) * added isFake * using isFake instead of checkNonce * Revert "using isFake instead of checkNonce" This reverts commit 6a202bb. * Revert "added isFake" This reverts commit 2c48024. * only checking EOA if we are checking for Nonce Co-authored-by: Enrique Jose Avila Asapche <eavilaasapche@gmail.com>
bgelb
added a commit
to bgelb/erigon
that referenced
this pull request
Mar 19, 2022
* block by timestamp for stable (ledgerwatch#3617) * Add timings of forward stages to logs (ledgerwatch#3621) * save * save * deleted bor and starknet from doc (ledgerwatch#3627) * add nosqlite tag (ledgerwatch#3653) * add nosqlite tag * save * save (ledgerwatch#3665) * save (ledgerwatch#3663) * linter up (ledgerwatch#3672) (ledgerwatch#3673) * linter up (ledgerwatch#3672) * save * save * Revert node DB cache (ledgerwatch#3581) (ledgerwatch#3674) (ledgerwatch#3675) Revert "Prevent frequent commits to the node DB in sentries (ledgerwatch#2505)". This reverts commit 65a9a26. * [stable] Fixes to discovery nodedb (ledgerwatch#3691) * Update to erigon-lib stable * Discovery: throttle node DB commits (ledgerwatch#3581) (ledgerwatch#3656) UpdateFindFails/UpdateLastPingReceived/UpdateLastPongReceived events are causing bursty DB commits (100 per minute). This optimization throttles the disk writes to happen at most once in a few seconds, because this info doesn't need to be persisted immediately. This helps on HDD drives. * Update erigon-lib * Discovery: split node records to a sepatate DB table (ledgerwatch#3581) (ledgerwatch#3667) Problem: QuerySeeds will poke 150 random entries in the whole node DB and ignore hitting "field" entries. In a bootstrap scenario it might hit hundreds of :lastping :lastpong entries, and very few true "node record" entries. After running for 15 minutes I've got totalEntryCount=1508 nodeRecordCount=114 entries. There's a 1/16 chance of hitting a "node record" entry. It means finding just about 10 nodes of 114 total on average from 150 attempts. Solution: Split "node record" entries to a separate table such that QuerySeeds doesn't do idle cycle hits. * Discovery: add Context to Listen. (ledgerwatch#3577) Add explicit Context to ListenV4 and ListenV5. This makes it possible to stop listening by an external signal. * Discovery: refactor public key to node ID conversions. (ledgerwatch#3634) Encode and hash logic was duplicated in multiple places. * Move encoding to p2p/discover/v4wire * Move hashing to p2p/enode/idscheme * Change newRandomLookup to create a proper random key on a curve. * Discovery: speed up lookup tests (ledgerwatch#3677) * Update erigon-lib Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> Co-authored-by: battlmonstr <battlmonstr@users.noreply.github.com> * [stable] Fixes for state overrides in RPC (ledgerwatch#3693) * State override support (ledgerwatch#3628) * added stateOverride type * solved import cycle * refactoring * imported wrong package * fixed Call arguments * typo * override for traceCall * Fix eth call (ledgerwatch#3618) * added isFake * using isFake instead of checkNonce * Revert "using isFake instead of checkNonce" This reverts commit 6a202bb. * Revert "added isFake" This reverts commit 2c48024. * only checking EOA if we are checking for Nonce Co-authored-by: Enrique Jose Avila Asapche <eavilaasapche@gmail.com> * new bootnodes (ledgerwatch#3591) (ledgerwatch#3695) Co-authored-by: Enrique Jose Avila Asapche <eavilaasapche@gmail.com> * Update skip analysis and preverified hashes (ledgerwatch#3700) (ledgerwatch#3704) Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * Update version.go (ledgerwatch#3701) * simulate future blocks: timestamp and block number incremented by 1 for calls to trace_call(Many), debug_traceCall, eth_createAccessList * expose UsedGas through trace_call and trace_callMany * expose accessList via trace_call(Many) * plumb error into trace_call(many) outer json response Co-authored-by: Enrique Jose Avila Asapche <eavilaasapche@gmail.com> Co-authored-by: Alex Sharov <AskAlexSharov@gmail.com> Co-authored-by: battlmonstr <battlmonstr@users.noreply.github.com> Co-authored-by: ledgerwatch <akhounov@gmail.com> Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local>
bgelb
added a commit
to bgelb/erigon
that referenced
this pull request
Jun 16, 2022
* block by timestamp for stable (ledgerwatch#3617) * Add timings of forward stages to logs (ledgerwatch#3621) * save * save * deleted bor and starknet from doc (ledgerwatch#3627) * add nosqlite tag (ledgerwatch#3653) * add nosqlite tag * save * save (ledgerwatch#3665) * save (ledgerwatch#3663) * linter up (ledgerwatch#3672) (ledgerwatch#3673) * linter up (ledgerwatch#3672) * save * save * Revert node DB cache (ledgerwatch#3581) (ledgerwatch#3674) (ledgerwatch#3675) Revert "Prevent frequent commits to the node DB in sentries (ledgerwatch#2505)". This reverts commit 65a9a26. * [stable] Fixes to discovery nodedb (ledgerwatch#3691) * Update to erigon-lib stable * Discovery: throttle node DB commits (ledgerwatch#3581) (ledgerwatch#3656) UpdateFindFails/UpdateLastPingReceived/UpdateLastPongReceived events are causing bursty DB commits (100 per minute). This optimization throttles the disk writes to happen at most once in a few seconds, because this info doesn't need to be persisted immediately. This helps on HDD drives. * Update erigon-lib * Discovery: split node records to a sepatate DB table (ledgerwatch#3581) (ledgerwatch#3667) Problem: QuerySeeds will poke 150 random entries in the whole node DB and ignore hitting "field" entries. In a bootstrap scenario it might hit hundreds of :lastping :lastpong entries, and very few true "node record" entries. After running for 15 minutes I've got totalEntryCount=1508 nodeRecordCount=114 entries. There's a 1/16 chance of hitting a "node record" entry. It means finding just about 10 nodes of 114 total on average from 150 attempts. Solution: Split "node record" entries to a separate table such that QuerySeeds doesn't do idle cycle hits. * Discovery: add Context to Listen. (ledgerwatch#3577) Add explicit Context to ListenV4 and ListenV5. This makes it possible to stop listening by an external signal. * Discovery: refactor public key to node ID conversions. (ledgerwatch#3634) Encode and hash logic was duplicated in multiple places. * Move encoding to p2p/discover/v4wire * Move hashing to p2p/enode/idscheme * Change newRandomLookup to create a proper random key on a curve. * Discovery: speed up lookup tests (ledgerwatch#3677) * Update erigon-lib Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> Co-authored-by: battlmonstr <battlmonstr@users.noreply.github.com> * [stable] Fixes for state overrides in RPC (ledgerwatch#3693) * State override support (ledgerwatch#3628) * added stateOverride type * solved import cycle * refactoring * imported wrong package * fixed Call arguments * typo * override for traceCall * Fix eth call (ledgerwatch#3618) * added isFake * using isFake instead of checkNonce * Revert "using isFake instead of checkNonce" This reverts commit 6a202bb. * Revert "added isFake" This reverts commit 2c48024. * only checking EOA if we are checking for Nonce Co-authored-by: Enrique Jose Avila Asapche <eavilaasapche@gmail.com> * new bootnodes (ledgerwatch#3591) (ledgerwatch#3695) Co-authored-by: Enrique Jose Avila Asapche <eavilaasapche@gmail.com> * Update skip analysis and preverified hashes (ledgerwatch#3700) (ledgerwatch#3704) Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * Update version.go (ledgerwatch#3701) * rpcdaemon: fix TxContext in traceBlock (ledgerwatch#3716) Previously `txCtx` is not updated for every tx, which leads to wrong tracing results. * Mdbx: WriteMap fallback on error (ledgerwatch#3714) * save * save * Pool cost fix (ledgerwatch#3725) * save * save * Update to erigon-lib stable Co-authored-by: Alex Sharp <alexsharp@Alexs-MacBook-Pro.local> * mdbx v0.11.6 (ledgerwatch#3771) * mdbx fix after v0.11.6 (ledgerwatch#3775) * save * save * save * [stable] Event log subscription (ledgerwatch#3773) * Logs sub (ledgerwatch#3666) * save * Add onLogs * Fix lint * Add proper logs * Update go.mod * goimports * Add unwind * feat/rpcadaemon_logs_sub (ledgerwatch#3751) * Fixes to subscribe logs (ledgerwatch#3769) * Fixes to subscribe logs * Add criteria to logs subscription * Skeleton of RPC daemon event log distribution * Simplify * Send aggregated filter to Erigon * Change API * Print * Fixes * Fix topics filtering * Fill txHash and blockHash * Timing logs, fill tx index * Print * More print * Print * Asynchronous sending of log events to RPC daemon * Remove prints * Only extract logs if there are subscribers * Check empty when RPC daemon is removed Co-authored-by: Alex Sharp <alexsharp@Alexs-MacBook-Pro.local> Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * Fix up * Update to erigon-lib stable * Update to erigon-lib stable Co-authored-by: primal_concrete_sledge <ryban92@gmail.com> Co-authored-by: Alex Sharp <alexsharp@Alexs-MacBook-Pro.local> Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * Update version.go (ledgerwatch#3776) * Update Skip analysis and preverified hashes (ledgerwatch#3777) (ledgerwatch#3778) * Update skip analysis * Add preverified hashes for mainnet and ropsten * preverified hashes and bootnode for sepolia Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * Integration: reset StageFinish also (ledgerwatch#3783) * docker hub - fetch git tags before build ledgerwatch#3781 * fix nil pointer in fetch.go (ledgerwatch#3802) * Update preverified hashes and skip analysis (ledgerwatch#3831) (ledgerwatch#3832) * Update skip_analysis * Preverified hashes Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * Fix 'all defaults' case for eth_estimateGas (ledgerwatch#3790) (ledgerwatch#3824) * Fix 'all defaults' case for eth_estimateGas * fix tests Co-authored-by: Igor Mandrigin <i@mandrigin.ru> Co-authored-by: Igor Mandrigin <mandrigin@users.noreply.github.com> Co-authored-by: Igor Mandrigin <i@mandrigin.ru> * Update version.go (ledgerwatch#3829) * Change libmdbx submodule origin (ledgerwatch#3894) * save * Restore testdata Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * Update to erigon-lib stable (ledgerwatch#3895) Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * Update version.go (ledgerwatch#3896) * Update skip_analysis.go (ledgerwatch#3897) (ledgerwatch#3898) * save (ledgerwatch#3904) * [stable] Fixes for header download (ledgerwatch#3911) * Rollback preverified hashes for mainnet * Not remove header * Set verified = true * Fix verified extendUp and connect * Skip already persisted links * Prevent rewriting historical headers * Not load links after highestInDb * Restore preverified * Fix tests * Fix error handling Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * save (ledgerwatch#3916) * Update libmdbx source (ledgerwatch#3974) Same change as already merged in `devel` * Makefile (ledgerwatch#3779): pass docker build arguments (ledgerwatch#4239) Dockerfile requires some --build-arg options. Fix "docker" target to pass them. Fix GIT_TAG to reflect the most recent tag related to HEAD, instead of an unrelated most recent tag. Use it as the image VERSION. Image tags need to be passed explicitly if needed: DOCKER_FLAGS='-t erigon:latest' make docker * save (ledgerwatch#4346) * Gray Glacier bomb delay (ledgerwatch#4444) * Update version.go on stable branch (ledgerwatch#4447) * Update version.go * Fix lint Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> * Clean up * in transaction execution, subtract from account balance only after enough gaspool is ensured (ledgerwatch#4450) - noticed the difference when executing testdata#10 in go-ethereum and erigon * Update skip_analysis.go (ledgerwatch#4452) * Adjust version Co-authored-by: Enrique Jose Avila Asapche <eavilaasapche@gmail.com> Co-authored-by: Alex Sharov <AskAlexSharov@gmail.com> Co-authored-by: battlmonstr <battlmonstr@users.noreply.github.com> Co-authored-by: ledgerwatch <akhounov@gmail.com> Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local> Co-authored-by: can <can@canx.me> Co-authored-by: Alex Sharp <alexsharp@Alexs-MacBook-Pro.local> Co-authored-by: primal_concrete_sledge <ryban92@gmail.com> Co-authored-by: Igor Mandrigin <mandrigin@users.noreply.github.com> Co-authored-by: Igor Mandrigin <i@mandrigin.ru> Co-authored-by: Andrea Lanfranchi <andrea.lanfranchi@gmail.com> Co-authored-by: Andrew Ashikhmin <34320705+yperbasis@users.noreply.github.com> Co-authored-by: sudeep <sudeepdino008@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#3537