fix: disable automatic resigning for the signer pkg (#3257)

Ref: #3256

pkg/user/e2e_test.go

@@ -70,9 +70,8 @@ func TestConcurrentTxSubmission(t *testing.T) {
 }
 
 func TestSignerTwins(t *testing.T) {
-	if testing.Short() {
-		t.Skip("skipping in short mode")
-	}
+	// Ref: https://github.com/celestiaorg/celestia-app/issues/3256
+	t.Skip()
 
 	// Setup network
 	tmConfig := testnode.DefaultTendermintConfig()

pkg/user/signer.go

@@ -294,9 +294,13 @@ func (s *Signer) broadcastTx(ctx context.Context, txBytes []byte, sequence uint6
 		}
 		s.networkSequence = nextSequence
 		s.localSequence = nextSequence
-		return s.retryBroadcastingTx(ctx, txBytes, nextSequence)
-	}
-	if resp.TxResponse.Code == abci.CodeTypeOK {
+		// FIXME: We can't actually resign the transaction. A malicious node
+		// may manipulate us into signing the same transaction several times
+		// and then executing them. We need some proof of what the last network
+		// sequence is rather than relying on an error provided by the node
+		// return s.retryBroadcastingTx(ctx, txBytes, nextSequence)
+		// Ref: https://github.com/celestiaorg/celestia-app/issues/3256
+	} else if resp.TxResponse.Code == abci.CodeTypeOK {
 		s.outboundSequences[sequence] = struct{}{}
 		s.reverseTxHashSequenceMap[resp.TxResponse.TxHash] = sequence
 		return resp.TxResponse, nil