Page MenuHomePhabricator

Merge #16152: Disable bloom filtering by default.
Changes PlannedPublic

Authored by jasonbcox on Jun 17 2020, 20:31.

Details

Reviewers
nakihito
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

bead32e31e399090af30b2ee3539995d4105a66d Add release notes for DEFAULT_BLOOM change (Matt Corallo)
f27309f55c4fa2b115525d72abb280757a568709 Move DEFAULT_PEERBLOOMFILTERS from validation.h to net_processing.h (Matt Corallo)
5efcb772838e404ca5757818d5548efcb872724b Disable bloom filtering by default. (Matt Corallo)

Pull request description:

BIP 37 bloom filters have been well-known to be a significant DoS
target for some time. However, in order to provide continuity for
SPV clients relying on it, the NODE_BLOOM service flag was added,
and left as a default, to ensure sufficient nodes exist with such a
flag.

NODE_BLOOM is, at this point, well-established and, as long as
there exist 0.18 nodes with default config (which I'd anticipate
will be true for many years), will be available from some peers. By
that time, the continued slowdown of BIP 37-based filtering will
likely have rendered it useless (though this is already largely the
case). Further, BIP 37 was deliberately never updated to support
witness-based filtering as newer wallets are expected to migrate to
some yet-to-be-network-exposed filters.

ACKs for top commit:

jnewbery:
  ACK bead32e31e399090af30b2ee3539995d4105a66d
kallewoof:
  ACK bead32e31e399090af30b2ee3539995d4105a66d

Tree-SHA512: ecd901898e8efe1a7c82b471af0acc2373c2282ac633eb58d9aae7c35deda1999d0f79fb0485e6cecbda7246aeda00206cd82c7fa36866e2ac64705ba93f9390

Backport of Core PR16152

Test Plan
ninja
ninja check-all

Event Timeline

Owners added a reviewer: Restricted Owners Package.Jun 17 2020, 20:31

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

jasonbcox requested changes to this revision.Jun 18 2020, 21:13
jasonbcox added a subscriber: jasonbcox.

I don't think we can make the same assumptions Core is making here. In ~5 months, no ABC nodes will be deployed with NODE_BLOOM unless explicitly set by the operator. This is likely to negatively impact SPV wallets, for which Bitcoin Cash has multiple.

This revision now requires changes to proceed.Jun 18 2020, 21:13
jasonbcox edited reviewers, added: nakihito; removed: jasonbcox.