Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: rendezvous unsubscribed peer is discoverable #1169

Open
AlejandroCabeza opened this issue Aug 7, 2024 · 0 comments
Open

bug: rendezvous unsubscribed peer is discoverable #1169

AlejandroCabeza opened this issue Aug 7, 2024 · 0 comments

Comments

@AlejandroCabeza
Copy link
Collaborator

AlejandroCabeza commented Aug 7, 2024

Problem

After some iterations of running advertise/unsubscribe on RendezVous, the unsubscribe doesn't make the peer non-discoverable.

To reproduce

  1. Instantiate some switches (peerA, peerB), both using RendezVous.
  2. Connect them.
  3. Advertise peerA in namespace N.
  4. From peerB, request a peer in namespace N.
  5. Repeat steps 3 and 4 until a peerA is discovered.
  • The first couple of times doing this, the request will fail as no peers will be found. After some iterations, peerA will be discoverable.

Alternatively, you can checkout the tests/gossipsub/peer-discovery branch, open tests/testdiscovery.nim and run the Frequent sub/unsub test case.

Expected behavior

After unsubscribing, the unsubscribed peer shouldn't be discoverable.

Versions

Nim: 2.0.8
PR: #1168

@AlejandroCabeza AlejandroCabeza changed the title bug: rendezvous doesn't unsubscribe bug: rendezvous unsubscribed peer is discoverable Aug 7, 2024
@kaiserd kaiserd moved this from In Progress to Pipeline in nim-libp2p Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Pipeline
Development

No branches or pull requests

1 participant