-
Notifications
You must be signed in to change notification settings - Fork 451
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 only used for get_peers #109
Comments
Ill try to get an answer from the libtorrent developers if such patch would be accepted. |
We now require that we can manually do a DHT announce for hidden services, so we can't remove pymdht yet. |
@NielsZeilemaker @egbertbouman @whirm @synctext I guess this one can be closed. |
Why? Is the pymdht already removed? |
@NielsZeilemaker Not yet. |
@devos50 do you want to take care of this guy? |
Libtorrent has |
@ardhipoetra Interesting API handle: http://www.rasterbar.com/products/libtorrent/manual.html#torrent-handle Looks somewhat similar! pymDHT only returns non-validated IP addresses, Libtorrent tries a handshake with them. |
should we move this to an early milestone? |
An update: in version 1.1.0.0 (the most recent one), there are new function called The bad news is, python binding implementation is not perfect, if I may say. The discussion in arvidn/libtorrent#602 still ongoing till now. |
nice, let us know when it gets into a usable state! |
update needed op dht_get_peers status in libtorrent and Python wrapper. Few years past since issue was created. |
It seems that the |
@devos50 I think |
I guess this issue has been sufficiently addressed. |
If we can expose the DHT of LibTorrent we can remove our DHT implementation. Only a get_peers method is required.
After, we can redirect the miniswarm get_peers request and the DHT libswift node to use the libtorrent DHT instead.
The text was updated successfully, but these errors were encountered: