Skip to content

Optimize validator verification#12

Merged
lvs007 merged 17 commits intorelease_1.0.1from
develop
Mar 9, 2022
Merged

Optimize validator verification#12
lvs007 merged 17 commits intorelease_1.0.1from
develop

Conversation

@zhang0125
Copy link
Copy Markdown
Contributor

Optimize validator verification for release.

  1. The validator event adds a nonce check to remove invalid transactions as soon as possible.
  2. Modify the verification bttc block to 10 blocks with confirmation to prevent the reorganization of the bttc chain.
  3. Add gasPrice limit to prevent main chain transaction fees from being too high.

Krishna Upadhyaya and others added 17 commits August 4, 2021 13:30
* 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
Optimize validator verification
@lvs007 lvs007 merged commit 9d19622 into release_1.0.1 Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants