HomePhabricator

Merge #13298: Net: Bucketing INV delays (1 bucket) for incoming connections to…

Description

Merge #13298: Net: Bucketing INV delays (1 bucket) for incoming connections to hide tx time

Summary:
d45b344ffd Bucket for inbound when scheduling invs to hide tx time (Gleb)

Pull request description:

It has been brought up to my attention that current random delays mechanism (originally intended to obfuscate transaction metadata) allows to easily estimate the time a transaction was received by a node.

It may be done by connecting multiple observer nodes to the same node. Each of those nodes will generate its own schedule of delays. Combined metadata regarding those events from different sources allows an observer to estimate transaction time.

After this patch a spy won't gain additional information by just creating multiple connections to a target.

Tree-SHA512: c71dae5ff350b614cb40a8e201fd0562d3e03e3e72a5099718cd451f0d84c66d5e52bbaf0d5b4b75137514c8efdedcc6ef4df90142b360153f04ad0721545ab1

Backport of Core PR13298
https://github.com/bitcoin/bitcoin/pull/13298/

Test Plan:

make check

Reviewers: deadalnix, Fabien, jasonbcox, O1 Bitcoin ABC, #bitcoin_abc

Reviewed By: jasonbcox, O1 Bitcoin ABC, #bitcoin_abc

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

Details

Provenance
Pieter Wuille <pieter.wuille@gmail.com>Authored on Jul 16 2018, 17:28
nakihitoCommitted on Dec 13 2019, 00:32
markblundebergPushed on Dec 13 2019, 04:35
Reviewer
Restricted Owners Package
Differential Revision
D4690: Merge #13298: Net: Bucketing INV delays (1 bucket) for incoming connections to hide tx time
Parents
rSTAGING2720d3db04d6: [wallet] Add wallet name to log messages
Branches
Unknown
Tags
Unknown