Policy: Penalize effective fee for sub-dust outputs#272
Open
kwsantiago wants to merge 4 commits intobitcoinknots:29.x-knotsfrom
Open
Policy: Penalize effective fee for sub-dust outputs#272kwsantiago wants to merge 4 commits intobitcoinknots:29.x-knotsfrom
kwsantiago wants to merge 4 commits intobitcoinknots:29.x-knotsfrom
Conversation
luke-jr
requested changes
Mar 5, 2026
Collaborator
luke-jr
left a comment
There was a problem hiding this comment.
How would this work, since dust is just rejected entirely still?
0d92d8a to
3ad0291
Compare
Author
|
The penalty applies to sub-dust outputs already tolerated by other policy settings (ephemeral dust, acceptnonstdtxn, etc.). Per #146, it targets tolerated sub-dust outputs rather than overriding the dust rejection. OP_RETURN is unaffected since GetDustThreshold returns 0 for unspendable outputs. |
rleed
suggested changes
Mar 19, 2026
luke-jr
reviewed
Mar 29, 2026
fb30f5c to
76c56cc
Compare
luke-jr
requested changes
Apr 11, 2026
Collaborator
luke-jr
left a comment
There was a problem hiding this comment.
Should "test: fix subdust fee penalty test for corepolicy min relay fee" be squashed?
Note for followup later: This doesn't allow for at least two useful configurations:
- Reject dust below fixed dust floor, but penalise based on smart dust logic.
- Allow all dust, but penalise based on dust configuration (fixed or smart).
76c56cc to
af7d6f6
Compare
Author
|
Squashed the test fix, moved SoftSetArg to the default-toggling commit, and removed |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #146
Reduces the effective fee by the dust threshold for each sub-dust output, creating an economic disincentive for dust creation without outright rejecting transactions.
-subdustfeepenaltyoption (default: enabled, disabled by-acceptlargefees)