HomePhabricator

qa: Read reject reasons from debug log, not p2p messages

Description

qa: Read reject reasons from debug log, not p2p messages

Summary:
Partial backport of Core PR14119
https://github.com/bitcoin/bitcoin/pull/14119/files

Rationale for partially completing this out-of-order backport:

  • We get the benefits of this change today with minimal negative impact on future backporting efforts.
  • The original PR is a bit large and could do with splitting up the review. Only two additional lines of code are necessary to make this backport digestible in chunks (and then all TODOs can be removed once completed).
  • Improves robustness for D3863

Depends on D3860, D3861

Test Plan:

test_runner.py --extended

Reviewers: #bitcoin_abc, deadalnix, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

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

Details

Provenance
jasonbcoxAuthored on Aug 12 2019, 17:16
jasonbcoxPushed on Aug 14 2019, 16:51
Reviewer
Restricted Project
Differential Revision
D3862: qa: Read reject reasons from debug log, not p2p messages
Parents
rSTAGINGead9b1d0f49d: Merge #14101: qa: Use named args in validation acceptance tests
Branches
Unknown
Tags
Unknown