fix: replace limit by peerLimit

vacp2p · Oct 3, 2024 · f6d1530 · f6d1530
1 parent 4f3df47
commit f6d1530
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/libp2p/protocols/rendezvous.nim b/libp2p/protocols/rendezvous.nim
@@ -558,14 +558,15 @@ proc requestLocally*(rdv: RendezVous, ns: string): seq[PeerRecord] =
     @[]
 
 proc request*(
-    rdv: RendezVous, ns: string, limit: uint64 = DiscoverLimit, peers: seq[PeerId]
+    rdv: RendezVous, ns: string, peerLimit: uint64 = DiscoverLimit, peers: seq[PeerId]
 ): Future[seq[PeerRecord]] {.async.} =
   ## This async procedure discovers and returns peers for a given namespace
   ## by sending requests and processing responses. It limits the number of
   ## peer records retrieved based on the provided limit.
   ##
   var
     s: Table[PeerId, (PeerRecord, Register)]
+    limit = peerLimit
     d = Discover(ns: ns)
 
   if limit > DiscoverLimit:

diff --git a/tests/testrendezvous.nim b/tests/testrendezvous.nim
@@ -130,8 +130,6 @@ suite "RendezVous":
       switch = createSwitch(rdv)
     expect RendezVousError:
       discard await rdv.request("A".repeat(300))
-    expect RendezVousError:
-      discard await rdv.request("A", -1)
     expect RendezVousError:
       discard await rdv.request("A", 3000)
     expect RendezVousError: