Fix/prevent reverts #408
Merged
Fix/prevent reverts #408
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bvbfan
requested review from
Bushstar,
fuxingloh,
Mixa84,
monstrobishi,
ShengguangXiao,
surangap and
uzyn
as code owners
Signed-off-by: Anthony Fieroni <[email protected]>
bvbfan
force-pushed
the
fix/prevent_reverts
branch
from
fc7922b to
35ad6c8
Compare
monstrobishi
suggested changes
May 20, 2021
Signed-off-by: Anthony Fieroni <[email protected]>
Bushstar
reviewed
May 20, 2021
Bushstar
approved these changes
May 21, 2021
monstrobishi
approved these changes
May 21, 2021
monstrobishi
added a commit
that referenced
this pull request
May 24, 2021
* Randomise winning anchor on blockhash (#314) * Randomise winning anchor on blockhash * Remove unused function argument * Lazy rewards redistribution (#279) * Split custom tx processing into parsing and applying Signed-off-by: Anthony Fieroni <[email protected]> * Prevent copies in serialize Signed-off-by: Anthony Fieroni <[email protected]> * Adopt pool tables to have historical records Per account rewards calculation Signed-off-by: Anthony Fieroni <[email protected]> * Do not redistribute rewards per block Signed-off-by: Anthony Fieroni <[email protected]> * Fix lint circular dependency Signed-off-by: Anthony Fieroni <[email protected]> * Introduce db version check at runtime Signed-off-by: Anthony Fieroni <[email protected]> * Use bidirectional storage iterators Signed-off-by: Anthony Fieroni <[email protected]> * Fix pool id serialization in update poolpair tx Signed-off-by: Anthony Fieroni <[email protected]> * Update rewards on every block as drop-in replacement Signed-off-by: Anthony Fieroni <[email protected]> * Split pool historical records * Added EunosHeight * Fork out rewards redistribution per block Signed-off-by: Anthony Fieroni <[email protected]> * Extend pool swap functional test to Eunos hieght Signed-off-by: Anthony Fieroni <[email protected]> * Minimize undo data size Signed-off-by: Anthony Fieroni <[email protected]> * Prune undo data prior last checkpoint Signed-off-by: Anthony Fieroni <[email protected]> * Prevent database fragmentation Signed-off-by: Anthony Fieroni <[email protected]> * Make account history own database (#323) Signed-off-by: Anthony Fieroni <[email protected]> Co-authored-by: surangap <[email protected]> Co-authored-by: Ahmed Hilali <[email protected]> * Split pool reward records (#332) Signed-off-by: Anthony Fieroni <[email protected]> * DFIP8 New emissions (#328) * New emissions * Use subsidy map * Assign swap, futures and options to unallocated * Show non-UTXO rewards in getblock * Oracle development (#287) * Oracle development Signed-off-by: Anthony Fieroni <[email protected]> * Fix rpc parameter not converted in defi-cli (#319) * Update rpc_oracles.cpp Fixed rpc documentation * Updated devnet heights Co-authored-by: Anthony Fieroni <[email protected]> Co-authored-by: Ahmed Hilali <[email protected]> * Fix compile error (#341) * Fix build issue on GCC5 Signed-off-by: Anthony Fieroni <[email protected]> * Fix unit test fail * Updated branch matching for dev builds * Updated dev build branch matching * Feature/decodecustomtx (#340) * Added GetBlockHash() function to retrieve the block hash given the transaction hash * Added new RPC decodecustomtx * Added tests for decodecustomtx RPC * Added iswitness parameter to vRPCConvertParams in client.cpp * Updated decodecustomtx RPC as per new requirements * Updated decodecustomtx to use INT_MAX as default block height. Updated testing. * Removed redundant code * Added GetBlockHash() function to retrieve the block hash given the transaction hash * Added new RPC decodecustomtx * Added tests for decodecustomtx RPC * Added iswitness parameter to vRPCConvertParams in client.cpp * Updated decodecustomtx RPC as per new requirements * Updated decodecustomtx to use INT_MAX as default block height. Updated testing. * Removed redundant code Co-authored-by: monstrobishi <[email protected]> * Extend block merkle root by hash of account changes (#337) Signed-off-by: Anthony Fieroni <[email protected]> Co-authored-by: monstrobishi <[email protected]> * spv_getfeerate. SPV wallet fee filter for HTLC. SPV reset on reindex. (#356) * Added header functions back in to fix compile * Refactor: RPC oracles take array instead of string (#367) Signed-off-by: Anthony Fieroni <[email protected]> * Track UTXO and tokens sent to burn address (#354) * Track TX and non-TX burns. Zero foundation balances. * Remove Qt from depends * Remove unused variables (#372) * lint: remove unused json (#373) * Change difficulty adjust period (#378) * Remove unused GUI support code (#379) * Use block height when getting last block time for coin-age (#380) * Added listaccounts RPC pagination test (#376) * Added listaccounts RPC pagination test Optimized Account iteration * Better handle pagination in listaccounts Signed-off-by: Anthony Fieroni <[email protected]> Co-authored-by: Anthony Fieroni <[email protected]> * Fix linter error * Added hardfork heights * Add creation fee for devnet (#388) * Masternode creation requires authorization (#386) Signed-off-by: Anthony Fieroni <[email protected]> * Remove backwards compatible anchor code (#383) * Change daily LP rewards automatically on subsidy reduction (#390) * Minor lint fixes for 1.7.x (#392) * Minor lint fix. Set Ubuntu 18.04 as minimum for tests and builds. (#393) * lint: remove "dead" code * Run lint on 1.x PRs. Ubuntu 18.04 minimum for tests and builds. * Fix listaccount pagination after start one (#394) Signed-off-by: Anthony Fieroni <[email protected]> * ICX Orderbook (#335) * Initial ICX orderbook development * Inital development of ICX Orderbook * Fixes and changes. * Remove prints and small fix. * Fix seed to hash calculation * Optimize data structures. * Fix searching HTLCs. * Refactor data structures and flow, fixed PR issues * Refactor data structures and flow, fixed PR issues * Fix issues. * Added incentives and fees. Fixed issue with CAmount multiply overflow. Fix tests and clean code. (#5) * Add incentives and fees. Fix multiply overflow. * Fix lint rpc mapping check * Add poolpair id to governance variable for DFI/BTC price used in takerFee, minimum timeout for 2nd HTLC, added initial unit test, various fixes and code clean. * Fix calculation of takerFee. Clean code. * Get poolpair id from governance variable. * Adding expiration for minimum timeout to EXT HTLC. * Remove locking funds on makeoffer, fixes. * Add initial test that add creation of poolpair. * Added takerFeePerBTC gov var, adapted timeouts for htlcs, check in rpc for correct htlc timeouts, added more checks in test. * Added key for ext htlc closed, addapted test and rpcs. * Fill partial offer to complete order * Test burn. Wip burn map in ConnectBlock. Tally tokens. * ICX Set order owner on creation * Fix regressions, reproducible crashes Polish Fixes: #374 Signed-off-by: Anthony Fieroni <[email protected]> * Remove chain data from DB, refactor address storing only in order/offer, fix auth, polish and clean code. * Add warning disclaimer * Add creation fee for devnet * Fixed HTLC timout check, refund partial takerFee on partial offer accept, added test case for BTC/DFI flow, added error test cases, polished code. * Fixed expiration of offer and htlcs, added check for open offer on submit htlc, added more test cases. * Fix order expiration refund, add checkin rpc for htlc if order expires first, fixed and added tests. * Fix findng poolpair and calculating DFI per BTC, adapted test, rpc fix. * Remove unnecessary check in GetBTCDFIPoolPair(), make BTC isDAT in error tests, change BTC blocks in DFI to 20. * Masternode creation requires authorization (#386) Signed-off-by: Anthony Fieroni <[email protected]> * Polish correct offer prices in blockchain layer Signed-off-by: Anthony Fieroni <[email protected]> * Moving more checks from RPC to mn_checks, cleanup code * Small fixes. * Follow getburninfo at all steps. Set correct amounts. * Fix lint * Fix lint * Fix typo. Co-authored-by: Peter Bushnell <[email protected]> Co-authored-by: monstrobishi <[email protected]> Co-authored-by: Anthony Fieroni <[email protected]> * Updated Eunos upgrade heights for mainnet and testnet (#391) * Update chainparams.cpp (#396) * Ignore but log merkle failures (#403) * Start burn search from default CScript (#401) * Skip old coinbase block post Eunos (#400) * Close expired orders simplifications Signed-off-by: Anthony Fieroni <[email protected]> * Skip higher coinbase spent block post Eunos Signed-off-by: Anthony Fieroni <[email protected]> * Updated code to return correct lastblockcreationattempt in getmininginfo RPC (#399) * Fixed operator == * Update chainparams.cpp * Add account destruction address on testnet (#404) * Skip merkle root check on startup (#405) * Fix/prevent reverts (#408) * Prevent chain revert past Eunos Signed-off-by: Anthony Fieroni <[email protected]> * Add consesus code to reject custom tx Signed-off-by: Anthony Fieroni <[email protected]> * Fix oracles rpc cli paramters (#409) * Fix oracles rpc cli paramters Signed-off-by: Anthony Fieroni <[email protected]> * Update oracle input parameter validation Signed-off-by: Anthony Fieroni <[email protected]> * Fix masternode hash generation by adding time to custom tx (#410) Signed-off-by: Anthony Fieroni <[email protected]> * Set new testnet fork height. Revert merkle root checks. (#411) * Prevent rewards calculation to shown in account history (#412) Signed-off-by: Anthony Fieroni <[email protected]> * Add checkpoint to testnet (#414) * Reduce new fork height to two hours ahead (#415) * Fix calculating owner rewards and returning takerFee on close offer. (#416) * Increase masternode registration and resign delay (#419) * SPV add call, fix Windows connection issue and set Eunos height. (#424) * SPV getpeers * SPV: Use non-blocking on Windows * Set Eunos height on mainnet * Use C String in peer_log to build on macOS * Client-name capitalization fix (#425) * Using ubuntu 18.04 to build for linux (#422) * Increment PROTOCOL_VERSION (#426) Co-authored-by: Peter John Bushnell <[email protected]> Co-authored-by: Anthony Fieroni <[email protected]> Co-authored-by: surangap <[email protected]> Co-authored-by: Xiao Shengguang <[email protected]> Co-authored-by: Mihailo Milenkovic <[email protected]> Co-authored-by: U-Zyn Chua <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
/kind fix
What this PR does / why we need it:
Including merkle root of account system we don't need to revert chain to last checkpoint / anchor since we validate ongoing changes.