HomePhabricator

Merge #14571: [tests] Test that nodes respond to getdata with notfound

Description

Merge #14571: [tests] Test that nodes respond to getdata with notfound

Summary:
PR14571 backport https://github.com/bitcoin/bitcoin/pull/14571/files
fa78a2fc67 [tests] Test that nodes respond to getdata with notfound (MarcoFalke)

Pull request description:

If a node has not announced a tx at all, then it should respond to
getdata messages for that tx with notfound, to avoid leaking tx
origination privacy.

In the future this could be adjusted such that a node responds with
notfound when a tx has not been announced to us, but that seems
to be a more involved change. See e.g.
https://github.com/jnewbery/bitcoin/commits/pr14220.1

Backport note: this was backported a bit early, but it runs nicely with
two extra lines included (as marked in comment).

Test Plan: test_runner.py

Reviewers: jasonbcox, Fabien, deadalnix, #bitcoin_abc

Reviewed By: deadalnix, #bitcoin_abc

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

Details

Provenance
MarcoFalke <falke.marco@gmail.com>Authored on Oct 27 2018, 11:17
Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>Committed on Jul 9 2019, 22:04
nakihitoPushed on Jul 9 2019, 22:40
Reviewer
Restricted Project
Differential Revision
D3614: Merge #14571: [tests] Test that nodes respond to getdata with notfound
Parents
rSTAGINGc9b857a42499: Move CheckBlock() call to critical section
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/10165, tag: phabricator/base/10164