vechain / thor

A general purpose blockchain highly compatible with Ethereum's ecosystem
GNU Lesser General Public License v3.0
796 stars 247 forks source link

feat: do not connect to v1 peers #767

Closed darrenvechain closed 4 months ago

darrenvechain commented 4 months ago

Description

Prevent connecting to v1 nodes

TODO: Test peer cache

Type of change

How Has This Been Tested?

:point_up: 1 good node, 1 old node

Checklist:

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 37.50000% with 10 lines in your changes missing coverage. Please review.

Project coverage is 62.68%. Comparing base (43d1fde) to head (c297567).

Files Patch % Lines
p2psrv/server.go 37.50% 10 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #767 +/- ## ========================================== + Coverage 62.57% 62.68% +0.11% ========================================== Files 199 199 Lines 18098 18113 +15 ========================================== + Hits 11324 11355 +31 + Misses 5691 5673 -18 - Partials 1083 1085 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.