|
|
|
package core
|
|
|
|
|
|
|
|
import (
|
|
|
|
"errors"
|
|
|
|
"fmt"
|
|
|
|
"math/big"
|
|
|
|
"strings"
|
|
|
|
"testing"
|
|
|
|
|
|
|
|
"github.com/harmony-one/harmony/internal/params"
|
|
|
|
|
Release Candidate 2023.2.0 ( dev -> main ) (#4399)
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* activate epoch
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review "--port flag".
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* Commented golangci.
* staged stream sync v1.0
* fix protocol tests
* fix spell
* remove unused struct
* fix rosetta test
* add comments and refactor verify sig
* add comments, remove extra function
* add comment
* refactor errors, rename metrics
* refactor p2p host creation
* fix initsync and host creation
* fix short range hash chain
* fix beacon node detection for p2p protocol
* refactor stream peer cooldown and fix protocol beacon node field
* refactor p2p host and routing
* fix p2p discovery test issue
* add MaxAdvertiseWaitTime to handle advertisements interval and address stream connection issue
* terminal print the peer id and proto id
* fix boot complete message when node is shut down
* add new config option ( ForceReachabilityPublic ) to fix local-net consensus issue
* fix self query issue
* fix test NewDNSSyncingPeerProvider
* [testnet] disable leader rotation
* fix discovery issue for legacy sync
* add watermark low/high options for p2p connection manager
* add test for new conn manager flags
* fix dedent
* add comment to inform about p2p connection manager options
* fix max height issue
* add a separate log for get max height error
* fix log
* feat: triesInMemory flag
* fix: panic if TriesInMemory is 1 to 2
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed timers locks.
* Removed fbft locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional logs.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* Fix formatting.
* Rebased onto dev.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Fix review comment.
* Go mod tidy.
* Set to EpochTBD.
* Fix tests.
* [core] fix state handling of self destruct
If a contract self destructs to self and then receives funds within the
same transaction, it is possible for its stale state to be saved. This
change removes that possibility by checking for deleted state objects
before returning them.
* Fixed race error.
* rpc: add configurable http and `eth_call` timeout
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fixed code review.
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* staged stream sync v1.0
* refactor errors, rename metrics
* fix p2p discovery test issue
* add watermark low/high options for p2p connection manager
* fix dedent
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Go mod tidy.
* remove default timeouts
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Fixes.
* Update singleton.go
* evm: don't return extcode for validators
Due to technical debt, validator information is stored in the code field
of the address. The code field can be accessed in Solidity for an
arbitrary address using `extcodesize`, `extcodehash`, and `extcodecopy`
or helper commands (such as `address.code.Length`). The presence of this
field is used by contract developers to (erroneously) deny smart
contract access to other smart contracts (and therefore, validators).
This PR fixes that oversight by returning the same values as other EOAs
for known validator addresses. Obviously, it needs a hard fork that will
be scheduled separately.
* Fix context passing.
* Clean up code.
* Removed engine dependency.
* Fix possible panic.
* Clean up code.
* Network type.
* Fix tests.
* Revert "Removed engine dependency." (#4392)
* Revert "Fix tests."
This reverts commit 597ba2d6f1ed54ff599b9d9b8940c7285ab1277a.
* Revert "Network type."
This reverts commit 5e1878aedca3989dc0f34161dae1833e43ca6a76.
* Revert "Clean up code."
This reverts commit 15885f4c9b9263746827172b4f4f56d0926d18e2.
* Revert "Fix possible panic."
This reverts commit 1a70d5eb66cdbf8a23791806b71a323eed320085.
* Revert "Removed engine dependency."
This reverts commit 8c2ff803f709f944cfc8b1278f35cf5b2cacf859.
* gitignore the cache folder (#4389)
* stable localnet with external validator (#4388)
* stable localnet with external validator
* ignore deploy config file comments
* reduce node launched in localnet
* update makefile
* localnet configuration - add more fn
* fix validator information command typo
* Configurable tx pool. (#4240)
* AccountQueue & GlobalQueue.
* Lifetime duration.
* [pool] make flags configurable
* [pool] use 4096 as default `GlobalSlots`
* [rosetta] update default values of tx pool
* [test] update value to default
* PriceLimit and PriceBump.
* Fix tests.
* Fix price limit & bump.
* Updated, fixed migrate version and tests.
* Rebased.
* Fix go toml version.
---------
Co-authored-by: Konstantin <k.potapov@softpro.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* Upgrade rawdb and statedb codes to add the latest functionalities of ethdb (#4374)
* added bloom filter
* upgrade rawdb and statedb
* change var name and remove extra comments
* return back fake storage in case if we need it for test later
* add the previous change back
* remove some extra entries from go.mod
* fix WritePreimages to use batch
* mark unused functions which are ported over from eth
---------
Co-authored-by: Casey Gardiner <117784577+ONECasey@users.noreply.github.com>
* update all ethereum rawdb pointers to use harmony rawdb (#4395)
* Fix reduce node dependencies. (#4379)
* Fix.
* Fix.
* Fix pool init.
* Clean up.
* add prefix for contract code (#4397)
* Rotate external validators for non-beacon shards. (#4373)
* Rotate only non beacon shards.
* Rotate all shards, but only hmy validators for beacon.
* Fix type.
* Revert "Fix type."
This reverts commit 0a8b506c763d9f8609abff7395ba32b18e43b149.
* Revert "Rotate all shards, but only hmy validators for beacon."
This reverts commit 70b09e2de81aa2cbffae3ccdfd4e334e7d938759.
* Fixed failed test.
* Revert "Revert "Rotate all shards, but only hmy validators for beacon.""
This reverts commit 66cfaa9817488be60ed5b5cfee1fe833ede237c8.
* Frequency by slots count.
* Fix config.
* First validator produce rest blocks.
* Updated.
* Add lock.
* Add prefix for validator wrapper (#4402)
* add separate prefix for validator wrapper
* update comments
* make read/write backward compatible
* add validator codes to stats
* goimports
* goimports accessor_state
* add snapshot feature to state db (#4406)
* Typed cache & Node cleanup. (#4409)
* Channels usage through methods.
* Fix retry count. Removed proposedBlock.
* keysToAddrs rewritten to lrucache.
* core, internal/configs: HIP28-v2 fee collection (#4410)
* core, internal/configs: HIP28-v2 fee collection
Based on the Snapshot vote that has passed, collect 50% of the fee to a
community maintained account and the remainder to an account used to pay
for infrastructure costs. Note that these accounts need to be decided
and set in the code at this moment, and the feature needs to be
activated by setting the `FeeCollectEpoch` of the `ChainConfig` object.
The implementation for devnet is a bit different than compared to
others because the feature was activated on devnet with 100% collection
to an account. I have handled this case separately in `devnet.go`.
* test: add test for StateTransition.ApplyMessage
The objective of this unit test is to check that the fees of a
transaction are appropriately credited to the fee collectors that are
set. This means, for a transaction of 21,000 gas limit and 100 gwei gas
price, two equal fee collectors get 10,500 * 100 gwei each. In addition,
to be clear that the refund mechanism (in case a transaction with extra
gas comes in) works, the tested transaction has a 50,000 gas limit of
which only 21,000 gas limit is actually consumed.
* sharding/config: clarify local fee collector pk
* sharding/config: set testnet fee collector same as devnet
* test: add test for truncated fee distribution
* sharding/config: set fee collector addresses
* test: hardcode the expected fee collected
* goimports
* params/config: set testnet fee collect epoch
Schedule testnet hard fork epoch to be 1296, which begins around the
time 2023-04-28 07:14:20+00:00
* params/config: schedule devnee fee collection
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* Minor: removed time.Sleep from tests. (#4412)
* Provide current time as argument.
* Fix test.
* Fix naming.
* Mainnet Release Candidate 2023.1.2 (#4376) (#4414)
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Bump github.com/aws/aws-sdk-go from 1.33.0 to 1.34.0
Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.33.0 to 1.34.0.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/v1.34.0/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.33.0...v1.34.0)
---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
dependency-type: direct:production
...
* Bump github.com/ipld/go-ipld-prime from 0.9.0 to 0.19.0
Bumps [github.com/ipld/go-ipld-prime](https://github.com/ipld/go-ipld-prime) from 0.9.0 to 0.19.0.
- [Release notes](https://github.com/ipld/go-ipld-prime/releases)
- [Changelog](https://github.com/ipld/go-ipld-prime/blob/master/CHANGELOG.md)
- [Commits](https://github.com/ipld/go-ipld-prime/compare/v0.9.0...v0.19.0)
---
updated-dependencies:
- dependency-name: github.com/ipld/go-ipld-prime
dependency-type: indirect
...
* Bump golang.org/x/net from 0.3.0 to 0.7.0
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.3.0 to 0.7.0.
- [Release notes](https://github.com/golang/net/releases)
- [Commits](https://github.com/golang/net/compare/v0.3.0...v0.7.0)
---
updated-dependencies:
- dependency-name: golang.org/x/net
dependency-type: indirect
...
* Small fixes.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* activate epoch
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review "--port flag".
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* Commented golangci.
* staged stream sync v1.0
* fix protocol tests
* fix spell
* remove unused struct
* fix rosetta test
* add comments and refactor verify sig
* add comments, remove extra function
* add comment
* refactor errors, rename metrics
* refactor p2p host creation
* fix initsync and host creation
* fix short range hash chain
* fix beacon node detection for p2p protocol
* refactor stream peer cooldown and fix protocol beacon node field
* refactor p2p host and routing
* fix p2p discovery test issue
* add MaxAdvertiseWaitTime to handle advertisements interval and address stream connection issue
* terminal print the peer id and proto id
* fix boot complete message when node is shut down
* add new config option ( ForceReachabilityPublic ) to fix local-net consensus issue
* fix self query issue
* fix test NewDNSSyncingPeerProvider
* [testnet] disable leader rotation
* fix discovery issue for legacy sync
* add watermark low/high options for p2p connection manager
* add test for new conn manager flags
* fix dedent
* add comment to inform about p2p connection manager options
* fix max height issue
* add a separate log for get max height error
* fix log
* feat: triesInMemory flag
* fix: panic if TriesInMemory is 1 to 2
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed timers locks.
* Removed fbft locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional logs.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* Fix formatting.
* Rebased onto dev.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Fix review comment.
* Go mod tidy.
* Set to EpochTBD.
* Fix tests.
* [core] fix state handling of self destruct
If a contract self destructs to self and then receives funds within the
same transaction, it is possible for its stale state to be saved. This
change removes that possibility by checking for deleted state objects
before returning them.
* Fixed race error.
* rpc: add configurable http and `eth_call` timeout
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fixed code review.
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* staged stream sync v1.0
* refactor errors, rename metrics
* fix p2p discovery test issue
* add watermark low/high options for p2p connection manager
* fix dedent
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Go mod tidy.
* remove default timeouts
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Fixes.
* Update singleton.go
* evm: don't return extcode for validators
Due to technical debt, validator information is stored in the code field
of the address. The code field can be accessed in Solidity for an
arbitrary address using `extcodesize`, `extcodehash`, and `extcodecopy`
or helper commands (such as `address.code.Length`). The presence of this
field is used by contract developers to (erroneously) deny smart
contract access to other smart contracts (and therefore, validators).
This PR fixes that oversight by returning the same values as other EOAs
for known validator addresses. Obviously, it needs a hard fork that will
be scheduled separately.
* Fix context passing.
* Clean up code.
* Removed engine dependency.
* Fix possible panic.
* Clean up code.
* Network type.
* Fix tests.
* Revert "Removed engine dependency." (#4392)
* Revert "Fix tests."
This reverts commit 597ba2d6f1ed54ff599b9d9b8940c7285ab1277a.
* Revert "Network type."
This reverts commit 5e1878aedca3989dc0f34161dae1833e43ca6a76.
* Revert "Clean up code."
This reverts commit 15885f4c9b9263746827172b4f4f56d0926d18e2.
* Revert "Fix possible panic."
This reverts commit 1a70d5eb66cdbf8a23791806b71a323eed320085.
* Revert "Removed engine dependency."
This reverts commit 8c2ff803f709f944cfc8b1278f35cf5b2cacf859.
* gitignore the cache folder (#4389)
* stable localnet with external validator (#4388)
* stable localnet with external validator
* ignore deploy config file comments
* reduce node launched in localnet
* update makefile
* localnet configuration - add more fn
* fix validator information command typo
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Casey Gardiner <117784577+ONECasey@users.noreply.github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
* chore: merge `main` into `dev` (#4415)
* Mainnet Release Candidate 2023.1.2 (#4376)
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Bump github.com/aws/aws-sdk-go from 1.33.0 to 1.34.0
Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.33.0 to 1.34.0.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/v1.34.0/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.33.0...v1.34.0)
---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump github.com/ipld/go-ipld-prime from 0.9.0 to 0.19.0
Bumps [github.com/ipld/go-ipld-prime](https://github.com/ipld/go-ipld-prime) from 0.9.0 to 0.19.0.
- [Release notes](https://github.com/ipld/go-ipld-prime/releases)
- [Changelog](https://github.com/ipld/go-ipld-prime/blob/master/CHANGELOG.md)
- [Commits](https://github.com/ipld/go-ipld-prime/compare/v0.9.0...v0.19.0)
---
updated-dependencies:
- dependency-name: github.com/ipld/go-ipld-prime
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump golang.org/x/net from 0.3.0 to 0.7.0
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.3.0 to 0.7.0.
- [Release notes](https://github.com/golang/net/releases)
- [Commits](https://github.com/golang/net/compare/v0.3.0...v0.7.0)
---
updated-dependencies:
- dependency-name: golang.org/x/net
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
* Small fixes.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* activate epoch
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review "--port flag".
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* Commented golangci.
* staged stream sync v1.0
* fix protocol tests
* fix spell
* remove unused struct
* fix rosetta test
* add comments and refactor verify sig
* add comments, remove extra function
* add comment
* refactor errors, rename metrics
* refactor p2p host creation
* fix initsync and host creation
* fix short range hash chain
* fix beacon node detection for p2p protocol
* refactor stream peer cooldown and fix protocol beacon node field
* refactor p2p host and routing
* fix p2p discovery test issue
* add MaxAdvertiseWaitTime to handle advertisements interval and address stream connection issue
* terminal print the peer id and proto id
* fix boot complete message when node is shut down
* add new config option ( ForceReachabilityPublic ) to fix local-net consensus issue
* fix self query issue
* fix test NewDNSSyncingPeerProvider
* [testnet] disable leader rotation
* fix discovery issue for legacy sync
* add watermark low/high options for p2p connection manager
* add test for new conn manager flags
* fix dedent
* add comment to inform about p2p connection manager options
* fix max height issue
* add a separate log for get max height error
* fix log
* feat: triesInMemory flag
* fix: panic if TriesInMemory is 1 to 2
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed timers locks.
* Removed fbft locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional logs.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* Fix formatting.
* Rebased onto dev.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Fix review comment.
* Go mod tidy.
* Set to EpochTBD.
* Fix tests.
* [core] fix state handling of self destruct
If a contract self destructs to self and then receives funds within the
same transaction, it is possible for its stale state to be saved. This
change removes that possibility by checking for deleted state objects
before returning them.
* Fixed race error.
* rpc: add configurable http and `eth_call` timeout
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fixed code review.
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* staged stream sync v1.0
* refactor errors, rename metrics
* fix p2p discovery test issue
* add watermark low/high options for p2p connection manager
* fix dedent
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Go mod tidy.
* remove default timeouts
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Fixes.
* Update singleton.go
* evm: don't return extcode for validators
Due to technical debt, validator information is stored in the code field
of the address. The code field can be accessed in Solidity for an
arbitrary address using `extcodesize`, `extcodehash`, and `extcodecopy`
or helper commands (such as `address.code.Length`). The presence of this
field is used by contract developers to (erroneously) deny smart
contract access to other smart contracts (and therefore, validators).
This PR fixes that oversight by returning the same values as other EOAs
for known validator addresses. Obviously, it needs a hard fork that will
be scheduled separately.
* Fix context passing.
* Clean up code.
* Removed engine dependency.
* Fix possible panic.
* Clean up code.
* Network type.
* Fix tests.
* Revert "Removed engine dependency." (#4392)
* Revert "Fix tests."
This reverts commit 597ba2d6f1ed54ff599b9d9b8940c7285ab1277a.
* Revert "Network type."
This reverts commit 5e1878aedca3989dc0f34161dae1833e43ca6a76.
* Revert "Clean up code."
This reverts commit 15885f4c9b9263746827172b4f4f56d0926d18e2.
* Revert "Fix possible panic."
This reverts commit 1a70d5eb66cdbf8a23791806b71a323eed320085.
* Revert "Removed engine dependency."
This reverts commit 8c2ff803f709f944cfc8b1278f35cf5b2cacf859.
* gitignore the cache folder (#4389)
* stable localnet with external validator (#4388)
* stable localnet with external validator
* ignore deploy config file comments
* reduce node launched in localnet
* update makefile
* localnet configuration - add more fn
* fix validator information command typo
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
* build: update pinned curl version (#4394)
Per the Alpine Linux package repositories, the version for cURL included
with v3.16 has changed to revision 6
* consensus: replace type assert with test (#4398)
If `consensus.finalityCounter` does not have anything stored (for
example in Syncing mode), the `Load()` returns an interface that cannot
be automatically asserted to an `int64`. This results in the node
crashing. This commit fixes that.
* Turn pprof default on with local saved files (#3894)
* Turn pprof default on with local saved files
* [pprof] change interval from 600s to 3600s
* Revert "Turn pprof default on with local saved files (#3894)" (#4400)
This reverts commit 78d26d7910a58ded3bfe689b3de07ea28d95d7d5.
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Casey Gardiner <117784577+ONECasey@users.noreply.github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
Co-authored-by: Jacky Wang <jackyw.se@gmail.com>
* internal/params: set validator code fix hard forks (#4413)
* internal/params: schedule hard forks
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* internal/params: set localnet fee collect epoch 2
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
---------
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* internal/params: schedule HIP28v2 + val code fix (#4416)
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* Dev fix conflicts. (#4417)
* Mainnet Release Candidate 2023.1.2 (#4376)
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Bump github.com/aws/aws-sdk-go from 1.33.0 to 1.34.0
Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.33.0 to 1.34.0.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/v1.34.0/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.33.0...v1.34.0)
---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump github.com/ipld/go-ipld-prime from 0.9.0 to 0.19.0
Bumps [github.com/ipld/go-ipld-prime](https://github.com/ipld/go-ipld-prime) from 0.9.0 to 0.19.0.
- [Release notes](https://github.com/ipld/go-ipld-prime/releases)
- [Changelog](https://github.com/ipld/go-ipld-prime/blob/master/CHANGELOG.md)
- [Commits](https://github.com/ipld/go-ipld-prime/compare/v0.9.0...v0.19.0)
---
updated-dependencies:
- dependency-name: github.com/ipld/go-ipld-prime
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump golang.org/x/net from 0.3.0 to 0.7.0
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.3.0 to 0.7.0.
- [Release notes](https://github.com/golang/net/releases)
- [Commits](https://github.com/golang/net/compare/v0.3.0...v0.7.0)
---
updated-dependencies:
- dependency-name: golang.org/x/net
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
* Small fixes.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* activate epoch
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review "--port flag".
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* Commented golangci.
* staged stream sync v1.0
* fix protocol tests
* fix spell
* remove unused struct
* fix rosetta test
* add comments and refactor verify sig
* add comments, remove extra function
* add comment
* refactor errors, rename metrics
* refactor p2p host creation
* fix initsync and host creation
* fix short range hash chain
* fix beacon node detection for p2p protocol
* refactor stream peer cooldown and fix protocol beacon node field
* refactor p2p host and routing
* fix p2p discovery test issue
* add MaxAdvertiseWaitTime to handle advertisements interval and address stream connection issue
* terminal print the peer id and proto id
* fix boot complete message when node is shut down
* add new config option ( ForceReachabilityPublic ) to fix local-net consensus issue
* fix self query issue
* fix test NewDNSSyncingPeerProvider
* [testnet] disable leader rotation
* fix discovery issue for legacy sync
* add watermark low/high options for p2p connection manager
* add test for new conn manager flags
* fix dedent
* add comment to inform about p2p connection manager options
* fix max height issue
* add a separate log for get max height error
* fix log
* feat: triesInMemory flag
* fix: panic if TriesInMemory is 1 to 2
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed timers locks.
* Removed fbft locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional logs.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* Fix formatting.
* Rebased onto dev.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Fix review comment.
* Go mod tidy.
* Set to EpochTBD.
* Fix tests.
* [core] fix state handling of self destruct
If a contract self destructs to self and then receives funds within the
same transaction, it is possible for its stale state to be saved. This
change removes that possibility by checking for deleted state objects
before returning them.
* Fixed race error.
* rpc: add configurable http and `eth_call` timeout
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fixed code review.
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* staged stream sync v1.0
* refactor errors, rename metrics
* fix p2p discovery test issue
* add watermark low/high options for p2p connection manager
* fix dedent
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Go mod tidy.
* remove default timeouts
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Fixes.
* Update singleton.go
* evm: don't return extcode for validators
Due to technical debt, validator information is stored in the code field
of the address. The code field can be accessed in Solidity for an
arbitrary address using `extcodesize`, `extcodehash`, and `extcodecopy`
or helper commands (such as `address.code.Length`). The presence of this
field is used by contract developers to (erroneously) deny smart
contract access to other smart contracts (and therefore, validators).
This PR fixes that oversight by returning the same values as other EOAs
for known validator addresses. Obviously, it needs a hard fork that will
be scheduled separately.
* Fix context passing.
* Clean up code.
* Removed engine dependency.
* Fix possible panic.
* Clean up code.
* Network type.
* Fix tests.
* Revert "Removed engine dependency." (#4392)
* Revert "Fix tests."
This reverts commit 597ba2d6f1ed54ff599b9d9b8940c7285ab1277a.
* Revert "Network type."
This reverts commit 5e1878aedca3989dc0f34161dae1833e43ca6a76.
* Revert "Clean up code."
This reverts commit 15885f4c9b9263746827172b4f4f56d0926d18e2.
* Revert "Fix possible panic."
This reverts commit 1a70d5eb66cdbf8a23791806b71a323eed320085.
* Revert "Removed engine dependency."
This reverts commit 8c2ff803f709f944cfc8b1278f35cf5b2cacf859.
* gitignore the cache folder (#4389)
* stable localnet with external validator (#4388)
* stable localnet with external validator
* ignore deploy config file comments
* reduce node launched in localnet
* update makefile
* localnet configuration - add more fn
* fix validator information command typo
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
* build: update pinned curl version (#4394)
Per the Alpine Linux package repositories, the version for cURL included
with v3.16 has changed to revision 6
* consensus: replace type assert with test (#4398)
If `consensus.finalityCounter` does not have anything stored (for
example in Syncing mode), the `Load()` returns an interface that cannot
be automatically asserted to an `int64`. This results in the node
crashing. This commit fixes that.
* Turn pprof default on with local saved files (#3894)
* Turn pprof default on with local saved files
* [pprof] change interval from 600s to 3600s
* Revert "Turn pprof default on with local saved files (#3894)" (#4400)
This reverts commit 78d26d7910a58ded3bfe689b3de07ea28d95d7d5.
* go mod tidy.
* Increased wait time.
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Casey Gardiner <117784577+ONECasey@users.noreply.github.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
Co-authored-by: Jacky Wang <jackyw.se@gmail.com>
---------
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
Co-authored-by: Konstantin <k.potapov@softpro.com>
Co-authored-by: Gheis Mohammadi <Gheis.Mohammadi@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jacky Wang <jackyw.se@gmail.com>
2 years ago
|
|
|
"github.com/harmony-one/harmony/core/rawdb"
|
|
|
|
|
|
|
|
"github.com/harmony-one/harmony/crypto/bls"
|
|
|
|
|
|
|
|
"github.com/ethereum/go-ethereum/common"
|
|
|
|
"github.com/harmony-one/harmony/block"
|
|
|
|
consensus_engine "github.com/harmony-one/harmony/consensus/engine"
|
|
|
|
"github.com/harmony-one/harmony/core/state"
|
|
|
|
"github.com/harmony-one/harmony/core/vm"
|
|
|
|
"github.com/harmony-one/harmony/crypto/hash"
|
|
|
|
"github.com/harmony-one/harmony/numeric"
|
Resolve harmony-one/bounties#77: Staking precompiles (#3906)
* Resolve harmony-one/bounties#77: Staking precompiles
Create write capable precompiles that can perform staking transactions
Add hard fork logic (EpochTBD) for these precompiles
Tests for new code with at least 80% unit test coverage
Staking library + tests in MaxMustermann2/harmony-staking-precompiles
* Fix small typo in comment
* Run goimports on files to fix Travis
* Do not activate staking precompile on shard 0
* Cascade readOnly to WriteCapableContract
* No overlap in readOnly + writeCapable precompiles
* Use function selector instead of directive
From Solidity, use abi.encodeWithSelector and match it against the
exact ABI of the functions. This allows us to remove the need for
a directive (32) being encoded, and thus saves 28 bytes of data.
* Do not allow contracts to become validators
As discussed with Jacky on #3906
* Merge harmony-one/harmony/main properly this time
* Run goimports
* Update gas calculation for staking precompile
Please see comment in core/vm/contracts_write.go RequiredGas
* Do not allow contract to become validator (2/2)
* Cache StakeMsgs from precompiled transactions
Add the StakeMsgs to ProcessorResult and cascade them in insertChain
* Remove ContractCode fields from validators
Since smart contracts can no longer beecome validators,
this field is superfluous. Remove it from the Wrapper
structure, and do not assign it a value when creating
a validator. Build and goimports checked
* Update comments in response to feedback
(1) Comments to start with function names
(2) Comments for public variables
(3) Comment to match function name RunPrecompiledContract
(4) Clarify that CreateValidatorFunc + EditValidatorFunc are still used
* Fix Travis build by reverting rosetta change
* Add revert capability to 3 staking tx types
- Delegate
- Undelegate
- CollectRewards
* Fix build: Update evm_test for ValidatorWrapper
* Merge main into harmony-staking-precompiles
* Add gas for precompile calls and allow EOA usage
- Each time the precompile is called, charge the base gas fee plus data
cost (if data can be parsed successfully). A gas fee is added to
prevent benevolent contract deployers from subsidizing the staking
transactions for EOAs through repeated assembly `delegatecall`.
- Allow EOAs to use the staking precompile directly. Some changes to
the Solidity library are associated with this change.
- Remove bytes from parsing address, since the ABI unpacks it into an
address format correctly.
- Add or update tests. Test coverage report to be attached to the PR
shortly.
* Run goimports
* Check read only and write capable for overlap
* Handle precompile stakeMsgs for block proposer
The staking precompile generates staking messages which are cascaded to
the block via the EVM in `state_processor.go`. This change cascades them
in `worker.go` to allow block proposers and block verifiers to keep the
same state.
* Run goimports for cf2dfac4081444e36a120c9432f4e..
* Update staking precompile epoch to 2 for localnet
Bring it in line with staking epoch. Change effects all configurations
except mainnet and testnet. `goimports` included.
* Add read only precompile to fetch the epoch num
* Move epoch precompile to 250
* precompiles: left pad the returned epoch number
* chainConfig: check epochs for precompiles
panic if staking precompile epoch < pre staking epoch
* Add staking migration precompile
- Lives at address 251
- Migrates delegations + pending undelegations from address A to B
- Useful if address A is hacked
- Charges gas of 21k + cost of bytes for two addresses
- Does not remove existing delegations, just sets them to zero.
Replicates current undelegate setup
- Unit tests and `goimports` included. Integration test following
shortly in MaxMustermann2/harmony-staking-precompiles
* Migration precompile: merge into staking
Merge the two precompiles into one, add gas calculation for migration
precompile. Move epoch precompile to 251 as a result. When migrating,
add undelegations to `To`'s existing undelegations, if any match the
epoch.
* Add migration gas test, remove panic, add check
In response to review comments, add tests for migration gas wherein
there are 0/1/2 delegations to migrate. Add the index out of bound check
to migration gas calculator and remove panics. Lastly, re-sort
migrated undelegations if no existing undelegation in the same epoch was
found on `To`.
* Move undelegations sorting to end of loop
3 years ago
|
|
|
"github.com/harmony-one/harmony/shard"
|
|
|
|
"github.com/harmony-one/harmony/staking/effective"
|
|
|
|
staking "github.com/harmony-one/harmony/staking/types"
|
|
|
|
staketest "github.com/harmony-one/harmony/staking/types/test"
|
|
|
|
)
|
|
|
|
|
|
|
|
const (
|
|
|
|
defNumWrappersInState = 5
|
|
|
|
defNumPubPerAddr = 2
|
|
|
|
|
|
|
|
validatorIndex = 0
|
|
|
|
validator2Index = 1
|
|
|
|
delegatorIndex = 6
|
|
|
|
)
|
|
|
|
|
|
|
|
var (
|
|
|
|
blsKeys = makeKeyPairs(20)
|
|
|
|
|
|
|
|
createValidatorAddr = makeTestAddr("validator")
|
|
|
|
validatorAddr = makeTestAddr(validatorIndex)
|
|
|
|
validatorAddr2 = makeTestAddr(validator2Index)
|
|
|
|
delegatorAddr = makeTestAddr(delegatorIndex)
|
|
|
|
)
|
|
|
|
|
|
|
|
var (
|
|
|
|
oneBig = big.NewInt(1e18)
|
|
|
|
fiveKOnes = new(big.Int).Mul(big.NewInt(5000), oneBig)
|
|
|
|
tenKOnes = new(big.Int).Mul(big.NewInt(10000), oneBig)
|
|
|
|
twelveKOnes = new(big.Int).Mul(big.NewInt(12000), oneBig)
|
|
|
|
fifteenKOnes = new(big.Int).Mul(big.NewInt(15000), oneBig)
|
|
|
|
twentyKOnes = new(big.Int).Mul(big.NewInt(20000), oneBig)
|
|
|
|
twentyFiveKOnes = new(big.Int).Mul(big.NewInt(25000), oneBig)
|
|
|
|
thirtyKOnes = new(big.Int).Mul(big.NewInt(30000), oneBig)
|
|
|
|
hundredKOnes = new(big.Int).Mul(big.NewInt(100000), oneBig)
|
|
|
|
|
|
|
|
negRate = numeric.NewDecWithPrec(-1, 10)
|
|
|
|
pointZeroOneDec = numeric.NewDecWithPrec(1, 2)
|
|
|
|
pointOneDec = numeric.NewDecWithPrec(1, 1)
|
|
|
|
pointTwoDec = numeric.NewDecWithPrec(2, 1)
|
|
|
|
pointFourDec = numeric.NewDecWithPrec(4, 1)
|
|
|
|
pointFiveDec = numeric.NewDecWithPrec(5, 1)
|
|
|
|
pointSevenDec = numeric.NewDecWithPrec(7, 1)
|
|
|
|
pointEightFiveDec = numeric.NewDecWithPrec(85, 2)
|
|
|
|
pointNineDec = numeric.NewDecWithPrec(9, 1)
|
|
|
|
oneDec = numeric.OneDec()
|
|
|
|
)
|
|
|
|
|
|
|
|
const (
|
|
|
|
defaultEpoch = 5
|
|
|
|
defaultNextEpoch = 6
|
|
|
|
defaultSnapBlockNumber = 90
|
|
|
|
defaultBlockNumber = 100
|
|
|
|
)
|
|
|
|
|
|
|
|
var (
|
|
|
|
defaultDesc = staking.Description{
|
|
|
|
Name: "SuperHero",
|
|
|
|
Identity: "YouWouldNotKnow",
|
|
|
|
Website: "Secret Website",
|
|
|
|
SecurityContact: "LicenseToKill",
|
|
|
|
Details: "blah blah blah",
|
|
|
|
}
|
|
|
|
|
|
|
|
defaultCommissionRates = staking.CommissionRates{
|
|
|
|
Rate: pointOneDec,
|
|
|
|
MaxRate: pointNineDec,
|
|
|
|
MaxChangeRate: pointFiveDec,
|
|
|
|
}
|
|
|
|
)
|
|
|
|
|
|
|
|
func TestCheckDuplicateFields(t *testing.T) {
|
|
|
|
tests := []struct {
|
|
|
|
bc ChainContext
|
|
|
|
sdb *state.DB
|
|
|
|
validator common.Address
|
|
|
|
identity string
|
|
|
|
pubs []bls.SerializedPublicKey
|
|
|
|
|
|
|
|
expErr error
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
// new validator
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
validator: createValidatorAddr,
|
|
|
|
identity: makeIdentityStr("new validator"),
|
|
|
|
pubs: []bls.SerializedPublicKey{blsKeys[11].pub},
|
|
|
|
|
|
|
|
expErr: nil,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// validator skip self check
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
validator: makeTestAddr(0),
|
|
|
|
identity: makeIdentityStr(0),
|
|
|
|
pubs: []bls.SerializedPublicKey{blsKeys[0].pub, blsKeys[1].pub},
|
|
|
|
|
|
|
|
expErr: nil,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// empty bls keys
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
validator: createValidatorAddr,
|
|
|
|
identity: makeIdentityStr("new validator"),
|
|
|
|
pubs: []bls.SerializedPublicKey{},
|
|
|
|
|
|
|
|
expErr: nil,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// empty identity will not collide
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
sdb: func(t *testing.T) *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
Resolve harmony-one/bounties#90: Add revert mechanism for UpdateValidatorWrapper (#3939)
* Add revert mechanism for UpdateValidatorWrapper
Closes harmony-one/bounties#90
(1) Use LRU for ValidatorWrapper objects in stateDB to plug a potential
memory leak
(2) Merge ValidatorWrapper and ValidatorWrapperCopy to let callers ask
for either a copy, or a pointer to the cached object. Additionally,
give callers the option to not deep copy delegations (which is a
heavy process). Copies need to be explicitly committed (and thus
can be reverted), while the pointers are committed when Finalise
is called.
(3) Add a UpdateValidatorWrapperWithRevert function, which is used by
staking txs `Delegate`, `Undelegate`, and `CollectRewards`. Other
2 types of staking txs and `db.Finalize` continue to use
UpdateValidateWrapper without revert, again, to save memoery
(4) Add unit tests which check
a) Revert goes through
b) Wrapper is as expected after revert
c) State is as expected after revert
* Change back to dictionary for stateValidators
Since the memory / CPU usage saved is not significantly different when
using an LRU + map structure, go back to the original dictionary
structure to keep code easy to read and have limited modifications.
* Add tests for validator wrapper reverts
As requested by @rlan35, add tests beyond just adding and reverting a
delegation. The tests are successive in the sense that we do multiple
modifications to the wrapper, save a snapshot before each modification
and revert to each of them to confirm everything works well. This change
improves test coverage of statedb.go to 66.7% from 64.8% and that of
core/state to 71.9% from 70.8%, and covers all the code that has been
modified by this PR in statedb.go.
For clarity, the modifications to the wrapper include (1) creation of
wrapper in state, (2) adding a delegation to the wrapper, (3)
increasing the blocks signed, and (4) a change in the validator Name and
the BlockReward. Two additional tests have been added to cover the
`panic` and the `GetCode` cases.
3 years ago
|
|
|
vw, err := sdb.ValidatorWrapper(makeTestAddr(0), false, true)
|
|
|
|
if err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
vw.Identity = ""
|
|
|
|
|
|
|
|
err = sdb.UpdateValidatorWrapper(makeTestAddr(0), vw)
|
|
|
|
if err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
return sdb
|
|
|
|
}(t),
|
|
|
|
validator: createValidatorAddr,
|
|
|
|
identity: makeIdentityStr("new validator"),
|
|
|
|
pubs: []bls.SerializedPublicKey{blsKeys[11].pub},
|
|
|
|
|
|
|
|
expErr: nil,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// chain error
|
|
|
|
bc: &fakeErrChainContext{},
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
validator: createValidatorAddr,
|
|
|
|
identity: makeIdentityStr("new validator"),
|
|
|
|
pubs: []bls.SerializedPublicKey{blsKeys[11].pub},
|
|
|
|
|
|
|
|
expErr: errors.New("error intended"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// validators read from chain not in state
|
|
|
|
bc: func() *fakeChainContext {
|
|
|
|
chain := makeFakeChainContextForStake()
|
|
|
|
addr := makeTestAddr("not exist in state")
|
|
|
|
w := staketest.GetDefaultValidatorWrapper()
|
|
|
|
chain.vWrappers[addr] = &w
|
|
|
|
return chain
|
|
|
|
}(),
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
validator: createValidatorAddr,
|
|
|
|
identity: makeIdentityStr("new validator"),
|
|
|
|
pubs: []bls.SerializedPublicKey{blsKeys[11].pub},
|
|
|
|
|
|
|
|
expErr: errors.New("address not present in state"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// duplicate identity
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
validator: createValidatorAddr,
|
|
|
|
identity: makeIdentityStr(0),
|
|
|
|
pubs: []bls.SerializedPublicKey{blsKeys[11].pub},
|
|
|
|
|
|
|
|
expErr: errDupIdentity,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// bls key duplication
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
validator: createValidatorAddr,
|
|
|
|
identity: makeIdentityStr("new validator"),
|
|
|
|
pubs: []bls.SerializedPublicKey{blsKeys[0].pub},
|
|
|
|
|
|
|
|
expErr: errDupBlsKey,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
for i, test := range tests {
|
|
|
|
err := checkDuplicateFields(test.bc, test.sdb, test.validator, test.identity, test.pubs)
|
|
|
|
|
|
|
|
if assErr := assertError(err, test.expErr); assErr != nil {
|
|
|
|
t.Errorf("Test %v: %v", i, assErr)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestVerifyAndCreateValidatorFromMsg(t *testing.T) {
|
|
|
|
tests := []struct {
|
|
|
|
sdb vm.StateDB
|
|
|
|
chain ChainContext
|
|
|
|
epoch *big.Int
|
|
|
|
blockNum *big.Int
|
|
|
|
msg staking.CreateValidator
|
|
|
|
|
|
|
|
expWrapper staking.ValidatorWrapper
|
|
|
|
expErr error
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
// 0: valid request
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgCreateValidator(),
|
|
|
|
|
|
|
|
expWrapper: defaultExpWrapperCreateValidator(),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 1: nil state db
|
|
|
|
sdb: nil,
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgCreateValidator(),
|
|
|
|
|
|
|
|
expErr: errStateDBIsMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 2: nil chain context
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
chain: nil,
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgCreateValidator(),
|
|
|
|
|
|
|
|
expErr: errChainContextMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 3: nil epoch
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: nil,
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgCreateValidator(),
|
|
|
|
|
|
|
|
expErr: errEpochMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 4: nil block number
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: nil,
|
|
|
|
msg: defaultMsgCreateValidator(),
|
|
|
|
|
|
|
|
expErr: errBlockNumMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 5: negative amount
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.CreateValidator {
|
|
|
|
m := defaultMsgCreateValidator()
|
|
|
|
m.Amount = big.NewInt(-1)
|
|
|
|
return m
|
|
|
|
}(),
|
|
|
|
expErr: errNegativeAmount,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 6: the address isValidatorFlag is true
|
|
|
|
sdb: func() *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
sdb.SetValidatorFlag(createValidatorAddr)
|
|
|
|
return sdb
|
|
|
|
}(),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgCreateValidator(),
|
|
|
|
|
|
|
|
expErr: errValidatorExist,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 7: bls collision (checkDuplicateFields)
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.CreateValidator {
|
|
|
|
m := defaultMsgCreateValidator()
|
|
|
|
m.SlotPubKeys = []bls.SerializedPublicKey{blsKeys[0].pub}
|
|
|
|
return m
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("BLS key exists"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 8: insufficient balance
|
|
|
|
sdb: func() *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
bal := new(big.Int).Sub(staketest.DefaultDelAmount, common.Big1)
|
|
|
|
sdb.SetBalance(createValidatorAddr, bal)
|
|
|
|
return sdb
|
|
|
|
}(),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgCreateValidator(),
|
|
|
|
|
|
|
|
expErr: errInsufficientBalanceForStake,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 9: incorrect signature
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.CreateValidator {
|
|
|
|
m := defaultMsgCreateValidator()
|
|
|
|
m.SlotKeySigs = []bls.SerializedSignature{blsKeys[12].sig}
|
|
|
|
return m
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("bls keys and corresponding signatures"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 10: small self delegation amount (fail sanity check)
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.CreateValidator {
|
|
|
|
m := defaultMsgCreateValidator()
|
|
|
|
m.Amount = new(big.Int).Sub(m.MinSelfDelegation, common.Big1)
|
|
|
|
return m
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("self delegation can not be less than min_self_delegation"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 11: amount exactly minSelfDelegation. Should not return error
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
chain: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.CreateValidator {
|
|
|
|
m := defaultMsgCreateValidator()
|
|
|
|
m.Amount = new(big.Int).Set(m.MinSelfDelegation)
|
|
|
|
return m
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expWrapper: func() staking.ValidatorWrapper {
|
|
|
|
w := defaultExpWrapperCreateValidator()
|
|
|
|
w.Delegations[0].Amount = new(big.Int).Set(w.MinSelfDelegation)
|
|
|
|
return w
|
|
|
|
}(),
|
|
|
|
},
|
|
|
|
}
|
|
|
|
for i, test := range tests {
|
|
|
|
w, err := VerifyAndCreateValidatorFromMsg(test.sdb, test.chain, test.epoch,
|
|
|
|
test.blockNum, &test.msg)
|
|
|
|
|
|
|
|
if assErr := assertError(err, test.expErr); assErr != nil {
|
|
|
|
t.Errorf("Test %v: %v", i, err)
|
|
|
|
}
|
|
|
|
if err != nil || test.expErr != nil {
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
if err := staketest.CheckValidatorWrapperEqual(*w, test.expWrapper); err != nil {
|
|
|
|
t.Errorf("Test %v: %v", i, err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultMsgCreateValidator() staking.CreateValidator {
|
|
|
|
pub, sig := blsKeys[11].pub, blsKeys[11].sig
|
|
|
|
cv := staking.CreateValidator{
|
|
|
|
ValidatorAddress: createValidatorAddr,
|
|
|
|
Description: defaultDesc,
|
|
|
|
CommissionRates: defaultCommissionRates,
|
|
|
|
MinSelfDelegation: staketest.DefaultMinSelfDel,
|
|
|
|
MaxTotalDelegation: staketest.DefaultMaxTotalDel,
|
|
|
|
SlotPubKeys: []bls.SerializedPublicKey{pub},
|
|
|
|
SlotKeySigs: []bls.SerializedSignature{sig},
|
|
|
|
Amount: staketest.DefaultDelAmount,
|
|
|
|
}
|
|
|
|
return cv
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultExpWrapperCreateValidator() staking.ValidatorWrapper {
|
|
|
|
pub := blsKeys[11].pub
|
|
|
|
v := staking.Validator{
|
|
|
|
Address: createValidatorAddr,
|
|
|
|
SlotPubKeys: []bls.SerializedPublicKey{pub},
|
|
|
|
LastEpochInCommittee: new(big.Int),
|
|
|
|
MinSelfDelegation: staketest.DefaultMinSelfDel,
|
|
|
|
MaxTotalDelegation: staketest.DefaultMaxTotalDel,
|
|
|
|
Status: effective.Active,
|
|
|
|
Commission: staking.Commission{
|
|
|
|
CommissionRates: defaultCommissionRates,
|
|
|
|
UpdateHeight: big.NewInt(defaultBlockNumber),
|
|
|
|
},
|
|
|
|
Description: defaultDesc,
|
|
|
|
CreationHeight: big.NewInt(defaultBlockNumber),
|
|
|
|
}
|
|
|
|
ds := staking.Delegations{
|
|
|
|
staking.NewDelegation(createValidatorAddr, staketest.DefaultDelAmount),
|
|
|
|
}
|
|
|
|
w := staking.ValidatorWrapper{
|
|
|
|
Validator: v,
|
|
|
|
Delegations: ds,
|
|
|
|
BlockReward: big.NewInt(0),
|
|
|
|
}
|
|
|
|
w.Counters.NumBlocksSigned = common.Big0
|
|
|
|
w.Counters.NumBlocksToSign = common.Big0
|
|
|
|
return w
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestVerifyAndEditValidatorFromMsg(t *testing.T) {
|
|
|
|
tests := []struct {
|
|
|
|
sdb vm.StateDB
|
|
|
|
bc ChainContext
|
|
|
|
epoch, blockNum *big.Int
|
|
|
|
msg staking.EditValidator
|
|
|
|
expWrapper staking.ValidatorWrapper
|
|
|
|
expErr error
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
// 0: positive case
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgEditValidator(),
|
|
|
|
|
|
|
|
expWrapper: defaultExpWrapperEditValidator(),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 1: If the rate is not changed, UpdateHeight is not update
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.CommissionRate = nil
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expWrapper: func() staking.ValidatorWrapper {
|
|
|
|
vw := defaultExpWrapperEditValidator()
|
|
|
|
vw.UpdateHeight = big.NewInt(defaultSnapBlockNumber)
|
|
|
|
vw.Rate = pointFourDec
|
|
|
|
return vw
|
|
|
|
}(),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 2: nil state db
|
|
|
|
sdb: nil,
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgEditValidator(),
|
|
|
|
|
|
|
|
expErr: errStateDBIsMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 3: nil chain
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: nil,
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgEditValidator(),
|
|
|
|
|
|
|
|
expErr: errChainContextMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 4: nil block number
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: nil,
|
|
|
|
msg: defaultMsgEditValidator(),
|
|
|
|
|
|
|
|
expErr: errBlockNumMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 5: edited validator flag not set in state
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.ValidatorAddress = makeTestAddr("addr not in chain")
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errValidatorNotExist,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 6: bls key collision
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.SlotKeyToAdd = &blsKeys[3].pub
|
|
|
|
msg.SlotKeyToAddSig = &blsKeys[3].sig
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errDupBlsKey,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 7: validatorWrapper not in state
|
|
|
|
sdb: func() *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
sdb.SetValidatorFlag(makeTestAddr("someone"))
|
|
|
|
return sdb
|
|
|
|
}(),
|
|
|
|
bc: func() *fakeChainContext {
|
|
|
|
chain := makeFakeChainContextForStake()
|
|
|
|
addr := makeTestAddr("someone")
|
|
|
|
w := staketest.GetDefaultValidatorWrapperWithAddr(addr, nil)
|
|
|
|
chain.vWrappers[addr] = &w
|
|
|
|
return chain
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.ValidatorAddress = makeTestAddr("someone")
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("address not present in state"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 8: signature cannot be verified
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.SlotKeyToAddSig = &blsKeys[13].sig
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("bls keys and corresponding signatures could not be verified"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 9: Rate is greater the maxRate
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: func() *fakeChainContext {
|
|
|
|
chain := makeFakeChainContextForStake()
|
|
|
|
vw := chain.vWrappers[validatorAddr]
|
|
|
|
vw.Rate = pointSevenDec
|
|
|
|
chain.vWrappers[validatorAddr] = vw
|
|
|
|
return chain
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.CommissionRate = &oneDec
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errCommissionRateChangeTooHigh,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 10: validator not in snapshot
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: func() *fakeChainContext {
|
|
|
|
chain := makeFakeChainContextForStake()
|
|
|
|
delete(chain.vWrappers, validatorAddr)
|
|
|
|
return chain
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgEditValidator(),
|
|
|
|
|
|
|
|
expErr: errors.New("validator snapshot not found"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 11: rate is greater than maxChangeRate
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.CommissionRate = &pointEightFiveDec
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errCommissionRateChangeTooFast,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 12: fails in sanity check (rate below zero)
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: func() *fakeChainContext {
|
|
|
|
chain := makeFakeChainContextForStake()
|
|
|
|
vw := chain.vWrappers[validatorAddr]
|
|
|
|
vw.Rate = pointOneDec
|
|
|
|
chain.vWrappers[validatorAddr] = vw
|
|
|
|
return chain
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.CommissionRate = &negRate
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("rate should be a value ranging from 0.0 to 1.0"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 13: cannot update a banned validator
|
|
|
|
sdb: func(t *testing.T) *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
Resolve harmony-one/bounties#90: Add revert mechanism for UpdateValidatorWrapper (#3939)
* Add revert mechanism for UpdateValidatorWrapper
Closes harmony-one/bounties#90
(1) Use LRU for ValidatorWrapper objects in stateDB to plug a potential
memory leak
(2) Merge ValidatorWrapper and ValidatorWrapperCopy to let callers ask
for either a copy, or a pointer to the cached object. Additionally,
give callers the option to not deep copy delegations (which is a
heavy process). Copies need to be explicitly committed (and thus
can be reverted), while the pointers are committed when Finalise
is called.
(3) Add a UpdateValidatorWrapperWithRevert function, which is used by
staking txs `Delegate`, `Undelegate`, and `CollectRewards`. Other
2 types of staking txs and `db.Finalize` continue to use
UpdateValidateWrapper without revert, again, to save memoery
(4) Add unit tests which check
a) Revert goes through
b) Wrapper is as expected after revert
c) State is as expected after revert
* Change back to dictionary for stateValidators
Since the memory / CPU usage saved is not significantly different when
using an LRU + map structure, go back to the original dictionary
structure to keep code easy to read and have limited modifications.
* Add tests for validator wrapper reverts
As requested by @rlan35, add tests beyond just adding and reverting a
delegation. The tests are successive in the sense that we do multiple
modifications to the wrapper, save a snapshot before each modification
and revert to each of them to confirm everything works well. This change
improves test coverage of statedb.go to 66.7% from 64.8% and that of
core/state to 71.9% from 70.8%, and covers all the code that has been
modified by this PR in statedb.go.
For clarity, the modifications to the wrapper include (1) creation of
wrapper in state, (2) adding a delegation to the wrapper, (3)
increasing the blocks signed, and (4) a change in the validator Name and
the BlockReward. Two additional tests have been added to cover the
`panic` and the `GetCode` cases.
3 years ago
|
|
|
vw, err := sdb.ValidatorWrapper(validatorAddr, false, true)
|
|
|
|
if err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
vw.Status = effective.Banned
|
|
|
|
if err := sdb.UpdateValidatorWrapper(validatorAddr, vw); err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
return sdb
|
|
|
|
}(t),
|
|
|
|
bc: makeFakeChainContextForStake(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: defaultMsgEditValidator(),
|
|
|
|
|
|
|
|
expErr: errors.New("banned status"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 14: Rate is lower than min rate of 5%
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: func() *fakeChainContext {
|
|
|
|
chain := makeFakeChainContextForStake()
|
|
|
|
vw := chain.vWrappers[validatorAddr]
|
|
|
|
vw.Rate = pointFourDec
|
|
|
|
chain.vWrappers[validatorAddr] = vw
|
|
|
|
return chain
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(20),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.CommissionRate = &pointZeroOneDec
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errCommissionRateChangeTooLowT,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 15: Rate is ok within the promo period
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
bc: func() *fakeChainContext {
|
|
|
|
chain := makeFakeChainContextForStake()
|
|
|
|
vw := chain.vWrappers[validatorAddr]
|
|
|
|
vw.Rate = pointFourDec
|
|
|
|
chain.vWrappers[validatorAddr] = vw
|
|
|
|
return chain
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(15),
|
|
|
|
blockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
msg: func() staking.EditValidator {
|
|
|
|
msg := defaultMsgEditValidator()
|
|
|
|
msg.CommissionRate = &pointZeroOneDec
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expWrapper: func() staking.ValidatorWrapper {
|
|
|
|
vw := defaultExpWrapperEditValidator()
|
|
|
|
vw.UpdateHeight = big.NewInt(defaultBlockNumber)
|
|
|
|
vw.Rate = pointZeroOneDec
|
|
|
|
return vw
|
|
|
|
}(),
|
|
|
|
},
|
|
|
|
}
|
|
|
|
for i, test := range tests {
|
|
|
|
w, err := VerifyAndEditValidatorFromMsg(test.sdb, test.bc, test.epoch, test.blockNum,
|
|
|
|
&test.msg)
|
|
|
|
|
|
|
|
if assErr := assertError(err, test.expErr); assErr != nil {
|
|
|
|
t.Errorf("Test %v: unexpected Error: %v", i, assErr)
|
|
|
|
}
|
|
|
|
if err != nil || test.expErr != nil {
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
if err := staketest.CheckValidatorWrapperEqual(*w, test.expWrapper); err != nil {
|
|
|
|
t.Errorf("Test %v: %v", i, err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
var (
|
|
|
|
editDesc = staking.Description{
|
|
|
|
Name: "batman",
|
|
|
|
Identity: "batman",
|
|
|
|
Website: "",
|
|
|
|
SecurityContact: "",
|
|
|
|
Details: "",
|
|
|
|
}
|
|
|
|
|
|
|
|
editExpDesc = staking.Description{
|
|
|
|
Name: "batman",
|
|
|
|
Identity: "batman",
|
|
|
|
Website: "Secret Website",
|
|
|
|
SecurityContact: "LicenseToKill",
|
|
|
|
Details: "blah blah blah",
|
|
|
|
}
|
|
|
|
)
|
|
|
|
|
|
|
|
func defaultMsgEditValidator() staking.EditValidator {
|
|
|
|
var (
|
|
|
|
pub0Copy bls.SerializedPublicKey
|
|
|
|
pub12Copy bls.SerializedPublicKey
|
|
|
|
sig12Copy bls.SerializedSignature
|
|
|
|
)
|
|
|
|
copy(pub0Copy[:], blsKeys[0].pub[:])
|
|
|
|
copy(pub12Copy[:], blsKeys[12].pub[:])
|
|
|
|
copy(sig12Copy[:], blsKeys[12].sig[:])
|
|
|
|
|
|
|
|
return staking.EditValidator{
|
|
|
|
ValidatorAddress: validatorAddr,
|
|
|
|
Description: editDesc,
|
|
|
|
CommissionRate: &pointTwoDec,
|
|
|
|
SlotKeyToRemove: &pub0Copy,
|
|
|
|
SlotKeyToAdd: &pub12Copy,
|
|
|
|
SlotKeyToAddSig: &sig12Copy,
|
|
|
|
EPOSStatus: effective.Inactive,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultExpWrapperEditValidator() staking.ValidatorWrapper {
|
|
|
|
w := makeVWrapperByIndex(validatorIndex)
|
|
|
|
w.SlotPubKeys = append(w.SlotPubKeys[1:], blsKeys[12].pub)
|
|
|
|
w.Description = editExpDesc
|
|
|
|
w.Rate = pointTwoDec
|
|
|
|
w.UpdateHeight = big.NewInt(defaultBlockNumber)
|
|
|
|
w.Status = effective.Inactive
|
|
|
|
return w
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestVerifyAndDelegateFromMsg(t *testing.T) {
|
|
|
|
tests := []struct {
|
|
|
|
sdb vm.StateDB
|
|
|
|
msg staking.Delegate
|
|
|
|
ds []staking.DelegationIndex
|
|
|
|
epoch *big.Int
|
|
|
|
redelegate bool
|
|
|
|
|
|
|
|
expVWrappers []staking.ValidatorWrapper
|
|
|
|
expAmt *big.Int
|
|
|
|
expRedel map[common.Address]*big.Int
|
|
|
|
expErr error
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
// 0: new delegate
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
msg: defaultMsgDelegate(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expVWrappers: []staking.ValidatorWrapper{defaultExpVWrapperDelegate()},
|
|
|
|
expAmt: tenKOnes,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 1: add amount to current delegate
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
msg: defaultMsgSelfDelegate(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expVWrappers: []staking.ValidatorWrapper{defaultExpVWrapperSelfDelegate()},
|
|
|
|
expAmt: tenKOnes,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 2: nil state db
|
|
|
|
sdb: nil,
|
|
|
|
msg: defaultMsgDelegate(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expErr: errStateDBIsMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 3: validatorFlag not set
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
msg := defaultMsgDelegate()
|
|
|
|
msg.ValidatorAddress = makeTestAddr("not in state")
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expErr: errValidatorNotExist,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 4: negative amount
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
msg := defaultMsgDelegate()
|
|
|
|
msg.Amount = big.NewInt(-1)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expErr: errNegativeAmount,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 5: small amount
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
msg := defaultMsgDelegate()
|
|
|
|
msg.Amount = big.NewInt(100)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expErr: errDelegationTooSmall,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 6: missing validator wrapper
|
|
|
|
sdb: func() *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
sdb.SetValidatorFlag(createValidatorAddr)
|
|
|
|
return sdb
|
|
|
|
}(),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
d := defaultMsgDelegate()
|
|
|
|
d.ValidatorAddress = createValidatorAddr
|
|
|
|
return d
|
|
|
|
}(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expErr: errors.New("address not present in state"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 7: cannot transfer since not enough amount
|
|
|
|
sdb: func() *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
sdb.SetBalance(delegatorAddr, big.NewInt(100))
|
|
|
|
return sdb
|
|
|
|
}(),
|
|
|
|
msg: defaultMsgDelegate(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expErr: errInsufficientBalanceForStake,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 8: self delegation not pass sanity check
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
d := defaultMsgSelfDelegate()
|
|
|
|
d.Amount = hundredKOnes
|
|
|
|
return d
|
|
|
|
}(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expErr: errors.New(" total delegation can not be bigger than max_total_delegation"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 9: Delegation does not pass sanity check
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
d := defaultMsgDelegate()
|
|
|
|
d.Amount = hundredKOnes
|
|
|
|
return d
|
|
|
|
}(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expErr: errors.New(" total delegation can not be bigger than max_total_delegation"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 10: full redelegate without using balance
|
|
|
|
sdb: makeStateForRedelegate(t),
|
|
|
|
msg: defaultMsgDelegate(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(7),
|
|
|
|
redelegate: true,
|
|
|
|
|
|
|
|
expVWrappers: defaultExpVWrappersRedelegate(),
|
|
|
|
expAmt: big.NewInt(0),
|
|
|
|
expRedel: map[common.Address]*big.Int{
|
|
|
|
validatorAddr: fiveKOnes,
|
|
|
|
validatorAddr2: fiveKOnes,
|
|
|
|
},
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 11: redelegate with undelegation epoch too recent, have to use some balance
|
|
|
|
sdb: makeStateForRedelegate(t),
|
|
|
|
msg: defaultMsgDelegate(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(6),
|
|
|
|
redelegate: true,
|
|
|
|
|
|
|
|
expVWrappers: func() []staking.ValidatorWrapper {
|
|
|
|
wrappers := defaultExpVWrappersRedelegate()
|
|
|
|
wrappers[1].Delegations[1].Undelegations = append(
|
|
|
|
wrappers[1].Delegations[1].Undelegations,
|
|
|
|
staking.Undelegation{Amount: fiveKOnes, Epoch: big.NewInt(defaultNextEpoch)})
|
|
|
|
return wrappers
|
|
|
|
}(),
|
|
|
|
expAmt: fiveKOnes,
|
|
|
|
expRedel: map[common.Address]*big.Int{
|
|
|
|
validatorAddr: fiveKOnes,
|
|
|
|
},
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 12: redelegate with not enough undelegated token, have to use some balance
|
|
|
|
sdb: makeStateForRedelegate(t),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
del := defaultMsgDelegate()
|
|
|
|
del.Amount = twentyKOnes
|
|
|
|
return del
|
|
|
|
}(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(7),
|
|
|
|
redelegate: true,
|
|
|
|
|
|
|
|
expVWrappers: func() []staking.ValidatorWrapper {
|
|
|
|
wrappers := defaultExpVWrappersRedelegate()
|
|
|
|
wrappers[0].Delegations[1].Amount = big.NewInt(0).Add(twentyKOnes, twentyKOnes)
|
|
|
|
return wrappers
|
|
|
|
}(),
|
|
|
|
expAmt: tenKOnes,
|
|
|
|
expRedel: map[common.Address]*big.Int{
|
|
|
|
validatorAddr: fiveKOnes,
|
|
|
|
validatorAddr2: fiveKOnes,
|
|
|
|
},
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 13: no redelegation and full balance used
|
|
|
|
sdb: makeStateForRedelegate(t),
|
|
|
|
msg: defaultMsgDelegate(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(5),
|
|
|
|
redelegate: true,
|
|
|
|
|
|
|
|
expVWrappers: func() []staking.ValidatorWrapper {
|
|
|
|
wrappers := defaultExpVWrappersRedelegate()
|
|
|
|
wrappers[0].Delegations[1].Undelegations = append(
|
|
|
|
wrappers[0].Delegations[1].Undelegations,
|
|
|
|
staking.Undelegation{Amount: fiveKOnes, Epoch: big.NewInt(defaultEpoch)})
|
|
|
|
wrappers[1].Delegations[1].Undelegations = append(
|
|
|
|
wrappers[1].Delegations[1].Undelegations,
|
|
|
|
staking.Undelegation{Amount: fiveKOnes, Epoch: big.NewInt(defaultNextEpoch)})
|
|
|
|
return wrappers
|
|
|
|
}(),
|
|
|
|
expAmt: tenKOnes,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 14: redelegate error delegation index out of bound
|
|
|
|
sdb: makeStateForRedelegate(t),
|
|
|
|
msg: defaultMsgSelfDelegate(),
|
|
|
|
ds: func() []staking.DelegationIndex {
|
|
|
|
dis := makeMsgCollectRewards()
|
|
|
|
dis[1].Index = 2
|
|
|
|
return dis
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: true,
|
|
|
|
|
|
|
|
expErr: errors.New("index out of bound"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 15: redelegate error delegation index out of bound
|
|
|
|
sdb: makeStateForRedelegate(t),
|
|
|
|
msg: defaultMsgSelfDelegate(),
|
|
|
|
ds: func() []staking.DelegationIndex {
|
|
|
|
dis := makeMsgCollectRewards()
|
|
|
|
dis[1].Index = 2
|
|
|
|
return dis
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(0),
|
|
|
|
redelegate: true,
|
|
|
|
|
|
|
|
expErr: errors.New("index out of bound"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 16: no redelegation and full balance used (validator delegate to self)
|
|
|
|
sdb: makeStateForRedelegate(t),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
del := defaultMsgDelegate()
|
|
|
|
del.DelegatorAddress = del.ValidatorAddress
|
|
|
|
return del
|
|
|
|
}(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(5),
|
|
|
|
redelegate: true,
|
|
|
|
|
|
|
|
expVWrappers: func() []staking.ValidatorWrapper {
|
|
|
|
wrappers := defaultExpVWrappersRedelegate()
|
|
|
|
wrappers[0].Delegations[1].Undelegations = append(
|
|
|
|
wrappers[0].Delegations[1].Undelegations,
|
|
|
|
staking.Undelegation{Amount: fiveKOnes, Epoch: big.NewInt(defaultEpoch)})
|
|
|
|
wrappers[1].Delegations[1].Undelegations = append(
|
|
|
|
wrappers[1].Delegations[1].Undelegations,
|
|
|
|
staking.Undelegation{Amount: fiveKOnes, Epoch: big.NewInt(defaultNextEpoch)})
|
|
|
|
|
|
|
|
wrappers[0].Delegations[1].Amount = twentyKOnes
|
|
|
|
wrappers[0].Delegations[0].Amount = big.NewInt(0).Add(twentyKOnes, tenKOnes)
|
|
|
|
return wrappers
|
|
|
|
}(),
|
|
|
|
expAmt: tenKOnes,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 17: small amount v2
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
msg := defaultMsgDelegate()
|
|
|
|
msg.Amount = new(big.Int).Mul(big.NewInt(90), oneBig)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(100),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expErr: errDelegationTooSmallV2,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 18: valid amount v2
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
msg: func() staking.Delegate {
|
|
|
|
msg := defaultMsgDelegate()
|
|
|
|
msg.Amount = new(big.Int).Mul(big.NewInt(500), oneBig)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
epoch: big.NewInt(100),
|
|
|
|
redelegate: false,
|
|
|
|
|
|
|
|
expVWrappers: func() []staking.ValidatorWrapper {
|
|
|
|
wrapper := defaultExpVWrapperDelegate()
|
|
|
|
wrapper.Delegations[1].Amount = new(big.Int).Mul(big.NewInt(500), oneBig)
|
|
|
|
return []staking.ValidatorWrapper{wrapper}
|
|
|
|
}(),
|
|
|
|
expAmt: new(big.Int).Mul(big.NewInt(500), oneBig),
|
|
|
|
},
|
|
|
|
}
|
|
|
|
for i, test := range tests {
|
|
|
|
config := ¶ms.ChainConfig{}
|
|
|
|
config.MinDelegation100Epoch = big.NewInt(100)
|
|
|
|
if test.redelegate {
|
|
|
|
config.RedelegationEpoch = test.epoch
|
|
|
|
} else {
|
|
|
|
config.RedelegationEpoch = big.NewInt(test.epoch.Int64() + 1)
|
|
|
|
}
|
|
|
|
ws, amt, amtRedel, err := VerifyAndDelegateFromMsg(test.sdb, test.epoch, &test.msg, test.ds, config)
|
|
|
|
|
|
|
|
if assErr := assertError(err, test.expErr); assErr != nil {
|
|
|
|
t.Errorf("Test %v: %v", i, assErr)
|
|
|
|
}
|
|
|
|
if err != nil || test.expErr != nil {
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
if amt.Cmp(test.expAmt) != 0 {
|
|
|
|
t.Errorf("Test %v: unexpected amount %v / %v", i, amt, test.expAmt)
|
|
|
|
}
|
|
|
|
|
|
|
|
if len(amtRedel) != len(test.expRedel) {
|
|
|
|
t.Errorf("Test %v: wrong expected redelegation length %d / %d", i, len(amtRedel), len(test.expRedel))
|
|
|
|
} else {
|
|
|
|
for key, value := range test.expRedel {
|
|
|
|
actValue, ok := amtRedel[key]
|
|
|
|
if !ok {
|
|
|
|
t.Errorf("Test %v: missing expected redelegation key/value %v / %v", i, key, value)
|
|
|
|
}
|
|
|
|
if value.Cmp(actValue) != 0 {
|
|
|
|
t.Errorf("Test %v: unexpeced redelegation value %v / %v", i, actValue, value)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
for j := range ws {
|
|
|
|
if err := staketest.CheckValidatorWrapperEqual(*ws[j], test.expVWrappers[j]); err != nil {
|
|
|
|
t.Errorf("Test %v: %v", i, err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeStateForRedelegate(t *testing.T) *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
|
|
|
|
if err := addStateUndelegationForAddr(sdb, validatorAddr, big.NewInt(defaultEpoch)); err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
if err := addStateUndelegationForAddr(sdb, validatorAddr2, big.NewInt(defaultNextEpoch)); err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
|
|
|
|
sdb.IntermediateRoot(true)
|
|
|
|
return sdb
|
|
|
|
}
|
|
|
|
|
|
|
|
func addStateUndelegationForAddr(sdb *state.DB, addr common.Address, epoch *big.Int) error {
|
Resolve harmony-one/bounties#90: Add revert mechanism for UpdateValidatorWrapper (#3939)
* Add revert mechanism for UpdateValidatorWrapper
Closes harmony-one/bounties#90
(1) Use LRU for ValidatorWrapper objects in stateDB to plug a potential
memory leak
(2) Merge ValidatorWrapper and ValidatorWrapperCopy to let callers ask
for either a copy, or a pointer to the cached object. Additionally,
give callers the option to not deep copy delegations (which is a
heavy process). Copies need to be explicitly committed (and thus
can be reverted), while the pointers are committed when Finalise
is called.
(3) Add a UpdateValidatorWrapperWithRevert function, which is used by
staking txs `Delegate`, `Undelegate`, and `CollectRewards`. Other
2 types of staking txs and `db.Finalize` continue to use
UpdateValidateWrapper without revert, again, to save memoery
(4) Add unit tests which check
a) Revert goes through
b) Wrapper is as expected after revert
c) State is as expected after revert
* Change back to dictionary for stateValidators
Since the memory / CPU usage saved is not significantly different when
using an LRU + map structure, go back to the original dictionary
structure to keep code easy to read and have limited modifications.
* Add tests for validator wrapper reverts
As requested by @rlan35, add tests beyond just adding and reverting a
delegation. The tests are successive in the sense that we do multiple
modifications to the wrapper, save a snapshot before each modification
and revert to each of them to confirm everything works well. This change
improves test coverage of statedb.go to 66.7% from 64.8% and that of
core/state to 71.9% from 70.8%, and covers all the code that has been
modified by this PR in statedb.go.
For clarity, the modifications to the wrapper include (1) creation of
wrapper in state, (2) adding a delegation to the wrapper, (3)
increasing the blocks signed, and (4) a change in the validator Name and
the BlockReward. Two additional tests have been added to cover the
`panic` and the `GetCode` cases.
3 years ago
|
|
|
w, err := sdb.ValidatorWrapper(addr, false, true)
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
w.Delegations = append(w.Delegations,
|
|
|
|
staking.NewDelegation(delegatorAddr, new(big.Int).Set(twentyKOnes)),
|
|
|
|
)
|
|
|
|
w.Delegations[1].Undelegations = staking.Undelegations{staking.Undelegation{Amount: fiveKOnes, Epoch: epoch}}
|
|
|
|
|
|
|
|
return sdb.UpdateValidatorWrapper(addr, w)
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultExpVWrappersRedelegate() []staking.ValidatorWrapper {
|
|
|
|
w1 := makeVWrapperByIndex(validatorIndex)
|
|
|
|
w1.Delegations = append(w1.Delegations,
|
|
|
|
staking.NewDelegation(delegatorAddr, new(big.Int).Set(thirtyKOnes)),
|
|
|
|
)
|
|
|
|
|
|
|
|
w2 := makeVWrapperByIndex(validator2Index)
|
|
|
|
w2.Delegations = append(w2.Delegations,
|
|
|
|
staking.NewDelegation(delegatorAddr, new(big.Int).Set(twentyKOnes)),
|
|
|
|
)
|
|
|
|
return []staking.ValidatorWrapper{w1, w2}
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultMsgDelegate() staking.Delegate {
|
|
|
|
return staking.Delegate{
|
|
|
|
DelegatorAddress: delegatorAddr,
|
|
|
|
ValidatorAddress: validatorAddr,
|
|
|
|
Amount: new(big.Int).Set(tenKOnes),
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultExpVWrapperDelegate() staking.ValidatorWrapper {
|
|
|
|
w := makeVWrapperByIndex(validatorIndex)
|
|
|
|
w.Delegations = append(w.Delegations, staking.NewDelegation(delegatorAddr, tenKOnes))
|
|
|
|
return w
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultMsgSelfDelegate() staking.Delegate {
|
|
|
|
return staking.Delegate{
|
|
|
|
DelegatorAddress: validatorAddr,
|
|
|
|
ValidatorAddress: validatorAddr,
|
|
|
|
Amount: new(big.Int).Set(tenKOnes),
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultExpVWrapperSelfDelegate() staking.ValidatorWrapper {
|
|
|
|
w := makeVWrapperByIndex(validatorIndex)
|
|
|
|
w.Delegations[0].Amount = new(big.Int).Add(tenKOnes, staketest.DefaultDelAmount)
|
|
|
|
return w
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestVerifyAndUndelegateFromMsg(t *testing.T) {
|
|
|
|
tests := []struct {
|
|
|
|
sdb vm.StateDB
|
|
|
|
epoch *big.Int
|
|
|
|
msg staking.Undelegate
|
|
|
|
|
|
|
|
expVWrapper staking.ValidatorWrapper
|
|
|
|
expErr error
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
// 0: Undelegate at delegation with an entry already exist at the same epoch.
|
|
|
|
// Will increase the amount in undelegate entry
|
|
|
|
sdb: makeDefaultStateForUndelegate(t),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: defaultMsgUndelegate(),
|
|
|
|
|
|
|
|
expVWrapper: defaultExpVWrapperUndelegateSameEpoch(t),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 1: Undelegate with undelegation entry exist but not in same epoch.
|
|
|
|
// Will create a new undelegate entry
|
|
|
|
sdb: makeDefaultStateForUndelegate(t),
|
|
|
|
epoch: big.NewInt(defaultNextEpoch),
|
|
|
|
msg: defaultMsgUndelegate(),
|
|
|
|
|
|
|
|
expVWrapper: defaultExpVWrapperUndelegateNextEpoch(t),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 2: Undelegate from a delegation record with no undelegation entry.
|
|
|
|
// Will create a new undelegate entry
|
|
|
|
sdb: makeDefaultStateForUndelegate(t),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: defaultMsgSelfUndelegate(),
|
|
|
|
|
|
|
|
expVWrapper: defaultVWrapperSelfUndelegate(t),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 3: Self delegation below min self delegation, change status to Inactive
|
|
|
|
sdb: makeDefaultStateForUndelegate(t),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: func() staking.Undelegate {
|
|
|
|
msg := defaultMsgSelfUndelegate()
|
|
|
|
msg.Amount = new(big.Int).Set(fifteenKOnes)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expVWrapper: func(t *testing.T) staking.ValidatorWrapper {
|
|
|
|
w := defaultVWrapperSelfUndelegate(t)
|
|
|
|
|
|
|
|
w.Delegations[0].Amount = new(big.Int).Set(fiveKOnes)
|
|
|
|
w.Delegations[0].Undelegations[0].Amount = new(big.Int).Set(fifteenKOnes)
|
|
|
|
w.Status = effective.Inactive
|
|
|
|
|
|
|
|
return w
|
|
|
|
}(t),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 4: Extract tokens from banned validator
|
|
|
|
sdb: func(t *testing.T) *state.DB {
|
|
|
|
sdb := makeDefaultStateForUndelegate(t)
|
Resolve harmony-one/bounties#90: Add revert mechanism for UpdateValidatorWrapper (#3939)
* Add revert mechanism for UpdateValidatorWrapper
Closes harmony-one/bounties#90
(1) Use LRU for ValidatorWrapper objects in stateDB to plug a potential
memory leak
(2) Merge ValidatorWrapper and ValidatorWrapperCopy to let callers ask
for either a copy, or a pointer to the cached object. Additionally,
give callers the option to not deep copy delegations (which is a
heavy process). Copies need to be explicitly committed (and thus
can be reverted), while the pointers are committed when Finalise
is called.
(3) Add a UpdateValidatorWrapperWithRevert function, which is used by
staking txs `Delegate`, `Undelegate`, and `CollectRewards`. Other
2 types of staking txs and `db.Finalize` continue to use
UpdateValidateWrapper without revert, again, to save memoery
(4) Add unit tests which check
a) Revert goes through
b) Wrapper is as expected after revert
c) State is as expected after revert
* Change back to dictionary for stateValidators
Since the memory / CPU usage saved is not significantly different when
using an LRU + map structure, go back to the original dictionary
structure to keep code easy to read and have limited modifications.
* Add tests for validator wrapper reverts
As requested by @rlan35, add tests beyond just adding and reverting a
delegation. The tests are successive in the sense that we do multiple
modifications to the wrapper, save a snapshot before each modification
and revert to each of them to confirm everything works well. This change
improves test coverage of statedb.go to 66.7% from 64.8% and that of
core/state to 71.9% from 70.8%, and covers all the code that has been
modified by this PR in statedb.go.
For clarity, the modifications to the wrapper include (1) creation of
wrapper in state, (2) adding a delegation to the wrapper, (3)
increasing the blocks signed, and (4) a change in the validator Name and
the BlockReward. Two additional tests have been added to cover the
`panic` and the `GetCode` cases.
3 years ago
|
|
|
w, err := sdb.ValidatorWrapper(validatorAddr, false, true)
|
|
|
|
if err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
w.Status = effective.Banned
|
|
|
|
if err := sdb.UpdateValidatorWrapper(validatorAddr, w); err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
return sdb
|
|
|
|
}(t),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: func() staking.Undelegate {
|
|
|
|
msg := defaultMsgSelfUndelegate()
|
|
|
|
msg.Amount = new(big.Int).Set(fifteenKOnes)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expVWrapper: func(t *testing.T) staking.ValidatorWrapper {
|
|
|
|
w := defaultVWrapperSelfUndelegate(t)
|
|
|
|
|
|
|
|
w.Delegations[0].Amount = new(big.Int).Set(fiveKOnes)
|
|
|
|
w.Delegations[0].Undelegations[0].Amount = new(big.Int).Set(fifteenKOnes)
|
|
|
|
w.Status = effective.Banned
|
|
|
|
|
|
|
|
return w
|
|
|
|
}(t),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 5: nil state db
|
|
|
|
sdb: nil,
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: defaultMsgUndelegate(),
|
|
|
|
|
|
|
|
expErr: errStateDBIsMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 6: nil epoch
|
|
|
|
sdb: makeDefaultStateForUndelegate(t),
|
|
|
|
epoch: nil,
|
|
|
|
msg: defaultMsgUndelegate(),
|
|
|
|
|
|
|
|
expErr: errEpochMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 7: negative amount
|
|
|
|
sdb: makeDefaultStateForUndelegate(t),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: func() staking.Undelegate {
|
|
|
|
msg := defaultMsgUndelegate()
|
|
|
|
msg.Amount = big.NewInt(-1)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errNegativeAmount,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 8: validator flag not set
|
|
|
|
sdb: func() *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
w := makeVWrapperByIndex(6)
|
|
|
|
if err := sdb.UpdateValidatorWrapper(makeTestAddr(6), &w); err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
return sdb
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: func() staking.Undelegate {
|
|
|
|
msg := defaultMsgUndelegate()
|
|
|
|
msg.ValidatorAddress = makeTestAddr(6)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errValidatorNotExist,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 9: vWrapper not in state
|
|
|
|
sdb: func() *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
sdb.SetValidatorFlag(makeTestAddr(6))
|
|
|
|
return sdb
|
|
|
|
}(),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: func() staking.Undelegate {
|
|
|
|
msg := defaultMsgUndelegate()
|
|
|
|
msg.ValidatorAddress = makeTestAddr(6)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("address not present in state"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 10: Insufficient balance to undelegate
|
|
|
|
sdb: makeDefaultStateForUndelegate(t),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: func() staking.Undelegate {
|
|
|
|
msg := defaultMsgUndelegate()
|
|
|
|
msg.Amount = new(big.Int).Set(hundredKOnes)
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("insufficient balance to undelegate"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 11: No delegation record
|
|
|
|
sdb: makeDefaultStateForUndelegate(t),
|
|
|
|
epoch: big.NewInt(defaultEpoch),
|
|
|
|
msg: func() staking.Undelegate {
|
|
|
|
msg := defaultMsgUndelegate()
|
|
|
|
msg.DelegatorAddress = makeTestAddr("not exist")
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errNoDelegationToUndelegate,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
for i, test := range tests {
|
|
|
|
w, err := VerifyAndUndelegateFromMsg(test.sdb, test.epoch, &test.msg)
|
|
|
|
|
|
|
|
if assErr := assertError(err, test.expErr); assErr != nil {
|
|
|
|
t.Errorf("Test %v: %v", i, assErr)
|
|
|
|
}
|
|
|
|
if err != nil || test.expErr != nil {
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
if err := staketest.CheckValidatorWrapperEqual(*w, test.expVWrapper); err != nil {
|
|
|
|
t.Errorf("Test %v: %v", i, err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeDefaultSnapVWrapperForUndelegate(t *testing.T) staking.ValidatorWrapper {
|
|
|
|
w := makeVWrapperByIndex(validatorIndex)
|
|
|
|
|
|
|
|
newDelegation := staking.NewDelegation(delegatorAddr, new(big.Int).Set(twentyKOnes))
|
|
|
|
if err := newDelegation.Undelegate(big.NewInt(defaultEpoch), fiveKOnes); err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
w.Delegations = append(w.Delegations, newDelegation)
|
|
|
|
|
|
|
|
return w
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeDefaultStateForUndelegate(t *testing.T) *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
w := makeDefaultSnapVWrapperForUndelegate(t)
|
|
|
|
|
|
|
|
if err := sdb.UpdateValidatorWrapper(validatorAddr, &w); err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
sdb.IntermediateRoot(true)
|
|
|
|
return sdb
|
|
|
|
}
|
|
|
|
|
|
|
|
// undelegate from delegator which has already go one entry for undelegation
|
|
|
|
func defaultMsgUndelegate() staking.Undelegate {
|
|
|
|
return staking.Undelegate{
|
|
|
|
DelegatorAddress: delegatorAddr,
|
|
|
|
ValidatorAddress: validatorAddr,
|
|
|
|
Amount: fiveKOnes,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultExpVWrapperUndelegateSameEpoch(t *testing.T) staking.ValidatorWrapper {
|
|
|
|
w := makeDefaultSnapVWrapperForUndelegate(t)
|
|
|
|
|
|
|
|
amt := w.Delegations[1].Undelegations[0].Amount
|
|
|
|
w.Delegations[1].Undelegations[0].Amount = new(big.Int).
|
|
|
|
Add(w.Delegations[1].Undelegations[0].Amount, amt)
|
|
|
|
w.Delegations[1].Amount = new(big.Int).Sub(w.Delegations[1].Amount, fiveKOnes)
|
|
|
|
|
|
|
|
return w
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultExpVWrapperUndelegateNextEpoch(t *testing.T) staking.ValidatorWrapper {
|
|
|
|
w := makeDefaultSnapVWrapperForUndelegate(t)
|
|
|
|
|
|
|
|
w.Delegations[1].Undelegations = append(w.Delegations[1].Undelegations,
|
|
|
|
staking.Undelegation{Amount: fiveKOnes, Epoch: big.NewInt(defaultNextEpoch)})
|
|
|
|
w.Delegations[1].Amount = new(big.Int).Sub(w.Delegations[1].Amount, fiveKOnes)
|
|
|
|
|
|
|
|
return w
|
|
|
|
}
|
|
|
|
|
|
|
|
// undelegate from self undelegation (new undelegates)
|
|
|
|
func defaultMsgSelfUndelegate() staking.Undelegate {
|
|
|
|
return staking.Undelegate{
|
|
|
|
DelegatorAddress: validatorAddr,
|
|
|
|
ValidatorAddress: validatorAddr,
|
|
|
|
Amount: fiveKOnes,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func defaultVWrapperSelfUndelegate(t *testing.T) staking.ValidatorWrapper {
|
|
|
|
w := makeDefaultSnapVWrapperForUndelegate(t)
|
|
|
|
|
|
|
|
w.Delegations[0].Undelegations = staking.Undelegations{
|
|
|
|
staking.Undelegation{Amount: fiveKOnes, Epoch: big.NewInt(defaultEpoch)},
|
|
|
|
}
|
|
|
|
w.Delegations[0].Amount = new(big.Int).Sub(w.Delegations[0].Amount, fiveKOnes)
|
|
|
|
|
|
|
|
return w
|
|
|
|
}
|
|
|
|
|
|
|
|
var (
|
|
|
|
reward00 = twentyKOnes
|
|
|
|
reward01 = tenKOnes
|
|
|
|
reward10 = thirtyKOnes
|
|
|
|
reward11 = twentyFiveKOnes
|
|
|
|
)
|
|
|
|
|
|
|
|
func TestVerifyAndCollectRewardsFromDelegation(t *testing.T) {
|
|
|
|
tests := []struct {
|
|
|
|
sdb vm.StateDB
|
|
|
|
ds []staking.DelegationIndex
|
|
|
|
|
|
|
|
expVWrappers []*staking.ValidatorWrapper
|
|
|
|
expTotalRewards *big.Int
|
|
|
|
expErr error
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
// 0: Positive test case
|
|
|
|
sdb: makeStateForReward(t),
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
|
|
|
|
expVWrappers: expVWrappersForReward(),
|
|
|
|
expTotalRewards: new(big.Int).Add(reward01, reward11),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 1: No rewards to collect
|
|
|
|
sdb: makeStateDBForStake(t),
|
|
|
|
ds: []staking.DelegationIndex{{ValidatorAddress: validatorAddr2, Index: 0}},
|
|
|
|
|
|
|
|
expErr: errNoRewardsToCollect,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 2: nil state db
|
|
|
|
sdb: nil,
|
|
|
|
ds: makeMsgCollectRewards(),
|
|
|
|
|
|
|
|
expErr: errStateDBIsMissing,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 3: ValidatorWrapper not in state
|
|
|
|
sdb: makeStateForReward(t),
|
|
|
|
ds: func() []staking.DelegationIndex {
|
|
|
|
msg := makeMsgCollectRewards()
|
|
|
|
msg[1].ValidatorAddress = makeTestAddr("addr not exist")
|
|
|
|
return msg
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("address not present in state"),
|
|
|
|
},
|
|
|
|
{
|
|
|
|
// 4: Wrong input message - index out of range
|
|
|
|
sdb: makeStateForReward(t),
|
|
|
|
ds: func() []staking.DelegationIndex {
|
|
|
|
dis := makeMsgCollectRewards()
|
|
|
|
dis[1].Index = 2
|
|
|
|
return dis
|
|
|
|
}(),
|
|
|
|
|
|
|
|
expErr: errors.New("index out of bound"),
|
|
|
|
},
|
|
|
|
}
|
|
|
|
for i, test := range tests {
|
|
|
|
ws, tReward, err := VerifyAndCollectRewardsFromDelegation(test.sdb, test.ds)
|
|
|
|
|
|
|
|
if assErr := assertError(err, test.expErr); assErr != nil {
|
|
|
|
t.Fatalf("Test %v: %v", i, err)
|
|
|
|
}
|
|
|
|
if err != nil || test.expErr != nil {
|
|
|
|
continue
|
|
|
|
}
|
|
|
|
|
|
|
|
if len(ws) != len(test.expVWrappers) {
|
|
|
|
t.Fatalf("vwrapper size unexpected: %v / %v", len(ws), len(test.expVWrappers))
|
|
|
|
}
|
|
|
|
for wi := range ws {
|
|
|
|
if err := staketest.CheckValidatorWrapperEqual(*ws[wi], *test.expVWrappers[wi]); err != nil {
|
|
|
|
t.Errorf("%v wrapper: %v", wi, err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if tReward.Cmp(test.expTotalRewards) != 0 {
|
|
|
|
t.Errorf("Test %v: total Rewards unexpected: %v / %v", i, tReward, test.expTotalRewards)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeMsgCollectRewards() []staking.DelegationIndex {
|
|
|
|
dis := []staking.DelegationIndex{
|
|
|
|
{
|
|
|
|
ValidatorAddress: validatorAddr,
|
|
|
|
Index: 1,
|
|
|
|
BlockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
}, {
|
|
|
|
ValidatorAddress: validatorAddr2,
|
|
|
|
Index: 1,
|
|
|
|
BlockNum: big.NewInt(defaultBlockNumber),
|
|
|
|
},
|
|
|
|
}
|
|
|
|
return dis
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeStateForReward(t *testing.T) *state.DB {
|
|
|
|
sdb := makeStateDBForStake(t)
|
|
|
|
|
|
|
|
rewards0 := []*big.Int{reward00, reward01}
|
|
|
|
if err := addStateRewardForAddr(sdb, validatorAddr, rewards0); err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
rewards1 := []*big.Int{reward10, reward11}
|
|
|
|
if err := addStateRewardForAddr(sdb, validatorAddr2, rewards1); err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
|
|
|
|
sdb.IntermediateRoot(true)
|
|
|
|
return sdb
|
|
|
|
}
|
|
|
|
|
|
|
|
func addStateRewardForAddr(sdb *state.DB, addr common.Address, rewards []*big.Int) error {
|
Resolve harmony-one/bounties#90: Add revert mechanism for UpdateValidatorWrapper (#3939)
* Add revert mechanism for UpdateValidatorWrapper
Closes harmony-one/bounties#90
(1) Use LRU for ValidatorWrapper objects in stateDB to plug a potential
memory leak
(2) Merge ValidatorWrapper and ValidatorWrapperCopy to let callers ask
for either a copy, or a pointer to the cached object. Additionally,
give callers the option to not deep copy delegations (which is a
heavy process). Copies need to be explicitly committed (and thus
can be reverted), while the pointers are committed when Finalise
is called.
(3) Add a UpdateValidatorWrapperWithRevert function, which is used by
staking txs `Delegate`, `Undelegate`, and `CollectRewards`. Other
2 types of staking txs and `db.Finalize` continue to use
UpdateValidateWrapper without revert, again, to save memoery
(4) Add unit tests which check
a) Revert goes through
b) Wrapper is as expected after revert
c) State is as expected after revert
* Change back to dictionary for stateValidators
Since the memory / CPU usage saved is not significantly different when
using an LRU + map structure, go back to the original dictionary
structure to keep code easy to read and have limited modifications.
* Add tests for validator wrapper reverts
As requested by @rlan35, add tests beyond just adding and reverting a
delegation. The tests are successive in the sense that we do multiple
modifications to the wrapper, save a snapshot before each modification
and revert to each of them to confirm everything works well. This change
improves test coverage of statedb.go to 66.7% from 64.8% and that of
core/state to 71.9% from 70.8%, and covers all the code that has been
modified by this PR in statedb.go.
For clarity, the modifications to the wrapper include (1) creation of
wrapper in state, (2) adding a delegation to the wrapper, (3)
increasing the blocks signed, and (4) a change in the validator Name and
the BlockReward. Two additional tests have been added to cover the
`panic` and the `GetCode` cases.
3 years ago
|
|
|
w, err := sdb.ValidatorWrapper(addr, false, true)
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
w.Delegations = append(w.Delegations,
|
|
|
|
staking.NewDelegation(delegatorAddr, new(big.Int).Set(twentyKOnes)),
|
|
|
|
)
|
|
|
|
w.Delegations[1].Undelegations = staking.Undelegations{}
|
|
|
|
w.Delegations[0].Reward = new(big.Int).Set(rewards[0])
|
|
|
|
w.Delegations[1].Reward = new(big.Int).Set(rewards[1])
|
|
|
|
|
|
|
|
return sdb.UpdateValidatorWrapper(addr, w)
|
|
|
|
}
|
|
|
|
|
|
|
|
func expVWrappersForReward() []*staking.ValidatorWrapper {
|
|
|
|
w1 := makeVWrapperByIndex(validatorIndex)
|
|
|
|
w1.Delegations = append(w1.Delegations,
|
|
|
|
staking.NewDelegation(delegatorAddr, new(big.Int).Set(twentyKOnes)),
|
|
|
|
)
|
|
|
|
w1.Delegations[1].Undelegations = staking.Undelegations{}
|
|
|
|
w1.Delegations[0].Reward = new(big.Int).Set(reward00)
|
|
|
|
w1.Delegations[1].Reward = new(big.Int).SetUint64(0)
|
|
|
|
|
|
|
|
w2 := makeVWrapperByIndex(validator2Index)
|
|
|
|
w2.Delegations = append(w2.Delegations,
|
|
|
|
staking.NewDelegation(delegatorAddr, new(big.Int).Set(twentyKOnes)),
|
|
|
|
)
|
|
|
|
w2.Delegations[1].Undelegations = staking.Undelegations{}
|
|
|
|
w2.Delegations[0].Reward = new(big.Int).Set(reward10)
|
|
|
|
w2.Delegations[1].Reward = new(big.Int).SetUint64(0)
|
|
|
|
return []*staking.ValidatorWrapper{&w1, &w2}
|
|
|
|
}
|
|
|
|
|
|
|
|
// makeFakeChainContextForStake makes the default fakeChainContext for staking test
|
|
|
|
func makeFakeChainContextForStake() *fakeChainContext {
|
|
|
|
ws := makeVWrappersForStake(defNumWrappersInState, defNumPubPerAddr)
|
|
|
|
return makeFakeChainContext(ws)
|
|
|
|
}
|
|
|
|
|
|
|
|
// makeStateDBForStake make the default state db for staking test
|
|
|
|
func makeStateDBForStake(t *testing.T) *state.DB {
|
|
|
|
sdb, err := newTestStateDB()
|
|
|
|
if err != nil {
|
|
|
|
t.Fatal(err)
|
|
|
|
}
|
|
|
|
ws := makeVWrappersForStake(defNumWrappersInState, defNumPubPerAddr)
|
|
|
|
if err := updateStateValidators(sdb, ws); err != nil {
|
|
|
|
t.Fatalf("make default state: %v", err)
|
|
|
|
}
|
|
|
|
sdb.AddBalance(createValidatorAddr, hundredKOnes)
|
|
|
|
sdb.AddBalance(delegatorAddr, hundredKOnes)
|
|
|
|
|
|
|
|
sdb.IntermediateRoot(true)
|
|
|
|
|
|
|
|
return sdb
|
|
|
|
}
|
|
|
|
|
|
|
|
func updateStateValidators(sdb *state.DB, ws []*staking.ValidatorWrapper) error {
|
|
|
|
for i, w := range ws {
|
|
|
|
sdb.SetValidatorFlag(w.Address)
|
|
|
|
sdb.AddBalance(w.Address, hundredKOnes)
|
|
|
|
sdb.SetValidatorFirstElectionEpoch(w.Address, big.NewInt(10))
|
|
|
|
if err := sdb.UpdateValidatorWrapper(w.Address, w); err != nil {
|
|
|
|
return fmt.Errorf("update %v vw error: %v", i, err)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeVWrapperByIndex(index int) staking.ValidatorWrapper {
|
|
|
|
pubGetter := newBLSPubGetter(blsKeys[index*defNumPubPerAddr:])
|
|
|
|
|
|
|
|
return makeStateVWrapperFromGetter(index, defNumPubPerAddr, pubGetter)
|
|
|
|
}
|
|
|
|
|
|
|
|
func newTestStateDB() (*state.DB, error) {
|
Release Candidate 2023.2.0 ( dev -> main ) (#4399)
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* activate epoch
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review "--port flag".
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* Commented golangci.
* staged stream sync v1.0
* fix protocol tests
* fix spell
* remove unused struct
* fix rosetta test
* add comments and refactor verify sig
* add comments, remove extra function
* add comment
* refactor errors, rename metrics
* refactor p2p host creation
* fix initsync and host creation
* fix short range hash chain
* fix beacon node detection for p2p protocol
* refactor stream peer cooldown and fix protocol beacon node field
* refactor p2p host and routing
* fix p2p discovery test issue
* add MaxAdvertiseWaitTime to handle advertisements interval and address stream connection issue
* terminal print the peer id and proto id
* fix boot complete message when node is shut down
* add new config option ( ForceReachabilityPublic ) to fix local-net consensus issue
* fix self query issue
* fix test NewDNSSyncingPeerProvider
* [testnet] disable leader rotation
* fix discovery issue for legacy sync
* add watermark low/high options for p2p connection manager
* add test for new conn manager flags
* fix dedent
* add comment to inform about p2p connection manager options
* fix max height issue
* add a separate log for get max height error
* fix log
* feat: triesInMemory flag
* fix: panic if TriesInMemory is 1 to 2
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed timers locks.
* Removed fbft locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional logs.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* Fix formatting.
* Rebased onto dev.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Fix review comment.
* Go mod tidy.
* Set to EpochTBD.
* Fix tests.
* [core] fix state handling of self destruct
If a contract self destructs to self and then receives funds within the
same transaction, it is possible for its stale state to be saved. This
change removes that possibility by checking for deleted state objects
before returning them.
* Fixed race error.
* rpc: add configurable http and `eth_call` timeout
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fixed code review.
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* staged stream sync v1.0
* refactor errors, rename metrics
* fix p2p discovery test issue
* add watermark low/high options for p2p connection manager
* fix dedent
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Go mod tidy.
* remove default timeouts
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Fixes.
* Update singleton.go
* evm: don't return extcode for validators
Due to technical debt, validator information is stored in the code field
of the address. The code field can be accessed in Solidity for an
arbitrary address using `extcodesize`, `extcodehash`, and `extcodecopy`
or helper commands (such as `address.code.Length`). The presence of this
field is used by contract developers to (erroneously) deny smart
contract access to other smart contracts (and therefore, validators).
This PR fixes that oversight by returning the same values as other EOAs
for known validator addresses. Obviously, it needs a hard fork that will
be scheduled separately.
* Fix context passing.
* Clean up code.
* Removed engine dependency.
* Fix possible panic.
* Clean up code.
* Network type.
* Fix tests.
* Revert "Removed engine dependency." (#4392)
* Revert "Fix tests."
This reverts commit 597ba2d6f1ed54ff599b9d9b8940c7285ab1277a.
* Revert "Network type."
This reverts commit 5e1878aedca3989dc0f34161dae1833e43ca6a76.
* Revert "Clean up code."
This reverts commit 15885f4c9b9263746827172b4f4f56d0926d18e2.
* Revert "Fix possible panic."
This reverts commit 1a70d5eb66cdbf8a23791806b71a323eed320085.
* Revert "Removed engine dependency."
This reverts commit 8c2ff803f709f944cfc8b1278f35cf5b2cacf859.
* gitignore the cache folder (#4389)
* stable localnet with external validator (#4388)
* stable localnet with external validator
* ignore deploy config file comments
* reduce node launched in localnet
* update makefile
* localnet configuration - add more fn
* fix validator information command typo
* Configurable tx pool. (#4240)
* AccountQueue & GlobalQueue.
* Lifetime duration.
* [pool] make flags configurable
* [pool] use 4096 as default `GlobalSlots`
* [rosetta] update default values of tx pool
* [test] update value to default
* PriceLimit and PriceBump.
* Fix tests.
* Fix price limit & bump.
* Updated, fixed migrate version and tests.
* Rebased.
* Fix go toml version.
---------
Co-authored-by: Konstantin <k.potapov@softpro.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* Upgrade rawdb and statedb codes to add the latest functionalities of ethdb (#4374)
* added bloom filter
* upgrade rawdb and statedb
* change var name and remove extra comments
* return back fake storage in case if we need it for test later
* add the previous change back
* remove some extra entries from go.mod
* fix WritePreimages to use batch
* mark unused functions which are ported over from eth
---------
Co-authored-by: Casey Gardiner <117784577+ONECasey@users.noreply.github.com>
* update all ethereum rawdb pointers to use harmony rawdb (#4395)
* Fix reduce node dependencies. (#4379)
* Fix.
* Fix.
* Fix pool init.
* Clean up.
* add prefix for contract code (#4397)
* Rotate external validators for non-beacon shards. (#4373)
* Rotate only non beacon shards.
* Rotate all shards, but only hmy validators for beacon.
* Fix type.
* Revert "Fix type."
This reverts commit 0a8b506c763d9f8609abff7395ba32b18e43b149.
* Revert "Rotate all shards, but only hmy validators for beacon."
This reverts commit 70b09e2de81aa2cbffae3ccdfd4e334e7d938759.
* Fixed failed test.
* Revert "Revert "Rotate all shards, but only hmy validators for beacon.""
This reverts commit 66cfaa9817488be60ed5b5cfee1fe833ede237c8.
* Frequency by slots count.
* Fix config.
* First validator produce rest blocks.
* Updated.
* Add lock.
* Add prefix for validator wrapper (#4402)
* add separate prefix for validator wrapper
* update comments
* make read/write backward compatible
* add validator codes to stats
* goimports
* goimports accessor_state
* add snapshot feature to state db (#4406)
* Typed cache & Node cleanup. (#4409)
* Channels usage through methods.
* Fix retry count. Removed proposedBlock.
* keysToAddrs rewritten to lrucache.
* core, internal/configs: HIP28-v2 fee collection (#4410)
* core, internal/configs: HIP28-v2 fee collection
Based on the Snapshot vote that has passed, collect 50% of the fee to a
community maintained account and the remainder to an account used to pay
for infrastructure costs. Note that these accounts need to be decided
and set in the code at this moment, and the feature needs to be
activated by setting the `FeeCollectEpoch` of the `ChainConfig` object.
The implementation for devnet is a bit different than compared to
others because the feature was activated on devnet with 100% collection
to an account. I have handled this case separately in `devnet.go`.
* test: add test for StateTransition.ApplyMessage
The objective of this unit test is to check that the fees of a
transaction are appropriately credited to the fee collectors that are
set. This means, for a transaction of 21,000 gas limit and 100 gwei gas
price, two equal fee collectors get 10,500 * 100 gwei each. In addition,
to be clear that the refund mechanism (in case a transaction with extra
gas comes in) works, the tested transaction has a 50,000 gas limit of
which only 21,000 gas limit is actually consumed.
* sharding/config: clarify local fee collector pk
* sharding/config: set testnet fee collector same as devnet
* test: add test for truncated fee distribution
* sharding/config: set fee collector addresses
* test: hardcode the expected fee collected
* goimports
* params/config: set testnet fee collect epoch
Schedule testnet hard fork epoch to be 1296, which begins around the
time 2023-04-28 07:14:20+00:00
* params/config: schedule devnee fee collection
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* Minor: removed time.Sleep from tests. (#4412)
* Provide current time as argument.
* Fix test.
* Fix naming.
* Mainnet Release Candidate 2023.1.2 (#4376) (#4414)
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Bump github.com/aws/aws-sdk-go from 1.33.0 to 1.34.0
Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.33.0 to 1.34.0.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/v1.34.0/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.33.0...v1.34.0)
---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
dependency-type: direct:production
...
* Bump github.com/ipld/go-ipld-prime from 0.9.0 to 0.19.0
Bumps [github.com/ipld/go-ipld-prime](https://github.com/ipld/go-ipld-prime) from 0.9.0 to 0.19.0.
- [Release notes](https://github.com/ipld/go-ipld-prime/releases)
- [Changelog](https://github.com/ipld/go-ipld-prime/blob/master/CHANGELOG.md)
- [Commits](https://github.com/ipld/go-ipld-prime/compare/v0.9.0...v0.19.0)
---
updated-dependencies:
- dependency-name: github.com/ipld/go-ipld-prime
dependency-type: indirect
...
* Bump golang.org/x/net from 0.3.0 to 0.7.0
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.3.0 to 0.7.0.
- [Release notes](https://github.com/golang/net/releases)
- [Commits](https://github.com/golang/net/compare/v0.3.0...v0.7.0)
---
updated-dependencies:
- dependency-name: golang.org/x/net
dependency-type: indirect
...
* Small fixes.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* activate epoch
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review "--port flag".
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* Commented golangci.
* staged stream sync v1.0
* fix protocol tests
* fix spell
* remove unused struct
* fix rosetta test
* add comments and refactor verify sig
* add comments, remove extra function
* add comment
* refactor errors, rename metrics
* refactor p2p host creation
* fix initsync and host creation
* fix short range hash chain
* fix beacon node detection for p2p protocol
* refactor stream peer cooldown and fix protocol beacon node field
* refactor p2p host and routing
* fix p2p discovery test issue
* add MaxAdvertiseWaitTime to handle advertisements interval and address stream connection issue
* terminal print the peer id and proto id
* fix boot complete message when node is shut down
* add new config option ( ForceReachabilityPublic ) to fix local-net consensus issue
* fix self query issue
* fix test NewDNSSyncingPeerProvider
* [testnet] disable leader rotation
* fix discovery issue for legacy sync
* add watermark low/high options for p2p connection manager
* add test for new conn manager flags
* fix dedent
* add comment to inform about p2p connection manager options
* fix max height issue
* add a separate log for get max height error
* fix log
* feat: triesInMemory flag
* fix: panic if TriesInMemory is 1 to 2
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed timers locks.
* Removed fbft locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional logs.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* Fix formatting.
* Rebased onto dev.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Fix review comment.
* Go mod tidy.
* Set to EpochTBD.
* Fix tests.
* [core] fix state handling of self destruct
If a contract self destructs to self and then receives funds within the
same transaction, it is possible for its stale state to be saved. This
change removes that possibility by checking for deleted state objects
before returning them.
* Fixed race error.
* rpc: add configurable http and `eth_call` timeout
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fixed code review.
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* staged stream sync v1.0
* refactor errors, rename metrics
* fix p2p discovery test issue
* add watermark low/high options for p2p connection manager
* fix dedent
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Go mod tidy.
* remove default timeouts
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Fixes.
* Update singleton.go
* evm: don't return extcode for validators
Due to technical debt, validator information is stored in the code field
of the address. The code field can be accessed in Solidity for an
arbitrary address using `extcodesize`, `extcodehash`, and `extcodecopy`
or helper commands (such as `address.code.Length`). The presence of this
field is used by contract developers to (erroneously) deny smart
contract access to other smart contracts (and therefore, validators).
This PR fixes that oversight by returning the same values as other EOAs
for known validator addresses. Obviously, it needs a hard fork that will
be scheduled separately.
* Fix context passing.
* Clean up code.
* Removed engine dependency.
* Fix possible panic.
* Clean up code.
* Network type.
* Fix tests.
* Revert "Removed engine dependency." (#4392)
* Revert "Fix tests."
This reverts commit 597ba2d6f1ed54ff599b9d9b8940c7285ab1277a.
* Revert "Network type."
This reverts commit 5e1878aedca3989dc0f34161dae1833e43ca6a76.
* Revert "Clean up code."
This reverts commit 15885f4c9b9263746827172b4f4f56d0926d18e2.
* Revert "Fix possible panic."
This reverts commit 1a70d5eb66cdbf8a23791806b71a323eed320085.
* Revert "Removed engine dependency."
This reverts commit 8c2ff803f709f944cfc8b1278f35cf5b2cacf859.
* gitignore the cache folder (#4389)
* stable localnet with external validator (#4388)
* stable localnet with external validator
* ignore deploy config file comments
* reduce node launched in localnet
* update makefile
* localnet configuration - add more fn
* fix validator information command typo
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Casey Gardiner <117784577+ONECasey@users.noreply.github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
* chore: merge `main` into `dev` (#4415)
* Mainnet Release Candidate 2023.1.2 (#4376)
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Bump github.com/aws/aws-sdk-go from 1.33.0 to 1.34.0
Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.33.0 to 1.34.0.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/v1.34.0/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.33.0...v1.34.0)
---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump github.com/ipld/go-ipld-prime from 0.9.0 to 0.19.0
Bumps [github.com/ipld/go-ipld-prime](https://github.com/ipld/go-ipld-prime) from 0.9.0 to 0.19.0.
- [Release notes](https://github.com/ipld/go-ipld-prime/releases)
- [Changelog](https://github.com/ipld/go-ipld-prime/blob/master/CHANGELOG.md)
- [Commits](https://github.com/ipld/go-ipld-prime/compare/v0.9.0...v0.19.0)
---
updated-dependencies:
- dependency-name: github.com/ipld/go-ipld-prime
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump golang.org/x/net from 0.3.0 to 0.7.0
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.3.0 to 0.7.0.
- [Release notes](https://github.com/golang/net/releases)
- [Commits](https://github.com/golang/net/compare/v0.3.0...v0.7.0)
---
updated-dependencies:
- dependency-name: golang.org/x/net
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
* Small fixes.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* activate epoch
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review "--port flag".
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* Commented golangci.
* staged stream sync v1.0
* fix protocol tests
* fix spell
* remove unused struct
* fix rosetta test
* add comments and refactor verify sig
* add comments, remove extra function
* add comment
* refactor errors, rename metrics
* refactor p2p host creation
* fix initsync and host creation
* fix short range hash chain
* fix beacon node detection for p2p protocol
* refactor stream peer cooldown and fix protocol beacon node field
* refactor p2p host and routing
* fix p2p discovery test issue
* add MaxAdvertiseWaitTime to handle advertisements interval and address stream connection issue
* terminal print the peer id and proto id
* fix boot complete message when node is shut down
* add new config option ( ForceReachabilityPublic ) to fix local-net consensus issue
* fix self query issue
* fix test NewDNSSyncingPeerProvider
* [testnet] disable leader rotation
* fix discovery issue for legacy sync
* add watermark low/high options for p2p connection manager
* add test for new conn manager flags
* fix dedent
* add comment to inform about p2p connection manager options
* fix max height issue
* add a separate log for get max height error
* fix log
* feat: triesInMemory flag
* fix: panic if TriesInMemory is 1 to 2
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed timers locks.
* Removed fbft locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional logs.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* Fix formatting.
* Rebased onto dev.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Fix review comment.
* Go mod tidy.
* Set to EpochTBD.
* Fix tests.
* [core] fix state handling of self destruct
If a contract self destructs to self and then receives funds within the
same transaction, it is possible for its stale state to be saved. This
change removes that possibility by checking for deleted state objects
before returning them.
* Fixed race error.
* rpc: add configurable http and `eth_call` timeout
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fixed code review.
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* staged stream sync v1.0
* refactor errors, rename metrics
* fix p2p discovery test issue
* add watermark low/high options for p2p connection manager
* fix dedent
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Go mod tidy.
* remove default timeouts
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Fixes.
* Update singleton.go
* evm: don't return extcode for validators
Due to technical debt, validator information is stored in the code field
of the address. The code field can be accessed in Solidity for an
arbitrary address using `extcodesize`, `extcodehash`, and `extcodecopy`
or helper commands (such as `address.code.Length`). The presence of this
field is used by contract developers to (erroneously) deny smart
contract access to other smart contracts (and therefore, validators).
This PR fixes that oversight by returning the same values as other EOAs
for known validator addresses. Obviously, it needs a hard fork that will
be scheduled separately.
* Fix context passing.
* Clean up code.
* Removed engine dependency.
* Fix possible panic.
* Clean up code.
* Network type.
* Fix tests.
* Revert "Removed engine dependency." (#4392)
* Revert "Fix tests."
This reverts commit 597ba2d6f1ed54ff599b9d9b8940c7285ab1277a.
* Revert "Network type."
This reverts commit 5e1878aedca3989dc0f34161dae1833e43ca6a76.
* Revert "Clean up code."
This reverts commit 15885f4c9b9263746827172b4f4f56d0926d18e2.
* Revert "Fix possible panic."
This reverts commit 1a70d5eb66cdbf8a23791806b71a323eed320085.
* Revert "Removed engine dependency."
This reverts commit 8c2ff803f709f944cfc8b1278f35cf5b2cacf859.
* gitignore the cache folder (#4389)
* stable localnet with external validator (#4388)
* stable localnet with external validator
* ignore deploy config file comments
* reduce node launched in localnet
* update makefile
* localnet configuration - add more fn
* fix validator information command typo
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
* build: update pinned curl version (#4394)
Per the Alpine Linux package repositories, the version for cURL included
with v3.16 has changed to revision 6
* consensus: replace type assert with test (#4398)
If `consensus.finalityCounter` does not have anything stored (for
example in Syncing mode), the `Load()` returns an interface that cannot
be automatically asserted to an `int64`. This results in the node
crashing. This commit fixes that.
* Turn pprof default on with local saved files (#3894)
* Turn pprof default on with local saved files
* [pprof] change interval from 600s to 3600s
* Revert "Turn pprof default on with local saved files (#3894)" (#4400)
This reverts commit 78d26d7910a58ded3bfe689b3de07ea28d95d7d5.
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Casey Gardiner <117784577+ONECasey@users.noreply.github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
Co-authored-by: Jacky Wang <jackyw.se@gmail.com>
* internal/params: set validator code fix hard forks (#4413)
* internal/params: schedule hard forks
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* internal/params: set localnet fee collect epoch 2
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
---------
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* internal/params: schedule HIP28v2 + val code fix (#4416)
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
* Dev fix conflicts. (#4417)
* Mainnet Release Candidate 2023.1.2 (#4376)
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Bump github.com/aws/aws-sdk-go from 1.33.0 to 1.34.0
Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.33.0 to 1.34.0.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/v1.34.0/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.33.0...v1.34.0)
---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump github.com/ipld/go-ipld-prime from 0.9.0 to 0.19.0
Bumps [github.com/ipld/go-ipld-prime](https://github.com/ipld/go-ipld-prime) from 0.9.0 to 0.19.0.
- [Release notes](https://github.com/ipld/go-ipld-prime/releases)
- [Changelog](https://github.com/ipld/go-ipld-prime/blob/master/CHANGELOG.md)
- [Commits](https://github.com/ipld/go-ipld-prime/compare/v0.9.0...v0.19.0)
---
updated-dependencies:
- dependency-name: github.com/ipld/go-ipld-prime
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
* Bump golang.org/x/net from 0.3.0 to 0.7.0
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.3.0 to 0.7.0.
- [Release notes](https://github.com/golang/net/releases)
- [Commits](https://github.com/golang/net/compare/v0.3.0...v0.7.0)
---
updated-dependencies:
- dependency-name: golang.org/x/net
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
* Small fixes.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* activate epoch
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review "--port flag".
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* Commented golangci.
* staged stream sync v1.0
* fix protocol tests
* fix spell
* remove unused struct
* fix rosetta test
* add comments and refactor verify sig
* add comments, remove extra function
* add comment
* refactor errors, rename metrics
* refactor p2p host creation
* fix initsync and host creation
* fix short range hash chain
* fix beacon node detection for p2p protocol
* refactor stream peer cooldown and fix protocol beacon node field
* refactor p2p host and routing
* fix p2p discovery test issue
* add MaxAdvertiseWaitTime to handle advertisements interval and address stream connection issue
* terminal print the peer id and proto id
* fix boot complete message when node is shut down
* add new config option ( ForceReachabilityPublic ) to fix local-net consensus issue
* fix self query issue
* fix test NewDNSSyncingPeerProvider
* [testnet] disable leader rotation
* fix discovery issue for legacy sync
* add watermark low/high options for p2p connection manager
* add test for new conn manager flags
* fix dedent
* add comment to inform about p2p connection manager options
* fix max height issue
* add a separate log for get max height error
* fix log
* feat: triesInMemory flag
* fix: panic if TriesInMemory is 1 to 2
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed timers locks.
* Removed fbft locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional logs.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* Fix formatting.
* Rebased onto dev.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* comment activation
* 295 epoch
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Fix review comment.
* Go mod tidy.
* Set to EpochTBD.
* Fix tests.
* [core] fix state handling of self destruct
If a contract self destructs to self and then receives funds within the
same transaction, it is possible for its stale state to be saved. This
change removes that possibility by checking for deleted state objects
before returning them.
* Fixed race error.
* rpc: add configurable http and `eth_call` timeout
* remove default timeouts
* store the evm call timeout in rosetta object
* [cmd] actually apply ToRPCServerConfig
* Removed unused method.
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* in progress.
* in progress.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fixed code review.
* Fix review comments.
* Returned locks in rotateLeader.
* Rebased onto dev.
* staged stream sync v1.0
* refactor errors, rename metrics
* fix p2p discovery test issue
* add watermark low/high options for p2p connection manager
* fix dedent
* in progress.
* consensus check is forked
* fix
* Cleanup and fix update pub keys.
* fix
fix
fix
fix
fix
* activate epoch
* EpochTBD for leader rotation epoch.
* 295 epoch
* Decider no longer requires public keys as a dependency. (#4289)
* Consensus doesn't require anymore `Node` as a circular dependency.
* Proper blockchain initialization.
* Rwlock consensus.
* Removed channels.
* Removed view change locks.
* Removed multiSigMutex locks.
* Removed leader locks.
* Removed additional locks and isViewChange.
* Added locks detected by race.
* Added locks detected by race.
* Locks for start.
* Removed additional locks.
* Removed additional locks.
* Make func private.
* Make VerifyBlock private.
* Make IsLeader private.
* Make ParseFBFTMessage private.
* Fix remove locks.
* Added additional locks.
* Added additional locks.
* Added readSignatureBitmapPayload locks.
* Added HandleMessageUpdate locks.
* Added LastMile locks.
* Locks for IsValidatorInCommittee.
* Fixed locks.
* Fixed tests.
* Fixed lock.
* Rebased over leader rotation.
* in progress.
* consensus check is forked
* update master
* fix leader
* check leader for N blocks
* fix
* fix
* Cleanup and fix update pub keys.
* Rotate leader.
* fix
fix
fix
fix
fix
* Cleaned.
* Cache for `GetLeaderPubKeyFromCoinbase`, removed `NthNextHmyExt`.
* Fix failed tests.
* Fixed code review.
* Fix review comments.
* Merged leader rotation.
* Rebased on dev.
* Rebased on dev.
* Fix usage of private methods.
* Fix usage of private methods.
* Fix usage of private methods.
* Removed deadcode, LockedFBFTPhase.
* Fix review comment.
* Go mod tidy.
* remove default timeouts
* Rotate external leaders on non-beacon chains.
* Fix nil panic.
* Fixes.
* Update singleton.go
* evm: don't return extcode for validators
Due to technical debt, validator information is stored in the code field
of the address. The code field can be accessed in Solidity for an
arbitrary address using `extcodesize`, `extcodehash`, and `extcodecopy`
or helper commands (such as `address.code.Length`). The presence of this
field is used by contract developers to (erroneously) deny smart
contract access to other smart contracts (and therefore, validators).
This PR fixes that oversight by returning the same values as other EOAs
for known validator addresses. Obviously, it needs a hard fork that will
be scheduled separately.
* Fix context passing.
* Clean up code.
* Removed engine dependency.
* Fix possible panic.
* Clean up code.
* Network type.
* Fix tests.
* Revert "Removed engine dependency." (#4392)
* Revert "Fix tests."
This reverts commit 597ba2d6f1ed54ff599b9d9b8940c7285ab1277a.
* Revert "Network type."
This reverts commit 5e1878aedca3989dc0f34161dae1833e43ca6a76.
* Revert "Clean up code."
This reverts commit 15885f4c9b9263746827172b4f4f56d0926d18e2.
* Revert "Fix possible panic."
This reverts commit 1a70d5eb66cdbf8a23791806b71a323eed320085.
* Revert "Removed engine dependency."
This reverts commit 8c2ff803f709f944cfc8b1278f35cf5b2cacf859.
* gitignore the cache folder (#4389)
* stable localnet with external validator (#4388)
* stable localnet with external validator
* ignore deploy config file comments
* reduce node launched in localnet
* update makefile
* localnet configuration - add more fn
* fix validator information command typo
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
* build: update pinned curl version (#4394)
Per the Alpine Linux package repositories, the version for cURL included
with v3.16 has changed to revision 6
* consensus: replace type assert with test (#4398)
If `consensus.finalityCounter` does not have anything stored (for
example in Syncing mode), the `Load()` returns an interface that cannot
be automatically asserted to an `int64`. This results in the node
crashing. This commit fixes that.
* Turn pprof default on with local saved files (#3894)
* Turn pprof default on with local saved files
* [pprof] change interval from 600s to 3600s
* Revert "Turn pprof default on with local saved files (#3894)" (#4400)
This reverts commit 78d26d7910a58ded3bfe689b3de07ea28d95d7d5.
* go mod tidy.
* Increased wait time.
---------
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Casey Gardiner <117784577+ONECasey@users.noreply.github.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
Co-authored-by: Jacky Wang <jackyw.se@gmail.com>
---------
Signed-off-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: frozen <355847+Frozen@users.noreply.github.com>
Co-authored-by: “GheisMohammadi” <“Gheis.Mohammadi@gmail.com”>
Co-authored-by: “GheisMohammadi” <36589218+GheisMohammadi@users.noreply.github.com>
Co-authored-by: MaxMustermann2 <82761650+MaxMustermann2@users.noreply.github.com>
Co-authored-by: Sun Hyuk Ahn <sunhyukahn@Suns-MacBook-Pro.local>
Co-authored-by: Soph <35721420+sophoah@users.noreply.github.com>
Co-authored-by: Konstantin <k.potapov@softpro.com>
Co-authored-by: Gheis Mohammadi <Gheis.Mohammadi@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jacky Wang <jackyw.se@gmail.com>
2 years ago
|
|
|
return state.New(common.Hash{}, state.NewDatabase(rawdb.NewMemoryDatabase()), nil)
|
|
|
|
}
|
|
|
|
|
|
|
|
// makeVWrappersForStake makes the default staking.ValidatorWrappers for
|
|
|
|
// initialization of default state db for staking test
|
|
|
|
func makeVWrappersForStake(num, numPubsPerVal int) []*staking.ValidatorWrapper {
|
|
|
|
ws := make([]*staking.ValidatorWrapper, 0, num)
|
|
|
|
pubGetter := newBLSPubGetter(blsKeys)
|
|
|
|
for i := 0; i != num; i++ {
|
|
|
|
w := makeStateVWrapperFromGetter(i, numPubsPerVal, pubGetter)
|
|
|
|
ws = append(ws, &w)
|
|
|
|
}
|
|
|
|
return ws
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeStateVWrapperFromGetter(index int, numPubs int, pubGetter *BLSPubGetter) staking.ValidatorWrapper {
|
|
|
|
addr := makeTestAddr(index)
|
|
|
|
pubs := make([]bls.SerializedPublicKey, 0, numPubs)
|
|
|
|
for i := 0; i != numPubs; i++ {
|
|
|
|
pubs = append(pubs, pubGetter.getPub())
|
|
|
|
}
|
|
|
|
w := staketest.GetDefaultValidatorWrapperWithAddr(addr, pubs)
|
|
|
|
w.Identity = makeIdentityStr(index)
|
|
|
|
w.UpdateHeight = big.NewInt(defaultSnapBlockNumber)
|
|
|
|
return w
|
|
|
|
}
|
|
|
|
|
|
|
|
type BLSPubGetter struct {
|
|
|
|
keys []blsPubSigPair
|
|
|
|
index int
|
|
|
|
}
|
|
|
|
|
|
|
|
func newBLSPubGetter(keys []blsPubSigPair) *BLSPubGetter {
|
|
|
|
return &BLSPubGetter{
|
|
|
|
keys: keys,
|
|
|
|
index: 0,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func (g *BLSPubGetter) getPub() bls.SerializedPublicKey {
|
|
|
|
key := g.keys[g.index]
|
|
|
|
g.index++
|
|
|
|
return key.pub
|
|
|
|
}
|
|
|
|
|
|
|
|
// fakeChainContext is the fake structure of ChainContext for testing
|
|
|
|
type fakeChainContext struct {
|
|
|
|
vWrappers map[common.Address]*staking.ValidatorWrapper
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeFakeChainContext(ws []*staking.ValidatorWrapper) *fakeChainContext {
|
|
|
|
m := make(map[common.Address]*staking.ValidatorWrapper)
|
|
|
|
for _, w := range ws {
|
|
|
|
wCpy := staketest.CopyValidatorWrapper(*w)
|
|
|
|
m[w.Address] = &wCpy
|
|
|
|
}
|
|
|
|
return &fakeChainContext{
|
|
|
|
vWrappers: m,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeChainContext) ReadValidatorList() ([]common.Address, error) {
|
|
|
|
vs := make([]common.Address, 0, len(chain.vWrappers))
|
|
|
|
for addr := range chain.vWrappers {
|
|
|
|
vs = append(vs, addr)
|
|
|
|
}
|
|
|
|
return vs, nil
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeChainContext) Engine() consensus_engine.Engine {
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeChainContext) Config() *params.ChainConfig {
|
|
|
|
config := ¶ms.ChainConfig{}
|
|
|
|
config.MinCommissionRateEpoch = big.NewInt(0)
|
|
|
|
config.MinCommissionPromoPeriod = big.NewInt(10)
|
|
|
|
return config
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeChainContext) GetHeader(common.Hash, uint64) *block.Header {
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeChainContext) ReadDelegationsByDelegator(common.Address) (staking.DelegationIndexes, error) {
|
|
|
|
return nil, nil
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeChainContext) ReadDelegationsByDelegatorAt(delegator common.Address, blockNum *big.Int) (staking.DelegationIndexes, error) {
|
|
|
|
return nil, nil
|
|
|
|
}
|
|
|
|
|
Resolve harmony-one/bounties#77: Staking precompiles (#3906)
* Resolve harmony-one/bounties#77: Staking precompiles
Create write capable precompiles that can perform staking transactions
Add hard fork logic (EpochTBD) for these precompiles
Tests for new code with at least 80% unit test coverage
Staking library + tests in MaxMustermann2/harmony-staking-precompiles
* Fix small typo in comment
* Run goimports on files to fix Travis
* Do not activate staking precompile on shard 0
* Cascade readOnly to WriteCapableContract
* No overlap in readOnly + writeCapable precompiles
* Use function selector instead of directive
From Solidity, use abi.encodeWithSelector and match it against the
exact ABI of the functions. This allows us to remove the need for
a directive (32) being encoded, and thus saves 28 bytes of data.
* Do not allow contracts to become validators
As discussed with Jacky on #3906
* Merge harmony-one/harmony/main properly this time
* Run goimports
* Update gas calculation for staking precompile
Please see comment in core/vm/contracts_write.go RequiredGas
* Do not allow contract to become validator (2/2)
* Cache StakeMsgs from precompiled transactions
Add the StakeMsgs to ProcessorResult and cascade them in insertChain
* Remove ContractCode fields from validators
Since smart contracts can no longer beecome validators,
this field is superfluous. Remove it from the Wrapper
structure, and do not assign it a value when creating
a validator. Build and goimports checked
* Update comments in response to feedback
(1) Comments to start with function names
(2) Comments for public variables
(3) Comment to match function name RunPrecompiledContract
(4) Clarify that CreateValidatorFunc + EditValidatorFunc are still used
* Fix Travis build by reverting rosetta change
* Add revert capability to 3 staking tx types
- Delegate
- Undelegate
- CollectRewards
* Fix build: Update evm_test for ValidatorWrapper
* Merge main into harmony-staking-precompiles
* Add gas for precompile calls and allow EOA usage
- Each time the precompile is called, charge the base gas fee plus data
cost (if data can be parsed successfully). A gas fee is added to
prevent benevolent contract deployers from subsidizing the staking
transactions for EOAs through repeated assembly `delegatecall`.
- Allow EOAs to use the staking precompile directly. Some changes to
the Solidity library are associated with this change.
- Remove bytes from parsing address, since the ABI unpacks it into an
address format correctly.
- Add or update tests. Test coverage report to be attached to the PR
shortly.
* Run goimports
* Check read only and write capable for overlap
* Handle precompile stakeMsgs for block proposer
The staking precompile generates staking messages which are cascaded to
the block via the EVM in `state_processor.go`. This change cascades them
in `worker.go` to allow block proposers and block verifiers to keep the
same state.
* Run goimports for cf2dfac4081444e36a120c9432f4e..
* Update staking precompile epoch to 2 for localnet
Bring it in line with staking epoch. Change effects all configurations
except mainnet and testnet. `goimports` included.
* Add read only precompile to fetch the epoch num
* Move epoch precompile to 250
* precompiles: left pad the returned epoch number
* chainConfig: check epochs for precompiles
panic if staking precompile epoch < pre staking epoch
* Add staking migration precompile
- Lives at address 251
- Migrates delegations + pending undelegations from address A to B
- Useful if address A is hacked
- Charges gas of 21k + cost of bytes for two addresses
- Does not remove existing delegations, just sets them to zero.
Replicates current undelegate setup
- Unit tests and `goimports` included. Integration test following
shortly in MaxMustermann2/harmony-staking-precompiles
* Migration precompile: merge into staking
Merge the two precompiles into one, add gas calculation for migration
precompile. Move epoch precompile to 251 as a result. When migrating,
add undelegations to `To`'s existing undelegations, if any match the
epoch.
* Add migration gas test, remove panic, add check
In response to review comments, add tests for migration gas wherein
there are 0/1/2 delegations to migrate. Add the index out of bound check
to migration gas calculator and remove panics. Lastly, re-sort
migrated undelegations if no existing undelegation in the same epoch was
found on `To`.
* Move undelegations sorting to end of loop
3 years ago
|
|
|
func (chain *fakeChainContext) ShardID() uint32 {
|
|
|
|
return shard.BeaconChainShardID
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeChainContext) ReadValidatorSnapshot(addr common.Address) (*staking.ValidatorSnapshot, error) {
|
|
|
|
w, ok := chain.vWrappers[addr]
|
|
|
|
if !ok {
|
|
|
|
return nil, fmt.Errorf("addr not exist in snapshot")
|
|
|
|
}
|
|
|
|
cp := staketest.CopyValidatorWrapper(*w)
|
|
|
|
return &staking.ValidatorSnapshot{
|
|
|
|
Validator: &cp,
|
|
|
|
Epoch: big.NewInt(defaultEpoch),
|
|
|
|
}, nil
|
|
|
|
}
|
|
|
|
|
|
|
|
type fakeErrChainContext struct{}
|
|
|
|
|
|
|
|
func (chain *fakeErrChainContext) ReadValidatorList() ([]common.Address, error) {
|
|
|
|
return nil, errors.New("error intended from chain")
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeErrChainContext) Engine() consensus_engine.Engine {
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeErrChainContext) GetHeader(common.Hash, uint64) *block.Header {
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeErrChainContext) Config() *params.ChainConfig {
|
|
|
|
config := ¶ms.ChainConfig{}
|
|
|
|
config.MinCommissionRateEpoch = big.NewInt(10)
|
|
|
|
return config
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeErrChainContext) ReadDelegationsByDelegator(common.Address) (staking.DelegationIndexes, error) {
|
|
|
|
return nil, nil
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeErrChainContext) ReadDelegationsByDelegatorAt(delegator common.Address, blockNum *big.Int) (staking.DelegationIndexes, error) {
|
|
|
|
return nil, nil
|
|
|
|
}
|
|
|
|
|
Resolve harmony-one/bounties#77: Staking precompiles (#3906)
* Resolve harmony-one/bounties#77: Staking precompiles
Create write capable precompiles that can perform staking transactions
Add hard fork logic (EpochTBD) for these precompiles
Tests for new code with at least 80% unit test coverage
Staking library + tests in MaxMustermann2/harmony-staking-precompiles
* Fix small typo in comment
* Run goimports on files to fix Travis
* Do not activate staking precompile on shard 0
* Cascade readOnly to WriteCapableContract
* No overlap in readOnly + writeCapable precompiles
* Use function selector instead of directive
From Solidity, use abi.encodeWithSelector and match it against the
exact ABI of the functions. This allows us to remove the need for
a directive (32) being encoded, and thus saves 28 bytes of data.
* Do not allow contracts to become validators
As discussed with Jacky on #3906
* Merge harmony-one/harmony/main properly this time
* Run goimports
* Update gas calculation for staking precompile
Please see comment in core/vm/contracts_write.go RequiredGas
* Do not allow contract to become validator (2/2)
* Cache StakeMsgs from precompiled transactions
Add the StakeMsgs to ProcessorResult and cascade them in insertChain
* Remove ContractCode fields from validators
Since smart contracts can no longer beecome validators,
this field is superfluous. Remove it from the Wrapper
structure, and do not assign it a value when creating
a validator. Build and goimports checked
* Update comments in response to feedback
(1) Comments to start with function names
(2) Comments for public variables
(3) Comment to match function name RunPrecompiledContract
(4) Clarify that CreateValidatorFunc + EditValidatorFunc are still used
* Fix Travis build by reverting rosetta change
* Add revert capability to 3 staking tx types
- Delegate
- Undelegate
- CollectRewards
* Fix build: Update evm_test for ValidatorWrapper
* Merge main into harmony-staking-precompiles
* Add gas for precompile calls and allow EOA usage
- Each time the precompile is called, charge the base gas fee plus data
cost (if data can be parsed successfully). A gas fee is added to
prevent benevolent contract deployers from subsidizing the staking
transactions for EOAs through repeated assembly `delegatecall`.
- Allow EOAs to use the staking precompile directly. Some changes to
the Solidity library are associated with this change.
- Remove bytes from parsing address, since the ABI unpacks it into an
address format correctly.
- Add or update tests. Test coverage report to be attached to the PR
shortly.
* Run goimports
* Check read only and write capable for overlap
* Handle precompile stakeMsgs for block proposer
The staking precompile generates staking messages which are cascaded to
the block via the EVM in `state_processor.go`. This change cascades them
in `worker.go` to allow block proposers and block verifiers to keep the
same state.
* Run goimports for cf2dfac4081444e36a120c9432f4e..
* Update staking precompile epoch to 2 for localnet
Bring it in line with staking epoch. Change effects all configurations
except mainnet and testnet. `goimports` included.
* Add read only precompile to fetch the epoch num
* Move epoch precompile to 250
* precompiles: left pad the returned epoch number
* chainConfig: check epochs for precompiles
panic if staking precompile epoch < pre staking epoch
* Add staking migration precompile
- Lives at address 251
- Migrates delegations + pending undelegations from address A to B
- Useful if address A is hacked
- Charges gas of 21k + cost of bytes for two addresses
- Does not remove existing delegations, just sets them to zero.
Replicates current undelegate setup
- Unit tests and `goimports` included. Integration test following
shortly in MaxMustermann2/harmony-staking-precompiles
* Migration precompile: merge into staking
Merge the two precompiles into one, add gas calculation for migration
precompile. Move epoch precompile to 251 as a result. When migrating,
add undelegations to `To`'s existing undelegations, if any match the
epoch.
* Add migration gas test, remove panic, add check
In response to review comments, add tests for migration gas wherein
there are 0/1/2 delegations to migrate. Add the index out of bound check
to migration gas calculator and remove panics. Lastly, re-sort
migrated undelegations if no existing undelegation in the same epoch was
found on `To`.
* Move undelegations sorting to end of loop
3 years ago
|
|
|
func (chain *fakeErrChainContext) ShardID() uint32 {
|
|
|
|
return 900 // arbitrary number different from BeaconChainShardID
|
|
|
|
}
|
|
|
|
|
|
|
|
func (chain *fakeErrChainContext) ReadValidatorSnapshot(common.Address) (*staking.ValidatorSnapshot, error) {
|
|
|
|
return nil, errors.New("error intended")
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeIdentityStr(item interface{}) string {
|
|
|
|
return fmt.Sprintf("harmony-one-%v", item)
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeTestAddr(item interface{}) common.Address {
|
|
|
|
s := fmt.Sprintf("harmony-one-%v", item)
|
|
|
|
return common.BytesToAddress([]byte(s))
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeKeyPairs(size int) []blsPubSigPair {
|
|
|
|
pairs := make([]blsPubSigPair, 0, size)
|
|
|
|
for i := 0; i != size; i++ {
|
|
|
|
pairs = append(pairs, makeBLSKeyPair())
|
|
|
|
}
|
|
|
|
return pairs
|
|
|
|
}
|
|
|
|
|
|
|
|
type blsPubSigPair struct {
|
|
|
|
pub bls.SerializedPublicKey
|
|
|
|
sig bls.SerializedSignature
|
|
|
|
}
|
|
|
|
|
|
|
|
func makeBLSKeyPair() blsPubSigPair {
|
|
|
|
blsPriv := bls.RandPrivateKey()
|
|
|
|
blsPub := blsPriv.GetPublicKey()
|
|
|
|
msgHash := hash.Keccak256([]byte(staking.BLSVerificationStr))
|
|
|
|
sig := blsPriv.SignHash(msgHash)
|
|
|
|
|
|
|
|
var shardPub bls.SerializedPublicKey
|
|
|
|
copy(shardPub[:], blsPub.Serialize())
|
|
|
|
|
|
|
|
var shardSig bls.SerializedSignature
|
|
|
|
copy(shardSig[:], sig.Serialize())
|
|
|
|
|
|
|
|
return blsPubSigPair{shardPub, shardSig}
|
|
|
|
}
|
|
|
|
|
|
|
|
func assertError(got, expect error) error {
|
|
|
|
if (got == nil) != (expect == nil) {
|
|
|
|
return fmt.Errorf("unexpected error [%v] / [%v]", got, expect)
|
|
|
|
}
|
|
|
|
if (got == nil) || (expect == nil) {
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
if !strings.Contains(got.Error(), expect.Error()) {
|
|
|
|
return fmt.Errorf("unexpected error [%v] / [%v]", got, expect)
|
|
|
|
}
|
|
|
|
return nil
|
|
|
|
}
|