network icon indicating copy to clipboard operation
network copied to clipboard

fix(dht): Do not limit neighbor list in `PeerManager#onKBucketPing`

Open teogeb opened this issue 1 year ago • 0 comments

When we sort the oldContacts, we should sort all contacts, not just a subset? In practice the oldContacts array length is already limited by the numberOfNodesPerKBucket (see PeerManager.ts:82), so there is no need to re-limit it in the PeerManager#onKBucketPing function?

teogeb avatar Apr 22 '24 14:04 teogeb