GlcoinCore/loop
1
0
Fork 0
mirror of https://github.com/lightninglabs/loop.git synced 2026-08-13 12:33:03 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #1158 from hieblmi/log-htlc-tx-fee
Some checks failed
CI / RPC compilation check (push) Has been cancelled
Details
CI / SQL compilation check (push) Has been cancelled
Details
CI / go mod check (push) Has been cancelled
Details
CI / build and lint code (push) Has been cancelled
Details
CI / verify that auto-generated documentation is up-to-date (push) Has been cancelled
Details
CI / run unit-test sqlite3 race (push) Has been cancelled
Details
CI / run unit-test postgres race (push) Has been cancelled
Details
CI / run LiT itests (push) Has been cancelled
Details
CI / run LiT unit tests (push) Has been cancelled
Details

Browse source 
staticaddr/loopin: log HTLC fee guard inputs
This commit is contained in:
Slyghtning 2026-06-24 14:39:36 +02:00 • committed by GitHub
commit a134cb1f22
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 40 additions and 21 deletions
Download patch file Download diff file Expand all files Collapse all files

61
staticaddr/loopin/actions.go
View file

@ -92,13 +92,14 @@ func (f *FSM) InitHtlcAction(ctx context.Context,
// swap amount minus the fees that the server charges for the swap. The
// swap amount is either the total value of the selected deposits, or
// the selected amount if a specific amount was requested.
swapAmount := f.loopIn.TotalDepositAmount()
totalDepositAmount := f.loopIn.TotalDepositAmount()
swapAmount := totalDepositAmount
var changeAmount btcutil.Amount
var hasChange bool
if f.loopIn.SelectedAmount > 0 {
swapAmount = f.loopIn.SelectedAmount
remainingAmount := f.loopIn.TotalDepositAmount() - swapAmount
hasChange = remainingAmount > 0 && remainingAmount <
f.loopIn.TotalDepositAmount()
changeAmount = totalDepositAmount - swapAmount
hasChange = changeAmount > 0 && changeAmount < totalDepositAmount
}
swapInvoiceAmt := swapAmount - f.loopIn.QuotedSwapFee
@ -247,44 +248,62 @@ func (f *FSM) InitHtlcAction(ctx context.Context,
maxHtlcTxBackupFee := btcutil.Amount(amt *
f.cfg.MaxStaticAddrHtlcBackupFeePercentage)
htlcWeight := f.loopIn.htlcWeight(hasChange)
feeRate := chainfee.SatPerKWeight(loopInResp.StandardHtlcInfo.FeeRate)
fee := feeRate.FeeForWeight(f.loopIn.htlcWeight(hasChange))
fee := feeRate.FeeForWeight(htlcWeight)
highFeeRate := chainfee.SatPerKWeight(loopInResp.HighFeeHtlcInfo.FeeRate)
highFee := highFeeRate.FeeForWeight(htlcWeight)
extremelyHighFeeRate := chainfee.SatPerKWeight(
loopInResp.ExtremeFeeHtlcInfo.FeeRate,
)
extremelyHighFee := extremelyHighFeeRate.FeeForWeight(htlcWeight)
f.Debugf("htlc fee validation: "+
"deposit_count=%d, total_deposit=%v, "+
"swap_amount=%v, change_amount=%v, has_change=%v, "+
"htlc_weight=%v, standard_fee_rate=%v, standard_fee=%v, "+
"high_fee_rate=%v, high_fee=%v, extreme_fee_rate=%v, "+
"extreme_fee=%v, max_fee=%v, max_backup_fee=%v",
len(f.loopIn.Deposits), totalDepositAmount, swapAmount,
changeAmount, hasChange, htlcWeight, feeRate, fee, highFeeRate,
highFee, extremelyHighFeeRate, extremelyHighFee, maxHtlcTxFee,
maxHtlcTxBackupFee)
if fee > maxHtlcTxFee {
// Abort the swap by pushing empty sigs to the server.
pushEmptySigs()
log.Errorf("server htlc tx fee is higher than the configured "+
"allowed maximum: %v > %v", fee, maxHtlcTxFee)
f.Errorf("server standard htlc tx fee is higher than the "+
"configured allowed maximum: %v > %v "+
"(fee_rate=%v, weight=%v)",
fee, maxHtlcTxFee, feeRate, htlcWeight)
return returnError(ErrFeeTooHigh)
}
f.loopIn.HtlcTxFeeRate = feeRate
highFeeRate := chainfee.SatPerKWeight(loopInResp.HighFeeHtlcInfo.FeeRate)
fee = highFeeRate.FeeForWeight(f.loopIn.htlcWeight(hasChange))
if fee > maxHtlcTxBackupFee {
if highFee > maxHtlcTxBackupFee {
// Abort the swap by pushing empty sigs to the server.
pushEmptySigs()
log.Errorf("server htlc backup tx fee is higher than the "+
"configured allowed maximum: %v > %v", fee,
maxHtlcTxBackupFee)
f.Errorf("server high-fee htlc backup tx fee is higher "+
"than the configured allowed maximum: %v > %v "+
"(fee_rate=%v, weight=%v)",
highFee, maxHtlcTxBackupFee, highFeeRate, htlcWeight)
return returnError(ErrFeeTooHigh)
}
f.loopIn.HtlcTxHighFeeRate = highFeeRate
extremelyHighFeeRate := chainfee.SatPerKWeight(
loopInResp.ExtremeFeeHtlcInfo.FeeRate,
)
fee = extremelyHighFeeRate.FeeForWeight(f.loopIn.htlcWeight(hasChange))
if fee > maxHtlcTxBackupFee {
if extremelyHighFee > maxHtlcTxBackupFee {
// Abort the swap by pushing empty sigs to the server.
pushEmptySigs()
log.Errorf("server htlc backup tx fee is higher than the "+
"configured allowed maximum: %v > %v", fee,
maxHtlcTxBackupFee)
f.Errorf("server extreme-fee htlc backup tx fee is "+
"higher than the configured allowed maximum: %v > %v "+
"(fee_rate=%v, weight=%v)",
extremelyHighFee, maxHtlcTxBackupFee,
extremelyHighFeeRate, htlcWeight)
return returnError(ErrFeeTooHigh)
}