Skip to content

Comments

Add details for post-anchor dust_limit_satoshis#1301

Open
t-bast wants to merge 2 commits intolightning:masterfrom
t-bast:dust-limit-recommandations
Open

Add details for post-anchor dust_limit_satoshis#1301
t-bast wants to merge 2 commits intolightning:masterfrom
t-bast:dust-limit-recommandations

Conversation

@t-bast
Copy link
Collaborator

@t-bast t-bast commented Nov 19, 2025

Since the introduction of option_anchors, pre-signed HTLC transactions don't include any fees, and thus don't contribute to trimming. This results in HTLC outputs in the commitment transactions that may be forever uneconomical to spend, and will pollute the utxo set.

We should use a dust_limit_satoshis that is higher than Bitcoin Core's standard values to take into account the cost of these second-stage txs at a reasonable (TM) feerate. If we want for example to be able to claim HTLC outputs at 20 sat/byte, we need a dust_limit_satoshis of around 4 000 sats.

Since the introduction of `option_anchors`, pre-signed HTLC transactions
don't include any fees, and thus don't contribute to trimming. This
results in HTLC outputs in the commitment transactions that may be
forever uneconomical to spend, and will pollute the utxo set.

We should use a `dust_limit_satoshis` that is higher than Bitcoin Core's
standard values to take into account the cost of these second-stage txs
at a reasonable (TM) feerate.
Copy link
Contributor

@ziggie1984 ziggie1984 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK, had some minor comments.

Copy link
Collaborator

@Roasbeef Roasbeef left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 📸

@t-bast
Copy link
Collaborator Author

t-bast commented Feb 9, 2026

@TheBlueMatt @rustyrussell can I go ahead and merge this?

@t-bast
Copy link
Collaborator Author

t-bast commented Feb 24, 2026

Gentle ping @TheBlueMatt @rustyrussell 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants