sweep: log tx label in GetSweepFee methods

Added argument 'label' to GetSweepFee and GetSweepFeeDetails. It is logged
together with expected weight, fee and feerate.

Author: starius

client.go

@@ -570,8 +570,11 @@ func (s *Client) getLoopOutSweepFee(ctx context.Context, confTarget int32) (
 		htlc = swap.QuoteHtlcP2WSH
 	}
 
+	label := "loopout-quote"
+
 	return s.sweeper.GetSweepFee(
 		ctx, htlc.AddSuccessToEstimator, p2wshAddress, confTarget,
+		label,
 	)
 }
 

loopin.go

@@ -1077,10 +1077,12 @@ func (s *loopInSwap) publishTimeoutTx(ctx context.Context,
 		}
 	}
 
+	label := fmt.Sprintf("loopin-timeout-%x", s.hash[:6])
+
 	// Calculate sweep tx fee.
 	fee, err := s.sweeper.GetSweepFee(
 		ctx, s.htlc.AddTimeoutToEstimator, s.timeoutAddr,
-		TimeoutTxConfTarget,
+		TimeoutTxConfTarget, label,
 	)
 	if err != nil {
 		return 0, err

loopin_test.go

@@ -312,11 +312,13 @@ func handleHtlcExpiry(t *testing.T, ctx *loopInTestContext, inSwap *loopInSwap,
 	// Expect timeout tx to be published.
 	timeoutTx := <-ctx.lnd.TxPublishChannel
 
+	label := fmt.Sprintf("loopin-timeout-%x", inSwap.hash[:6])
+
 	// We can just get our sweep fee as we would in the swap code because
 	// our estimate is static.
 	fee, err := inSwap.sweeper.GetSweepFee(
 		context.Background(), inSwap.htlc.AddTimeoutToEstimator,
-		inSwap.timeoutAddr, TimeoutTxConfTarget,
+		inSwap.timeoutAddr, TimeoutTxConfTarget, label,
 	)
 	require.NoError(t, err)
 	cost.Onchain += fee

sweep/sweeper.go

@@ -177,26 +177,27 @@ func (s *Sweeper) CreateSweepTx(
 
 // GetSweepFee calculates the required tx fee to spend to P2WKH. It takes a
 // function that is expected to add the weight of the input to the weight
-// estimator.
+// estimator. It also takes a label used for logging.
 func (s *Sweeper) GetSweepFee(ctx context.Context,
 	addInputEstimate func(*input.TxWeightEstimator) error,
-	destAddr btcutil.Address, sweepConfTarget int32) (
+	destAddr btcutil.Address, sweepConfTarget int32, label string) (
 	btcutil.Amount, error) {
 
 	// Use GetSweepFeeDetails to get the fee and other unused data.
 	fee, _, _, err := s.GetSweepFeeDetails(
-		ctx, addInputEstimate, destAddr, sweepConfTarget,
+		ctx, addInputEstimate, destAddr, sweepConfTarget, label,
 	)
 
 	return fee, err
 }
 
 // GetSweepFeeDetails calculates the required tx fee to spend to P2WKH. It takes
 // a function that is expected to add the weight of the input to the weight
-// estimator. It returns also the fee rate and transaction weight.
+// estimator. It also takes a label used for logging. It returns also the fee
+// rate and transaction weight.
 func (s *Sweeper) GetSweepFeeDetails(ctx context.Context,
 	addInputEstimate func(*input.TxWeightEstimator) error,
-	destAddr btcutil.Address, sweepConfTarget int32) (
+	destAddr btcutil.Address, sweepConfTarget int32, label string) (
 	btcutil.Amount, chainfee.SatPerKWeight, lntypes.WeightUnit, error) {
 
 	// Get fee estimate from lnd.
@@ -224,7 +225,14 @@ func (s *Sweeper) GetSweepFeeDetails(ctx context.Context,
 	// Find weight.
 	weight := weightEstimate.Weight()
 
-	return feeRate.FeeForWeight(weight), feeRate, weight, nil
+	// Find fee.
+	fee := feeRate.FeeForWeight(weight)
+
+	log.Debugf("Estimations for a tx (label=%s): weight=%v, fee=%v, "+
+		"feerate=%v, sweepConfTarget=%d.", label, weight, fee, feeRate,
+		sweepConfTarget)
+
+	return fee, feeRate, weight, nil
 }
 
 // AddOutputEstimate adds output to weight estimator.