Allow disconnecting a netgroup with only one member in eviction.
With the latest additions there are enough protective measures that we can take the training wheels off.
This commit is contained in:
parent
5d0ca81f74
commit
6ee7f05622
1 changed files with 0 additions and 7 deletions
|
@ -965,13 +965,6 @@ static bool AttemptToEvictConnection() {
|
|||
// Reduce to the network group with the most connections
|
||||
vEvictionCandidates = std::move(mapAddrCounts[naMostConnections]);
|
||||
|
||||
// Do not disconnect peers if there is only one unprotected connection from their network group.
|
||||
// This step excessively favors netgroup diversity, and should be removed once more protective criteria are established.
|
||||
if (vEvictionCandidates.size() <= 1)
|
||||
// unless we prefer the new connection (for whitelisted peers)
|
||||
if (!fPreferNewConnection)
|
||||
return false;
|
||||
|
||||
// Disconnect from the network group with the most connections
|
||||
NodeId evicted = vEvictionCandidates.front().id;
|
||||
LOCK(cs_vNodes);
|
||||
|
|
Loading…
Reference in a new issue