Normally feefilter messages are sent only based on the mempool min fee which only gets set once your mempool is limited. But if you are not allowing free relay, you'll always require at least minRelayTxFee anyway, so this PR sends feefilter messages taking that into account.
If we don't allow free txs, always send a fee filter #9313
pull morcos wants to merge 1 commits into bitcoin:master from morcos:minminfee changing 1 files +3 −0-
morcos commented at 8:51 PM on December 9, 2016: member
-
If we don't allow free txs, always send a fee filter 01fea7a048
-
luke-jr commented at 9:49 PM on December 9, 2016: member
utACK
-
MarcoFalke commented at 11:05 PM on December 9, 2016: member
Concept ACK 01fea7a04834b7fd5eca6f3678f56fed13375d1d
- MarcoFalke added the label Resource usage on Dec 9, 2016
- MarcoFalke added the label P2P on Dec 9, 2016
- MarcoFalke added the label TX fees and policy on Dec 9, 2016
- dcousens approved
-
gmaxwell commented at 12:01 AM on December 10, 2016: contributor
utACK utACK utACK.
-
Mirobit commented at 12:09 AM on December 10, 2016: contributor
ACK 01fea7a04834b7fd5eca6f3678f56fed13375d1d
2016-12-09 23:46:38 send: feefilter of 0.00010556 BTC/kB to peer=3 2016-12-09 23:46:40 send: feefilter of 0.00010556 BTC/kB to peer=4 2016-12-09 23:46:43 send: feefilter of 0.00009597 BTC/kB to peer=6 2016-12-09 23:46:46 send: feefilter of 0.00009597 BTC/kB to peer=7
-
sipa commented at 12:15 AM on December 10, 2016: member
utACK 01fea7a04834b7fd5eca6f3678f56fed13375d1d
-
paveljanik commented at 9:11 AM on December 10, 2016: contributor
ACK 01fea7a
- MarcoFalke added this to the milestone 0.14.0 on Dec 10, 2016
-
gmaxwell commented at 6:17 PM on December 15, 2016: contributor
ACK
- laanwj merged this on Dec 15, 2016
- laanwj closed this on Dec 15, 2016
- laanwj referenced this in commit 756374c522 on Dec 15, 2016
- DrahtBot locked this on Sep 8, 2021
Milestone
0.14.0