diff options
| author | Matt Corallo <git@bluematt.me> | 2019-06-05 11:02:35 -0400 |
|---|---|---|
| committer | Matt Corallo <git@bluematt.me> | 2019-06-06 10:01:13 -0400 |
| commit | 5efcb772838e404ca5757818d5548efcb872724b (patch) | |
| tree | 1879493ddc8b330b0fbc0373cd0ee89899375a57 /test/functional/test_framework | |
| parent | 52ec4c64e89f478cd2134dbf25f5987d39e9b8bc (diff) | |
| download | bitcoin-5efcb772838e404ca5757818d5548efcb872724b.tar.xz | |
Disable bloom filtering by default.
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.
Diffstat (limited to 'test/functional/test_framework')
| -rwxr-xr-x | test/functional/test_framework/messages.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/functional/test_framework/messages.py b/test/functional/test_framework/messages.py index 954ae3c4df..2ab9c37576 100755 --- a/test/functional/test_framework/messages.py +++ b/test/functional/test_framework/messages.py @@ -44,7 +44,7 @@ BIP125_SEQUENCE_NUMBER = 0xfffffffd # Sequence number that is BIP 125 opt-in an NODE_NETWORK = (1 << 0) # NODE_GETUTXO = (1 << 1) -NODE_BLOOM = (1 << 2) +# NODE_BLOOM = (1 << 2) NODE_WITNESS = (1 << 3) NODE_NETWORK_LIMITED = (1 << 10) |