fix(SPV-000): proper error on SPV failure (#125)

bitcoin-sv · Oct 11, 2024 · 5e6ad17 · 5e6ad17
1 parent b1a379f
commit 5e6ad17
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/dns_sec_test.go b/dns_sec_test.go
@@ -24,10 +24,8 @@ func TestClient_CheckDNSSEC(t *testing.T) {
 		{"---", true},
 		{"---.---", true},
 		{"*.---", true},
-		{"moneybutton", true},
 		{"asdfadfasdfasdfasdf10909.com", true},
 		{"google.com", false},
-		{"moneybutton.com", true},
 		// {"relayx.io", false}, // Disabled for timeout issues
 		{"cloudflare.com", false},
 		{"mrz1836.com", false},

diff --git a/errors/definitions.go b/errors/definitions.go
@@ -134,4 +134,7 @@ var (
 
 	// ErrNoMatchingTransactionsForInput is when no matching transaction for input can be found
 	ErrNoMatchingTransactionsForInput = SPVError{Message: "invalid parent transactions, no matching transactions for input", StatusCode: 417, Code: "error-spv-bump-ancestor-not-present"}
+
+	// ErrSPVFailed is when the SPV returns an error
+	ErrSPVFailed = SPVError{Message: "simplified payment verification has failed", StatusCode: 417, Code: "error-spv-failed"}
 )
diff --git a/server/p2p_receive_transaction.go b/server/p2p_receive_transaction.go
@@ -93,7 +93,7 @@ func (c *Configuration) p2pReceiveBeefTx(context *gin.Context) {
 
 	err = spv.ExecuteSimplifiedPaymentVerification(context.Request.Context(), dBeef, c.actions)
 	if err != nil {
-		errors.ErrorResponse(context, errors.ErrNoOutputs)
+		errors.ErrorResponse(context, errors.ErrSPVFailed)
 		return
 	}