Release 1.0.5

- Fix hitting error "coinbase has too many outputs: got 26 where the limit is 20" by doubling the coinbase output limit, because of developer fee is counted as output with coinbase outputs.
Hoosat-Oy · Sep 17, 2024 · 6b7fd31 · 6b7fd31
1 parent 3df7e2e
commit 6b7fd31
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/changelog.txt b/changelog.txt
@@ -1,3 +1,9 @@
+HTND v1.0.5 - 2024-08-17
+===========================
+
+* Fix hitting error "coinbase has too many outputs: got 26 where the limit is 20" by doubling the coinbase output limit, because of developer fee is counted as output with coinbase outputs. 
+
+
 HTND v1.0.4 - 2024-08-15
 ===========================
 

diff --git a/domain/consensus/processes/transactionvalidator/transaction_in_isolation.go b/domain/consensus/processes/transactionvalidator/transaction_in_isolation.go
@@ -132,6 +132,8 @@ func (v *transactionValidator) checkCoinbaseInIsolation(tx *externalapi.DomainTr
 	}
 
 	outputsLimit := uint64(v.ghostdagK) + 2
+	// Make the outputsLimits twice the size because of developer fee in outputs with coinbase outputs.
+	outputsLimit *= 2
 	if uint64(len(tx.Outputs)) > outputsLimit {
 		return errors.Wrapf(ruleerrors.ErrCoinbaseTooManyOutputs, "coinbase has too many outputs: got %d where the limit is %d", len(tx.Outputs), outputsLimit)
 	}

diff --git a/version/version.go b/version/version.go
@@ -11,7 +11,7 @@ const validCharacters = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrs
 const (
 	appMajor uint = 1
 	appMinor uint = 0
-	appPatch uint = 4
+	appPatch uint = 5
 )
 
 // appBuild is defined as a variable so it can be overridden during the build