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

DHT should not depend on boxo #846

Open
Wondertan opened this issue Jun 12, 2023 · 5 comments
Open

DHT should not depend on boxo #846

Wondertan opened this issue Jun 12, 2023 · 5 comments

Comments

@Wondertan
Copy link

Even though boxo/ipfs is the primary user of DHT, it does not mean that DHT should depend on it. It's used only for utility functions and the ipns validator. This basic usage does not justify importing the full-blown boxo for the tradeoff of conceptual cyclicity and go.mod bloat.

@Wondertan
Copy link
Author

Wondertan commented Jun 12, 2023

Also, my motivation here comes from seeing the recent versions and the nice improvements to DHT they bring, but the failure to update the project due to DHT's dependence on boxo. It requires fully migrating to it, which is PITA, especially because of ipfs/boxo#281.

@p-shahi
Copy link
Member

p-shahi commented Jun 13, 2023

cc @Jorropo @guillaumemichel

@guillaumemichel
Copy link
Contributor

@Wondertan agreed that kad-dht shouldn't depend on boxo. The (longer term) goal is to split the DHT implementation (currently go-libp2p-kad-dht), in 2 different repos. One generic dht implementation living in the libp2p org, and the IPFS instantiation of the DHT living in the ipfs org (maybe even within boxo).
There is an ongoing effort for a larger refactor and split of the repo, but no ETA yet.

@Wondertan
Copy link
Author

Do you mean ComposableDHT?

@guillaumemichel
Copy link
Contributor

It should be the case with the ComposableDHT, but we already want to split the repo before during a repo refactor.

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

No branches or pull requests

3 participants