diff options
author | fanquake <fanquake@gmail.com> | 2020-05-12 08:34:46 +0800 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2020-05-12 09:13:48 +0800 |
commit | 7a5767423f3757e876fbcb0f4ffb534172dd734d (patch) | |
tree | 4c5ea2c1e439aeabb4add0464cc5928ea5468b4c /doc/zmq.md | |
parent | eb2ffbb7c1347115e6a3d6fa30f909959b6170a5 (diff) | |
parent | 9847e205bf7edcac4c30ce4b6d62f482aa7bc1b7 (diff) | |
download | bitcoin-7a5767423f3757e876fbcb0f4ffb534172dd734d.tar.xz |
Merge #18808: [net processing] Drop unknown types in getdata
9847e205bf7edcac4c30ce4b6d62f482aa7bc1b7 [docs] Improve commenting in ProcessGetData() (John Newbery)
2f032556e08a04807c71eb02104ca9589eaadf1b [test] test that an invalid GETDATA doesn't prevent processing of future messages (Amiti Uttarwar)
e257cf71c851e25e1a533bf1d4296f6b55c81332 [net processing] ignore unknown INV types in GETDATA messages (Amiti Uttarwar)
047ceac142246b5d51056a51dbf4645b31802be4 [net processing] ignore tx GETDATA from blocks-only peers (Amiti Uttarwar)
Pull request description:
Currently we'll stall peers that send us an unknown INV type in a GETDATA message. Be a bit more friendly and just drop the invalid request.
Ditto for blocks-relay-only peers that send us a GETDATA for a transaction.
There's a test for the first part. The second is difficult to test in the functional test framework since we aren't able to make blocks-relay-only connections.
ACKs for top commit:
sipa:
utACK 9847e205bf7edcac4c30ce4b6d62f482aa7bc1b7
brakmic:
ACK https://github.com/bitcoin/bitcoin/commit/9847e205bf7edcac4c30ce4b6d62f482aa7bc1b7
luke-jr:
utACK 9847e205bf7edcac4c30ce4b6d62f482aa7bc1b7
naumenkogs:
utACK 9847e20
ajtowns:
utACK 9847e205bf7edcac4c30ce4b6d62f482aa7bc1b7
Tree-SHA512: 6007f2fd839ffe737727f6fb8e8f083b2d9e05a510748f1d40b8f9be8fdf7b5419a36d8f1039923eec1ba2983e8f6f0436ec5fc196d9f6dcb0657f2ff8ff8e4c
Diffstat (limited to 'doc/zmq.md')
0 files changed, 0 insertions, 0 deletions