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

Add implicit HTTP delegated router for cid.contact network indexer #9454

Closed
Tracked by #9417
BigLep opened this issue Dec 5, 2022 · 1 comment · Fixed by #9475
Closed
Tracked by #9417

Add implicit HTTP delegated router for cid.contact network indexer #9454

BigLep opened this issue Dec 5, 2022 · 1 comment · Fixed by #9475

Comments

@BigLep
Copy link
Contributor

BigLep commented Dec 5, 2022

Done Criteria

Kubo has an implicit HTTP delegated router set for cid.contact.

There is corresponding information in:

  1. Changelog
  2. Config docs

Why Important

  1. Make more content discoverable for Kubo nodes. The cid.contact InterPlanetary Network Indexer has providers that aren't publishing to the DHT like web3.storage, Filecoin Storage Providers, etc.
  2. Has the potential to provide faster lookup times.

Notes

  1. Generally: there has been a lot written and recorded about network indexers. I'm not going to recap it all here. I welcome anyone to paste in the best resources on this.
  2. Like Kubo's DNS handling of .eth domains, this will be an implicit default documented in config. Users can override/disable, and if they don't, they'll get whatever implicit defaults we make in the future (e.g., adding additional network indexers)
  3. This initial integration between Kubo will not be doing anything special from a privacy regard. For example, efforts to handle double hashing (doc 1, doc 2) will come in followup work. Reader privacy preserving updates to Kubo are tracked in Communicate with network indexers for content routing using reader privacy with double hashing #9455
@BigLep
Copy link
Contributor Author

BigLep commented Dec 7, 2022

This is being handled in #9475

lidel added a commit that referenced this issue Dec 8, 2022
This changes the default routing to use both DHT and IPNI
at the same time. Closes #9454 Closes #9422

Full context:
#9454
#9422

Co-authored-by: Steve Loeppky <biglep@protocol.ai>
Co-authored-by: Gus Eggert <gus@gus.dev>
Co-authored-by: Steve Loeppky <biglep@protocol.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant