fix: reorder auth checks for tests

Layr-Labs · Dec 18, 2024 · 11eec21 · 11eec21
1 parent c923767
commit 11eec21
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/disperser/apiserver/disperse_blob_v2.go b/disperser/apiserver/disperse_blob_v2.go
@@ -117,9 +117,6 @@ func (s *DispersalServerV2) validateDispersalRequest(ctx context.Context, req *p
 	if blobHeader.PaymentMetadata == (core.PaymentMetadata{}) {
 		return api.NewErrorInvalidArg("payment metadata is required")
 	}
-	if err = s.authenticator.AuthenticateBlobRequest(blobHeader); err != nil {
-		return api.NewErrorInvalidArg(fmt.Sprintf("authentication failed: %s", err.Error()))
-	}
 
 	if len(blobHeader.PaymentMetadata.AccountID) == 0 || blobHeader.PaymentMetadata.ReservationPeriod == 0 || blobHeader.PaymentMetadata.CumulativePayment == nil {
 		return api.NewErrorInvalidArg("invalid payment metadata")
@@ -139,6 +136,10 @@ func (s *DispersalServerV2) validateDispersalRequest(ctx context.Context, req *p
 		}
 	}
 
+	if err = s.authenticator.AuthenticateBlobRequest(blobHeader); err != nil {
+		return api.NewErrorInvalidArg(fmt.Sprintf("authentication failed: %s", err.Error()))
+	}
+
 	// validate every 32 bytes is a valid field element
 	_, err = rs.ToFrArray(data)
 	if err != nil {