Increase dust threshold multiplier #2368
Conversation
|
Modifying the existing value isn't reasonable as many higher level protocols such as LN rely on the ability to propagate transactions at the current dust level. If you were running an LN node against your node, and ran this patch, you'd find that you weren't able to force close at all. If you can't force close, then that means your funds my be lost in a breach event. I think the only thing that would make sense here is adding a config option to tweak this value. The public functions should be modified though, as callers expect stability from the return value of those functions (it only due to the witness scaling factor). As you as a user can end up with dust in your wallet even if you're only doing "monetary" transactions. If you have a small amount of change left over after a send, then that can be considered dust. |
I have a nonrouting node on top of Bitcoin Knots. Bitcoin Core and Knots they have |
Sure, want to update this PR to add a config option like that? As mentioned, all the other public functions in the sub-packages should remain unchanged. Also it's important that you understand this interaction:
If you set this value (once you add it to the code) to basically anything other than the current default, then you will willingly expose yourself to events that may cause irrevocable loss of funds on LN. |
Change Description
The purpose of this PR is to filter out more dust / nonmonetary transactions. The dust would be defined for outputs for which the cost to the network to spend coins is more than 1/9 of the minimum transaction relay fee.
P.S. I inadvertently deleted the checklist. Apologies for that. It would be great if someone could restore it.