policy: document dust threshold for Taproot outputs #22863

pull darosior wants to merge 1 commits into bitcoin:master from darosior:dust_threshold_doc changing 1 files +6 −1

darosior commented at 11:44 am on September 2, 2021: member

Following discussions in #22779 .

policy: document we intentionally don't lower the dust threshold for Taproot

A "correction" of what seemed to be an overlook was initially proposed in
PR #22779. It was deemed unnecessary to further reduce the dust level,
so document the intention.

Signed-off-by: Antoine Poinsot <darosior@protonmail.com>

d873db7f8f

darosior commented at 11:45 am on September 2, 2021: member

(Happy to bikeshed on the comment)
fanquake added the label Docs on Sep 2, 2021
fanquake added the label TX fees and policy on Sep 2, 2021
TheBlueMatt approved

in src/policy/policy.cpp:38 in d873db7f8f

33@@ -34,6 +34,11 @@ CAmount GetDustThreshold(const CTxOut& txout, const CFeeRate& dustRelayFeeIn)
34     int witnessversion = 0;
35     std::vector<unsigned char> witnessprogram;
36 
37+    // Note this computation is for spending a Segwit v0 P2WPKH output (a 33 bytes
38+    // public key + an ECDSA signature). For Segwit v1 Taproot outputs the minimum

benthecarman commented at 8:07 pm on September 2, 2021:

nit: maybe say 74 byte DER encoded signature

benthecarman commented at 8:07 pm on September 2, 2021: contributor

ACK d873db7f8ff37c37f9c72482d8ecd52282f4438e
ariard commented at 10:25 pm on September 10, 2021: member

Code Review ACK d873db7
theStack approved
theStack commented at 4:23 pm on October 5, 2021: member

ACK d873db7f8ff37c37f9c72482d8ecd52282f4438e
laanwj merged this on Oct 15, 2021
laanwj closed this on Oct 15, 2021
sidhujag referenced this in commit d79b962852 on Oct 15, 2021
DrahtBot locked this on Oct 30, 2022

Contributors
darosior TheBlueMatt benthecarman ariard theStack

Labels
Docs TX fees and policy