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

IPNS over PubSub as an Independent Transport #6758

Merged
merged 1 commit into from
Dec 13, 2019

Conversation

aschmahmann
Copy link
Contributor

Utilizes the persistent pubsub value store from libp2p along with the new pubsub support for topic-based peer discovery.

Blocked by libp2p/go-libp2p-pubsub-router#37

@aschmahmann aschmahmann self-assigned this Nov 11, 2019
@Stebalien Stebalien added the status/blocked Unable to be worked further until needs are met label Dec 2, 2019
@aschmahmann aschmahmann force-pushed the feat/ipns-persistent-pubsub branch 2 times, most recently from 0633640 to d3949aa Compare December 9, 2019 16:32
@aschmahmann
Copy link
Contributor Author

This is also blocked by #6785 since go-libp2p-pubsub-router was updated to use the latest datastore interface.

@Stebalien Stebalien removed the status/blocked Unable to be worked further until needs are met label Dec 11, 2019
@Stebalien
Copy link
Member

@aschmahmann could you rebase this PR on master (and release the new pubsub router)? I'd like to not block this on the async datastore refactor (IIRC, this is mergable as-is).

@aschmahmann aschmahmann marked this pull request as ready for review December 11, 2019 17:36
if bcfg.getOpt("pubsub") || bcfg.getOpt("ipnsps") {
disc = fx.Provide(libp2p.TopicDiscovery())
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: we can actually just register this regardless.

@Stebalien Stebalien merged commit 1b7f3a8 into master Dec 13, 2019
@Stebalien
Copy link
Member

🕺

@Stebalien Stebalien deleted the feat/ipns-persistent-pubsub branch December 13, 2019 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants