Process getheaders messages from whitelisted peers even if we are in initial block download. Whitelisted peers can always use a node as a block source.
Also log a debug message when the request is ignored, for troubleshooting.
Addresses #6971.
Always allow getheaders from whitelisted peers #6974
pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2015_11_whitelisted_allow_headers changing 1 files +3 −3-
laanwj commented at 1:33 PM on November 9, 2015: member
-
40b77d450d
Always allow getheaders from whitelisted peers
Process `getheaders` messages from whitelisted peers even if we are in initial block download. Whitelisted peers can always use a node as a block source. Also log a debug message when the request is ignored, for troubleshooting. Fixes #6971.
- laanwj added the label P2P on Nov 9, 2015
-
sdaftuar commented at 2:48 PM on November 9, 2015: member
utACK
-
dcousens commented at 10:13 PM on November 9, 2015: contributor
utACK
-
gmaxwell commented at 10:37 PM on November 9, 2015: contributor
I think this should also whitelist bypass maxuploadtarget-- for the same reason, no?
-
jonasschnelli commented at 8:15 AM on November 10, 2015: contributor
@gmaxwell: Agree with a pass for all whitelisted nodes
--maxuploadtarget. I think that should be addresses in another PR (which I will open soon). - laanwj merged this on Nov 11, 2015
- laanwj closed this on Nov 11, 2015
- laanwj referenced this in commit 2f71b07d5e on Nov 11, 2015
- luke-jr referenced this in commit 8766acdb0d on Nov 18, 2015
- rebroad cross-referenced this on Aug 26, 2016 from issue Provide bloom services to whitelisted nodes. by rebroad
- rebroad referenced this in commit 22a4a4fd5c on Nov 2, 2016
- rebroad cross-referenced this on Nov 2, 2016 from issue Ignore getheaders prior to passing all checkpoints. by rebroad
- rebroad referenced this in commit f5b82fe0bb on Nov 2, 2016
- rebroad referenced this in commit f032d43aa0 on Nov 2, 2016
- rebroad referenced this in commit 7f68d44c58 on Nov 2, 2016
- rebroad referenced this in commit 83d40a910f on Nov 2, 2016
- rebroad referenced this in commit cc61ff61ea on Nov 4, 2016
- rebroad referenced this in commit e58dcdd7ce on Nov 5, 2016
- rebroad referenced this in commit 6c0f1ba60f on Nov 5, 2016
- rebroad referenced this in commit 0ec255a812 on Nov 6, 2016
- rebroad referenced this in commit 65a43abf3a on Nov 6, 2016
- rebroad referenced this in commit ec7fac9a23 on Nov 6, 2016
- daira cross-referenced this on Mar 4, 2017 from issue Bitcoin 0.12 misc PRs 1 by str4d
- str4d cross-referenced this on Mar 4, 2017 from issue Improvements to getheaders handling by str4d
- str4d cross-referenced this on Jul 14, 2017 from issue Bitcoin 0.12 P2P/Net PRs 1 by str4d
- c0deright cross-referenced this on Aug 3, 2017 from issue 0.14.6 node not syncing blocks anymore -- Ignoring getheaders from peer=0 because node is in initial block download by c0deright
- MarcoFalke cross-referenced this on Jun 8, 2020 from issue net: Extract download permission from noban by MarcoFalke
- str4d cross-referenced this on Feb 18, 2021 from issue Bitcoin 0.12 misc P2P/Net PRs by str4d
- zkbot referenced this in commit e10008da66 on Feb 18, 2021
- zkbot referenced this in commit 777deea264 on Feb 19, 2021
- zkbot referenced this in commit b62e35dee8 on Feb 19, 2021
- bitcoin locked this on Sep 8, 2021
Contributors
Labels