HomePhabricator

Merge #14054: p2p: Disable BIP 61 by default

Description

Merge #14054: p2p: Disable BIP 61 by default

Summary:
faea5bfc5a975874acf763082852ed532ed81a95 doc: release notes for -enablebip61 default change (MarcoFalke)
fa14b54a872ef0ff755e3c1c0775904ca33cb5ff p2p: Disable BIP 61 by default (MarcoFalke)

Pull request description:

The live p2p network should not be used for debugging or as development aid when implementing the p2p protocol. Instead, applications should be tested locally (e.g. by inspecting the debug log of a validating node on the local network)

Using the p2p network for this purpose seems wasteful and even dangerous, as peers can not be trusted to send the correct reject messages or a reject message at all.

Tree-SHA512: 9c91ad035b5110942172a3b4b8a332a84e0c0aa9ee80f8134aeab63e66ac604841e68b04038681c288b716e5e0cbe38065eb5c7eb63fa72c3bdb3255a4b2d99d

Backport of Core PR14054

Test Plan:

ninja
ninja check
ninja check-functional

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D6455

Details

Provenance
Wladimir J. van der Laan <laanwj@gmail.com>Authored on Sep 10 2018, 15:57
nakihitoCommitted on Jun 9 2020, 18:07
nakihitoPushed on Jun 9 2020, 18:07
Reviewer
Restricted Owners Package
Differential Revision
D6455: Merge #14054: p2p: Disable BIP 61 by default
Parents
rABCb160ed70b766: [backport#14060] ZMQ: add options to configure outbound message high water markā€¦
Branches
Unknown
Tags
Unknown