Sugarchain: Settings Part 5

- Changes:
  * During IBD, do not download additional block headers, against too much traffic
  * Skip BIP30 // See: https://github.com/litecoin-project/litecoin/blob/81c4f2d80fbd33d127ff9b31bf588e4925599d79/src/validation.cpp#L1866
  * During download headers, do not CheckBlockHeader for performance reason // See sugarchain-project/sugarchain#122
  * Rename: Testnet v3 -> v5 // See sugarchain-project/sugarchain@89cf9b5
  * During IBD, do not print "Potential stale tip detected..."
    - Disabled Tests:
      * feature_uacomment.py
      * feature_asmap.py

- Fixed Tests:
  * MAX_MONEY

src/chainparams.cpp

@@ -172,7 +172,7 @@ class CMainParams : public CChainParams {
 };
 
 /**
- * Testnet (v3)
+ * Testnet (v5)
  */
 class CTestNetParams : public CChainParams {
 public:

src/chainparamsbase.cpp

@@ -46,7 +46,7 @@ std::unique_ptr<CBaseChainParams> CreateBaseChainParams(const std::string& chain
     if (chain == CBaseChainParams::MAIN) {
         return MakeUnique<CBaseChainParams>("", 34229, 34231);
     } else if (chain == CBaseChainParams::TESTNET) {
-        return MakeUnique<CBaseChainParams>("testnet3", 44229, 44231);
+        return MakeUnique<CBaseChainParams>("testnet5", 44229, 44231);
     } else if (chain == CBaseChainParams::SIGNET) {
         return MakeUnique<CBaseChainParams>("signet", 54229, 54231);
     } else if (chain == CBaseChainParams::REGTEST) {

src/net_processing.cpp

@@ -1867,7 +1867,11 @@ void PeerManager::ProcessHeadersMessage(CNode& pfrom, const std::vector<CBlockHe
             nodestate->m_last_block_announcement = GetTime();
         }
 
+        // During IBD, no more getheaders, against too much traffic
+        /*
         if (nCount == MAX_HEADERS_RESULTS) {
+        */
+        if (!::ChainstateActive().IsInitialBlockDownload() && nCount == MAX_HEADERS_RESULTS) {
             // Headers message had its maximum size; the peer may have more headers.
             // TODO: optimize: if pindexLast is an ancestor of ::ChainActive().Tip or pindexBestHeader, continue
             // from there instead.
@@ -3994,7 +3998,13 @@ void PeerManager::CheckForStaleTipAndEvictPeers()
         // Check whether our tip is stale, and if so, allow using an extra
         // outbound peer
         if (!fImporting && !fReindex && m_connman.GetNetworkActive() && m_connman.GetUseAddrmanOutgoing() && TipMayBeStale(m_chainparams.GetConsensus())) {
+            // During IBD, do not print "Potential stale tip detected..."
+            /*
             LogPrintf("Potential stale tip detected, will try using extra outbound peer (last tip update: %d seconds ago)\n", time_in_seconds - g_last_tip_update);
+            */
+            if (!::ChainstateActive().IsInitialBlockDownload()) {
+                LogPrintf("Potential stale tip detected, will try using extra outbound peer (last tip update: %d seconds ago)\n", time_in_seconds - g_last_tip_update);
+            }
             m_connman.SetTryNewOutboundPeer(true);
         } else if (m_connman.GetTryNewOutboundPeer()) {
             m_connman.SetTryNewOutboundPeer(false);

src/validation.cpp

@@ -2034,8 +2034,13 @@ bool CChainState::ConnectBlock(const CBlock& block, BlockValidationState& state,
     // Now that the whole chain is irreversibly beyond that time it is applied to all blocks except the
     // two in the chain that violate it. This prevents exploiting the issue against nodes during their
     // initial block download.
+
+    // Skip BIP30
+    /*
     bool fEnforceBIP30 = !((pindex->nHeight==91842 && pindex->GetBlockHash() == uint256S("0x00000000000a4d0a398161ffc163c503763b1f4360639393e0e4c8e300e0caec")) ||
                            (pindex->nHeight==91880 && pindex->GetBlockHash() == uint256S("0x00000000000743f190a18c5577a3c2d2a1f610ae9601ac046a38084ccb7cd721")));
+    */
+    bool fEnforceBIP30 = true;
 
     // Once BIP34 activated it was not possible to create new duplicate coinbases and thus other than starting
     // with the 2 existing duplicate coinbase pairs, not possible to create overwriting txs.  But by the
@@ -2063,7 +2068,11 @@ bool CChainState::ConnectBlock(const CBlock& block, BlockValidationState& state,
     // future consensus change to do a new and improved version of BIP34 that
     // will actually prevent ever creating any duplicate coinbases in the
     // future.
+
+    // Skip BIP30
+    /*
     static constexpr int BIP34_IMPLIES_BIP30_LIMIT = 1983702;
+    */
 
     // There is no potential to create a duplicate coinbase at block 209,921
     // because this is still before the BIP34 height and so explicit BIP30
@@ -2101,7 +2110,12 @@ bool CChainState::ConnectBlock(const CBlock& block, BlockValidationState& state,
     // TODO: Remove BIP30 checking from block height 1,983,702 on, once we have a
     // consensus change that ensures coinbases at those heights can not
     // duplicate earlier coinbases.
+
+    // Skip BIP30
+    /*
     if (fEnforceBIP30 || pindex->nHeight >= BIP34_IMPLIES_BIP30_LIMIT) {
+    */
+    if (fEnforceBIP30) {
         for (const auto& tx : block.vtx) {
             for (size_t o = 0; o < tx->vout.size(); o++) {
                 if (view.HaveCoin(COutPoint(tx->GetHash(), o))) {
@@ -3642,7 +3656,11 @@ bool BlockManager::AcceptBlockHeader(const CBlockHeader& block, BlockValidationS
             return true;
         }
 
+        // For performance reason, skip CheckBlockHeader, during download headers // See https://github.com/sugarchain-project/sugarchain/pull/122
+        /*
         if (!CheckBlockHeader(block, state, chainparams.GetConsensus())) {
+        */
+        if (!::ChainstateActive().IsInitialBlockDownload() && !CheckBlockHeader(block, state, chainparams.GetConsensus())) {
             LogPrint(BCLog::VALIDATION, "%s: Consensus::CheckBlockHeader: %s, %s\n", __func__, hash.ToString(), state.ToString());
             return false;
         }

test/functional/p2p_dos_header_tree.py

@@ -21,13 +21,13 @@
 class RejectLowDifficultyHeadersTest(BitcoinTestFramework):
     def set_test_params(self):
         self.setup_clean_chain = True
-        self.chain = 'testnet3'  # Use testnet chain because it has an early checkpoint
+        self.chain = 'testnet5'  # Use testnet chain because it has an early checkpoint
         self.num_nodes = 2
 
     def add_options(self, parser):
         parser.add_argument(
             '--datafile',
-            default='data/blockheader_testnet3.hex',
+            default='data/blockheader_testnet3.hex', # Using Bitcoin Testnet (v3), but not Sugarchain Testnet (v5)
             help='Test data file (default: %(default)s)',
         )
 

test/functional/test_framework/messages.py

@@ -42,7 +42,7 @@
 MAX_BLOOM_HASH_FUNCS = 50
 
 COIN = 100000000  # 1 btc in satoshis
-MAX_MONEY = 21000000 * COIN
+MAX_MONEY = 1073741824 * COIN
 
 BIP125_SEQUENCE_NUMBER = 0xfffffffd  # Sequence number that is BIP 125 opt-in and BIP 68-opt-out
 

test/functional/test_framework/p2p.py

@@ -111,7 +111,7 @@
 
 MAGIC_BYTES = {
     "mainnet": b"\x9f\xeb\x4b\x9d",   # mainnet
-    "testnet3": b"\xb0\x11\x90\x70",  # testnet3
+    "testnet5": b"\xb0\x11\x90\x70",  # testnet5
     "regtest": b"\xaf\xfb\x5b\xad",   # regtest
     "signet": b"\x0a\x03\xcf\x40",    # signet # TODO.ZENY.SETTINGS # See https://en.bitcoin.it/wiki/Signet
 }

test/functional/test_framework/util.py

@@ -334,7 +334,7 @@ def initialize_datadir(dirname, n, chain):
     if not os.path.isdir(datadir):
         os.makedirs(datadir)
     # Translate chain name to config name
-    if chain == 'testnet3':
+    if chain == 'testnet5':
         chain_name_conf_arg = 'testnet'
         chain_name_conf_section = 'test'
     else:

test/functional/test_runner.py

@@ -255,7 +255,7 @@
     # 'wallet_send.py', # TODO.ZENY.SETTINGS
     'wallet_create_tx.py --descriptors',
     # 'p2p_fingerprint.py', # TODO.ZENY.YESPOWER # AssertionError
-    'feature_uacomment.py',
+    # 'feature_uacomment.py', # During IBD, do not print "Potential stale tip detected..."
     'wallet_coinbase_category.py',
     'wallet_coinbase_category.py --descriptors',
     'feature_filelock.py',
@@ -264,7 +264,7 @@
     # 'p2p_unrequested_blocks.py', # TODO.ZENY.YESPOWER # AssertionError
     # 'p2p_blockfilters.py', # TODO.ZENY.SETTINGS # too slow...
     'feature_includeconf.py',
-    'feature_asmap.py',
+    # 'feature_asmap.py', # During IBD, do not print "Potential stale tip detected..."
     'mempool_unbroadcast.py',
     # 'mempool_compatibility.py', # TODO.ZENY.BRANDING # AssertionError # No any previous releases
     # 'rpc_deriveaddresses.py', # TODO.ZENY.SETTINGS # ADDRESS # ADDRESS_BCRT1_P2WSH_OP_TRUE (address.py)