Fix rpc.BlockNumberOrHash unmarshaling
#4218
Merged
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
AlexeyAkhunov
added a commit
that referenced
this pull request
May 21, 2022
* allow --syncmode=snap for bor-mainnnet (#4206) * save * save * allow --syncmode=snap for bor-mainnnet #4206 * allow snap sync for mumbai chain (#4208) * save * mumbai * Bor: GetTransactionReceipt (#4209) * fixed miner.sigfile option (#4210) * Snap: reduced memory footprint on building huffman table (#4214) * save * save * save * save * Remove dependency on leveldb (#4213) * save * save * save * save * save * save * methods to read single txn by txnID (#4215) * It's safe now to open snapshots at app start (#4216) * removed obsolete trie variant (#4172) * up gods lib version (#4217) * Fix `rpc.BlockNumberOrHash` unmarshaling (#4218) * add test * fix unmarshaling bug Co-authored-by: Igor Mandrigin <[email protected]> * return err on invalid syncmode (#4219) * save * save * save * fixed kiln bug (#4221) * Clean headers pointers when removing links (#4222) * Clean headers pointers when removing links * Replace the lock Co-authored-by: Alexey Sharp <[email protected]> * Makefile: pass docker build arguments (#4212) 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 * Update erigon-lib and version Co-authored-by: Alex Sharov <[email protected]> Co-authored-by: Giulio rebuffo <[email protected]> Co-authored-by: Artem Tsebrovskiy <[email protected]> Co-authored-by: Igor Mandrigin <[email protected]> Co-authored-by: Igor Mandrigin <[email protected]> Co-authored-by: Alexey Sharp <[email protected]> Co-authored-by: battlmonstr <[email protected]>
AlexeyAkhunov
added a commit
that referenced
this pull request
May 21, 2022
* allow --syncmode=snap for bor-mainnnet (#4206) * save * save * allow --syncmode=snap for bor-mainnnet #4206 * allow snap sync for mumbai chain (#4208) * save * mumbai * Bor: GetTransactionReceipt (#4209) * fixed miner.sigfile option (#4210) * Snap: reduced memory footprint on building huffman table (#4214) * save * save * save * save * Remove dependency on leveldb (#4213) * save * save * save * save * save * save * methods to read single txn by txnID (#4215) * It's safe now to open snapshots at app start (#4216) * removed obsolete trie variant (#4172) * up gods lib version (#4217) * Fix `rpc.BlockNumberOrHash` unmarshaling (#4218) * add test * fix unmarshaling bug Co-authored-by: Igor Mandrigin <[email protected]> * return err on invalid syncmode (#4219) * save * save * save * fixed kiln bug (#4221) * Clean headers pointers when removing links (#4222) * Clean headers pointers when removing links * Replace the lock Co-authored-by: Alexey Sharp <[email protected]> * Makefile: pass docker build arguments (#4212) 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 * More header download diagnostics (#4224) Co-authored-by: Alexey Sharp <[email protected]> * Clean anchors, forward sort of headers (#4225) Co-authored-by: Alexey Sharp <[email protected]> Co-authored-by: Alex Sharov <[email protected]> Co-authored-by: Giulio rebuffo <[email protected]> Co-authored-by: Artem Tsebrovskiy <[email protected]> Co-authored-by: Igor Mandrigin <[email protected]> Co-authored-by: Igor Mandrigin <[email protected]> Co-authored-by: Alexey Sharp <[email protected]> Co-authored-by: battlmonstr <[email protected]>
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.
rpc.BlockNumberOrHashcan be unmarshalled from a JSON dictionary.I found that if you provide an empty or an invalid dictionary, it still unmarshalls successfully but with
nilvalues. That crashes method handlers.Added tests and error handling for this case.