feat: add zikade DHT router#10154
Conversation
| const ( | ||
| RouterTypeHTTP RouterType = "http" // HTTP JSON API for delegated routing systems (IPIP-337). | ||
| RouterTypeDHT RouterType = "dht" // DHT router. | ||
| RouterTypeDHTv2 RouterType = "dhtv2" // DHT router using go-libp2p-kad-dht/v2. |
There was a problem hiding this comment.
I don't think this name is correct, this router is compatible with v1.
So I don't think v2 applies here.
There was a problem hiding this comment.
It will allow different custom parameters, which I haven't added yet.
There was a problem hiding this comment.
I mean network wise, it's just a different implementation, it's it will talk with other v1 dht servers and clients.
I would name it go-kademlia
The way I understand v2 here is that if I turn this I'm not part of the amino dht anymore.
There was a problem hiding this comment.
I see how this name could cause confusion between go-libp2p-kad-dht version 2 and version 2 of the dht protocols. I'll choose a better name - same for boxo
There was a problem hiding this comment.
Renamed router type to dht/zikade
|
Resubmit when/if ipfs/boxo#472 is merged |
Adds a
dht/zikaderouting type which corresponds to the refactored zikade DHT currently under development (previously go-libp2p-kad-dht/v2).Requires ipfs/boxo#472
This is intended as a future experimental routing method for Kubo.
Can be tested with configuration like this: