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

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 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 0:01 am on December 10, 2016: contributor

utACK utACK utACK.
Mirobit commented at 0: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 0: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

Contributors
morcos luke-jr MarcoFalke dcousens gmaxwell Mirobit sipa paveljanik

Labels
TX fees and policy P2P Resource usage

Milestone
0.14.0