Couldn't find a test for this and it seems something we should test, so I wrote one.
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
-
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: contributor
ACK faa94cb16
-
DrahtBot commented at 7:37 PM on September 28, 2020: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
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.
- DrahtBot cross-referenced this on Sep 28, 2020 from issue Complete the BIP155 implementation and upgrade to TORv3 by vasild
- DrahtBot cross-referenced this on Sep 29, 2020 from issue Implement ADDRv2 support (part of BIP155) by vasild
-
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
- bitcoin locked this on Feb 15, 2022
Contributors
Labels