test: Check that invalid peer traffic is accounted for #20028

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2009-testInvalidTraffic changing 1 files +10 −7

MarcoFalke commented at 8:16 AM on September 28, 2020: member

Couldn't find a test for this and it seems something we should test, so I wrote one.
test: Remove confusing cast to same type (int to int) fae243f0cb
test: Check that invalid peer traffic is accounted for faa94cb167
fanquake added the label Tests on Sep 28, 2020
vasild approved
vasild commented at 8:53 AM on September 28, 2020: member

ACK faa94cb16
DrahtBot commented at 7:37 PM on September 28, 2020: member


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #19954 (tor: make a TORv3 hidden service instead of TORv2 by vasild)
- #19031 (Implement ADDRv2 support (part of BIP155) by vasild)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
practicalswift commented at 6:06 AM on September 29, 2020: contributor

ACK faa94cb1675d8bd511eb593176cd07aa59465225: patch looks correct
fanquake merged this on Sep 29, 2020
fanquake closed this on Sep 29, 2020
MarcoFalke deleted the branch on Sep 29, 2020
sidhujag referenced this in commit ba4123d6e2 on Sep 29, 2020
Fabcien referenced this in commit 8932671382 on Oct 15, 2021
DrahtBot locked this on Feb 15, 2022

Contributors

Labels