Merge pull request #192 from XinFinOrg/feature-v0.3.2

increase tx size limit and better naming

Author: wanwiset25

.github/workflows/ci.yml

@@ -9,7 +9,7 @@ on:
 
 jobs:
   test_golang:
-    name: Test, build and push image to registry
+    name: Test (unit tests) 
     runs-on: ubuntu-latest
     env:
       GOPATH: ${{ github.workspace }}

core/tx_pool.go

@@ -597,7 +597,8 @@ func (pool *TxPool) validateTx(tx *types.Transaction, local bool) error {
 	}
 
 	// Heuristic limit, reject transactions over 32KB to prevent DOS attacks
-	if tx.Size() > 32*1024 {
+	// test size increase TODO: bring over EIP2464 from XDPoS to allow actual 128KB
+	if tx.Size() > 128*1024 {
 		return ErrOversizedData
 	}
 	// Transactions can't be negative. This may never happen using RLP decoded

eth/hooks/engine_v2_hooks.go

@@ -20,6 +20,7 @@ import (
 )
 
 func AttachConsensusV2Hooks(adaptor *XDPoS.XDPoS, bc *core.BlockChain, chainConfig *params.ChainConfig) {
+	
 	// Hook scans for bad masternodes and decide to penalty them
 	// Subnet penalty is triggered at gap block, and stopped at previous gap block
 	adaptor.EngineV2.HookPenalty = func(chain consensus.ChainReader, number *big.Int, currentHash common.Hash, candidates []common.Address, config *params.XDPoSConfig) ([]common.Address, error) {