From 82f591dbab569b6c0f98c2ed56299b8e97c92f17 Mon Sep 17 00:00:00 2001 From: Olaoluwa Osuntokun Date: Tue, 16 May 2017 19:02:47 -0700 Subject: [PATCH] lnwallet: add TODO to properly roll dust outputs into initiator's paid fee --- lnwallet/channel.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lnwallet/channel.go b/lnwallet/channel.go index 22fe00b7..08925f64 100644 --- a/lnwallet/channel.go +++ b/lnwallet/channel.go @@ -1283,6 +1283,10 @@ func (lc *LightningChannel) fetchCommitmentView(remoteChain bool, // Determine how many current HTLCs are over the dust limit, and should // be counted for the purpose of fee calculation. + // TODO(roasbeef): dust outputs need to be counted towards fees paid + // * tally in separate accumulator, subtract from fee amount + // * when dumping fees back into initiator output, only dumb explicit + // fee var dustLimit btcutil.Amount if remoteChain { dustLimit = lc.channelState.TheirDustLimit