Fetch Blocks and Txs Bug (Not found) in postBerlin Blocks and Transactions #69
Conversation
Thanks @Gustbel 👍 |
{{ message }}
This repository has been archived by the owner on Jan 13, 2023. It is now read-only.
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
This was referenced May 2, 2022
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
This PR solves the "Fetch Blocks and Txs Berlin/London version" issue. (Blocks and Tx not found).
The bug arises because the program is using an old version of
github.com/ethereum/go-ethereum(v1.9.25, Muir Glacier compatible), so the.Hash()function call (in both Blocks and Tx ) returns an incorrect Hash that does not correspond to a correct Berlin/London Hash.Because in parts of the code, the Geth functions
BlockByHash()andTransactionByHash()are called, the Blocks and Txs are never found since the hash was miscalculated with respect to newer versions.This PR only updates the Geth Mod to the latest stable version (v1.10.17) and adds other necessary new ones that the new version of Geth needs.
Another small bug has been detected but I prefer to present another PR for it, since it may be a little more controversial than this one that I consider necessary.