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

BitTorrent v2 Support #88

Open
leoherzog opened this issue Nov 15, 2020 · 11 comments
Open

BitTorrent v2 Support #88

leoherzog opened this issue Nov 15, 2020 · 11 comments

Comments

@leoherzog
Copy link
Contributor

What version of this package are you using?
9.1.0

What problem do you want to solve?
I want to be able to detect the SHA-256 v2 hash of a v2 Torrent that is parsed with parse-torrent. This also means detecting btmh in addition to btih in Magnet links.

What do you think is the correct solution to this problem?
Adding a infoHashv2 key (or similar) to the resulting object that is generated by parse-torrent

Are you willing to submit a pull request to implement this change?
I will try and learn how parse-torrent works and see what I can do

@leoherzog
Copy link
Contributor Author

It looks like we'd need to update magnet-uri upstream, as well as find an equivalent to simple-sha1 for SHA256?

@leoherzog
Copy link
Contributor Author

Of course, @feross already beat us to it 😆

https://github.com/feross/simple-sha256

@feross
Copy link
Member

feross commented Nov 17, 2020

We'd accept a PR that added support for this. @leoherzog Are you interested in sending a PR? I just noticed that there's a PR for this already sent to magnet-uri, though it needs some work: webtorrent/magnet-uri#43

@leoherzog
Copy link
Contributor Author

@feross What things need work still in webtorrent/magnet-uri#43 from your perspective?

@leoherzog
Copy link
Contributor Author

webtorrent/magnet-uri#43 has been merged, and magnet-uri correctly passes in an infoHashv2 key to parse-torrent. The remaining scope for this issue (and future pull request) would be to add support for Torrent files, as Magnet URLs are now covered.

@B1773rm4n
Copy link

How is the status here?

@mepanko91
Copy link

v2 .torrent file support added?

@leoherzog
Copy link
Contributor Author

I haven't had much time to look at or improve this, but feel free to contribute to the pull request

#91

@mepanko91
Copy link

I haven't had much time to look at or improve this, but feel free to contribute to the pull request

#91

Ok. Cool

@milahu
Copy link

milahu commented Aug 30, 2023

parse-torrent 11.0.14 throws an error on v2-only torrents

Error: Torrent is missing required field: info.pieces

edit:

related: webtorrent/node-bencode#94

@leoherzog
Copy link
Contributor Author

leoherzog commented Aug 30, 2023

When this project was converted to an ESM package (https://github.com/webtorrent/parse-torrent/releases/tag/v10.0.0), my pull request #91 became pretty irrelevant. My work here would have to be thrown out and rewritten to add compatibility with v2 torrents and I don't have time to dive in and learn what would need to be done... but if someone's willing to take that on, that would be nice. Don't expect to hear much from the maintainers though. It was awaiting review for over a year and never heard anything regarding feedback or pull request status, then they moved forward with converting to ESM instead.

#91 (comment)

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

5 participants