Feross Aboukhadijeh
Feross Aboukhadijeh
@mafintosh Do you have any thoughts on this PR? It's been a while and I think we should merge or close at this point. I'm not familiar with the spec...
@ssiloti Thanks for opening this issue. Agreed, we should fix this. Does anyone want to take a stab at sending a PR for this?
@Aaron1011 Nice, @mafintosh and I would be happy to merge your work in :)
@mafintosh What are your thoughts on getting in IPv6 support? @Aaron1011 offered to revive his PRs. I think together we can tackle reviewing them once he's updated them. Thoughts?
I haven't had time to dig into this yet. If other WebTorrent contributors or @mafintosh want to chime in and share whether the existing PRs look like a reasonable approach,...
https://github.com/Ayms/bittorrent-nodeid looks like a good start!
The third option seems the most usable to me, and it doesn't require new APIs to be added, or additional state to be tracked inside the DHT client. I like...
@lmatteis I still slightly prefer `{ force: true }` since that way we do the right thing by default. We can bump the major version. But, I'm fine with `{...
Cool -- thanks for giving it a shot. I'm probably not the best person to review this, tbh as I didn't write this part of the codebase. I think @mafintosh...
@lmatteis Re-announce should happen at the app level (option 3) to remain consistent with .announce/.lookup.