logos-storage/logos-storage-nim-dht
1
0
Fork 0
mirror of https://github.com/logos-storage/logos-storage-nim-dht.git synced 2026-03-30 07:43:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix addValue not enough nodes

Browse Source 
addValue cycle should consider number of nodes found.

Signed-off-by: Csaba Kiraly <csaba.kiraly@gmail.com>
This commit is contained in:
Csaba Kiraly 2023-09-12 11:23:24 +02:00
parent ec266ea478
commit 03c363e18d
No known key found for this signature in database
GPG Key ID: 0FE274EE8C95166E
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libp2pdht/private/eth/p2p/discoveryv5/protocol.nim
View File

@ -871,7 +871,9 @@ proc addValue*(
# TODO: lookup is specified as not returning local, even if that is the closest. Is this OK?
if res.len == 0:
res.add(d.localNode)
for toNode in res[0 ..< ValueReplication]:
if res.len < ValueReplication:
trace "addValue lookup did not return enough nodes", key=cId, len=res.len, ValueReplication
for toNode in res[0 ..< min(ValueReplication, res.len)]:
if toNode != d.localNode:
let reqId = RequestId.init(d.rng[])
d.sendRequest(toNode, AddValueMessage(cId: cId, value: value), reqId)