Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

[WIP] feat: Add rendezvous #1396

Closed
wants to merge 1 commit into from
Closed

Conversation

mkg20001
Copy link
Contributor

Adds support for libp2p-rendezvous protocol

ToDOs:

  • Make this less hacky
  • Hide it behind a flag for now?

@pgte
Copy link
Contributor

pgte commented Jul 17, 2018

@mkg20001 @alanshaw Any news on this? We need this for peer-base/peer-base#5

@alanshaw
Copy link
Member

alanshaw commented Jul 17, 2018

@pgte in lieu of it being a default discovery module can you pass it as libp2p.modules.peerDiscovery option to IPFS constructor temporarily in peer-star-app?

@mkg20001
Copy link
Contributor Author

@pgte This would have to be properly integrated into libp2p instead of IPFS, I think. Current blocker is that rendezvous requires swarm, and swarm should depend on rendezvous as a discovery module. (chicken&egg problem). There needs to be a better solution to that than just adding more hacks.

@alanshaw
Copy link
Member

@mkg20001 shall we close this PR for the time being?

@mkg20001
Copy link
Contributor Author

@alanshaw Yes. I think the whole "X requires swarm but is component for swarm" situation requires a refactor anyways

@mkg20001 mkg20001 closed this Jan 21, 2019
@ghost ghost removed the status/ready Ready to be worked label Jan 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants