Clarify fee credit check

ACINQ · Sep 24, 2024 · 5d21e08 · 5d21e08
1 parent 2b3062f
commit 5d21e08
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/commonMain/kotlin/fr/acinq/lightning/payment/IncomingPaymentHandler.kt b/src/commonMain/kotlin/fr/acinq/lightning/payment/IncomingPaymentHandler.kt
@@ -247,12 +247,16 @@ class IncomingPaymentHandler(val nodeParams: NodeParams, val db: PaymentsDb) {
                                             // in terms of fees we need to pay, otherwise we may not have enough to actually pay the liquidity fees.
                                             val maxFeerate = currentFeerate * AddLiquidityForIncomingPayment.SpliceWithNoBalanceFeerateRatio
                                             val maxLiquidityFees = fundingRate.fees(maxFeerate, requestedAmount, requestedAmount, isChannelCreation = true).total.toMilliSatoshi()
-                                            val feeCreditThreshold = when (val policy = nodeParams.liquidityPolicy.value) {
-                                                LiquidityPolicy.Disable -> maxLiquidityFees
-                                                is LiquidityPolicy.Auto -> maxLiquidityFees.min(policy.maxAllowedFeeCredit)
+                                            val maxFeeCredit = when (val policy = nodeParams.liquidityPolicy.value) {
+                                                LiquidityPolicy.Disable -> 0.msat
+                                                is LiquidityPolicy.Auto -> policy.maxAllowedFeeCredit
                                             }
-                                            val amountBelowThreshold = (willAddHtlcParts.map { it.amount }.sum() + currentFeeCredit) <= feeCreditThreshold
-                                            featureOk && amountBelowThreshold
+                                            val nextFeeCredit = willAddHtlcParts.map { it.amount }.sum() + currentFeeCredit
+                                            val cannotCoverLiquidityFees = nextFeeCredit <= maxLiquidityFees
+                                            val isBelowMaxFeeCredit = nextFeeCredit <= maxFeeCredit
+                                            val assessment = featureOk && cannotCoverLiquidityFees && isBelowMaxFeeCredit
+                                            logger.info { "fee credit assessment: result=$assessment featureOk=$featureOk cannotCoverLiquidityFees=$cannotCoverLiquidityFees isBelowMaxFeeCredit=$isBelowMaxFeeCredit (nextFeeCredit=$nextFeeCredit, maxLiquidityFees=$maxLiquidityFees, maxFeeCredit=$maxFeeCredit)" }
+                                            assessment
                                         }
                                         when {
                                             addToFeeCredit -> {