HomePhabricator

Merge #11638: [tests] Dead mininode code

Description

Merge #11638: [tests] Dead mininode code

Summary:
Backport of Core PR 11638
Completes T467

Note: Some changes in the original PR will appear to be missing or
different due to changes in 35638ea8b and ce143f3048.

Original PR description:

fb00c45c3 [tests] Explicitly disallow support for p2p versions below 60001 (John Newbery)
3858aabbd [tests] Remove support for p2p alert messages (John Newbery)
c0b127470 [tests] Remove support for bre-BIP31 ping messages (John Newbery)
2904e301c [tests] Remove dead code from mininode.py (John Newbery)

Pull request description:

  This is the first part of #11518. It removes a ~150 lines of unused code from the mininode module:

  - remove unused `deliver_sleep_time` and `EarlyDisconnectError` code
  - remove support for pre-BIP31 ping messages
  - remove support for alert message
  - explicitly don't support p2p versions lower than 60001

  Should be an easy ACK for reviewers. If all extended tests pass, then this code really was dead :)

Tree-SHA512: 508e612ceb0b094250d18e75522d51e6b14cd069443050ba4af34d6f890c58721cb5653e8bc000b60635b9474d035b0dcd9c509c0dcdb3a7501df17b787f83b0

Test Plan: test_runner.py

Reviewers: deadalnix, Fabien, #bitcoin_abc

Reviewed By: deadalnix, #bitcoin_abc

Subscribers: teamcity, schancel

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

Details

Provenance
MarcoFalke <falke.marco@gmail.com>Authored on Nov 9 2017, 18:48
jasonbcoxCommitted on Dec 21 2018, 21:59
jasonbcoxPushed on Dec 21 2018, 22:19
Reviewer
Restricted Project
Differential Revision
D2210: Merge #11638: [tests] Dead mininode code
Parents
rSTAGING5801452e54b1: Add an algorithm identifier to the nonce generation for ECDSA signatures
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/6404