Feat validate#9
Merged
zhang0125 merged 13 commits intobttcprotocol:developfrom Mar 9, 2022
Merged
Conversation
* new: add check to validate nonce in bridge stake update task * update: save last stake txn block in state instead of map in bridge * update: change variable name * update: used tx_query to fetch recent stake txn count * update: use helper package for query method * log: added more details to logger * update: added support for all staking txns * fix: update log message
* Matic tx confirmation also to be checked by nodes apart from the proposer too. * chainmanager keeper params to be fetcher from the checkpoint keeper. Fix * fix test cases Co-authored-by: Krishna Upadhyaya <krishnau1604@gmail.com>
* add: template for heimdall-params.go * update: README * fix: add make process-template to github ci flow * add: support for local network in process-template * update: moved process-template to build and install * update: formatting
* Update: README * fix README indentation
* add checkpoint-adjust * removed unnecessary params * add checkpoint buffer check * review comments fix * fix syntax * fix emit events * add testcases * Update side_handler.go * review comments * add test cases
zhang0125
approved these changes
Mar 9, 2022
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
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.
The validator event adds a nonce check to remove invalid transactions as soon as possible
Modify the verification bor block to 10 blocks with confirmation to prevent the reorganization of the bor chain
Add gasPrice limit to prevent main chain transaction fees from being too high