Fix: do not send signed staking transaction to btc

staker/stakerapp.go

@@ -1316,7 +1316,23 @@ func (app *StakerApp) handlePreApprovalCmd(cmd *stakingRequestCmd) error {
 func (app *StakerApp) handlePostApprovalCmd(cmd *stakingRequestCmd) error {
 	bestBlockHeight := app.currentBestBlockHeight.Load()
 
-	_, err := app.wc.SendRawTransaction(cmd.stakingTx, true)
+	err := app.wc.UnlockWallet(defaultWalletUnlockTimeout)
+
+	if err != nil {
+		return err
+	}
+
+	tx, fullySignd, err := app.wc.SignRawTransaction(cmd.stakingTx)
+
+	if err != nil {
+		return err
+	}
+
+	if !fullySignd {
+		return fmt.Errorf("failed to fully sign transaction with hash %s", cmd.stakingTxHash)
+	}
+
+	_, err = app.wc.SendRawTransaction(tx, true)
 
 	if err != nil {
 		return err
@@ -1780,8 +1796,9 @@ func (app *StakerApp) StakeFunds(
 
 	feeRate := app.feeEstimator.EstimateFeePerKb()
 
-	// Create unsigned transaction by wallet
-	tx, err := app.wc.CreateAndSignTx([]*wire.TxOut{stakingInfo.StakingOutput}, btcutil.Amount(feeRate), stakerAddress)
+	// Create unsigned transaction by wallet without signing. Signing will happen
+	// in next steps
+	tx, err := app.wc.CreateTransaction([]*wire.TxOut{stakingInfo.StakingOutput}, btcutil.Amount(feeRate), stakerAddress)
 
 	if err != nil {
 		return nil, err

stakerdb/trackedtransactionstore_test.go

@@ -195,7 +195,7 @@ func TestStateTransitions(t *testing.T) {
 	// Inital state
 	storedTx, err := s.GetTransaction(&txHash)
 	require.NoError(t, err)
-	require.Equal(t, proto.TransactionState_TRANSACTION_CREATED, storedTx.State)
+	require.Equal(t, proto.TransactionState_SENT_TO_BTC, storedTx.State)
 	require.Equal(t, uint64(1), storedTx.StoredTransactionIdx)
 	// Confirmed
 	hash := datagen.GenRandomBtcdHash(r)

walletcontroller/interface.go

@@ -49,7 +49,7 @@ type WalletController interface {
 	SignRawTransaction(tx *wire.MsgTx) (*wire.MsgTx, bool, error)
 	// requires wallet to be unlocked
 	CreateAndSignTx(
-		output []*wire.TxOut,
+		outputs []*wire.TxOut,
 		feeRatePerKb btcutil.Amount,
 		changeAddress btcutil.Address,
 	) (*wire.MsgTx, error)