diff options
| author | fanquake <[email protected]> | 2020-09-30 12:17:06 +0800 |
|---|---|---|
| committer | fanquake <[email protected]> | 2020-09-30 12:41:22 +0800 |
| commit | c7ad94428ab6f54661d7a5441e1fdd0ebf034903 (patch) | |
| tree | 3655a0b2976800227edb15bcd81ab7d53d6d531d /src/script/script.cpp | |
| parent | Merge #20015: doc: Added default signet config for linearize script (diff) | |
| parent | Improve docs about feeler connections (diff) | |
| download | discoin-c7ad94428ab6f54661d7a5441e1fdd0ebf034903.tar.xz discoin-c7ad94428ab6f54661d7a5441e1fdd0ebf034903.zip | |
Merge #19958: doc: Better document features of feelers
2ea62cae483b764e30f61c06d8ac65755bbd864c Improve docs about feeler connections (Gleb Naumenko)
Pull request description:
"feeler" and "test-before-evict" are two different strategies suggest in [Eclipse Attacks on Bitcoin’s Peer-to-Peer Network](https://www.usenix.org/system/files/conference/usenixsecurity15/sec15-paper-heilman.pdf). In our codebase, we use `ConnType::FEELER` to implement both.
It is confusing, up to the point that our documentation was just incorrect.
This PR:
- ~clarifies this aspect by renaming "ConnType::FEELER" to "ConnType::PROBE", meaning that this connections only probes that the node is operational, and then disconnects.~
- fixes the documentation
ACKs for top commit:
amitiuttarwar:
ACK 2ea62cae48. thank you!
practicalswift:
ACK 2ea62cae483b764e30f61c06d8ac65755bbd864c
Tree-SHA512: c9c03c09eefeacec28ea199cc3f697b0a98723f2f849f7a8115edc43791f8165e296e0e25a82f0b5a4a781a7de38c8954b48bf74c714eba02cdc21f7460673e5
Diffstat (limited to 'src/script/script.cpp')
0 files changed, 0 insertions, 0 deletions