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

Torrent file parsing error #65

Closed
jngbng opened this issue Feb 21, 2016 · 6 comments
Closed

Torrent file parsing error #65

jngbng opened this issue Feb 21, 2016 · 6 comments

Comments

@jngbng
Copy link

jngbng commented Feb 21, 2016

This package sometimes fails to parse a torrent file.
The problem turns out to be a type mismatch error:
"Nodes" field of "MetaInfo" struct is defined as [][]string type,
expecting both host and port are of type string.
However, torrent spec defines port value as of integer type.

http://bittorrent.org/beps/bep_0005.html

@anacrolix
Copy link
Owner

Can you provide a sample torrent file?

@jngbng
Copy link
Author

jngbng commented Feb 22, 2016

Here are torrent files that have "nodes" field.

2016-02-22 10:20 GMT+09:00 Matt Joiner notifications@github.com:

Can you provide a sample torrent file?


Reply to this email directly or view it on GitHub
#65 (comment).

@anacrolix
Copy link
Owner

@jngbng There's no attachment. I don't think you can do that on github. Can you provide a link?

@jngbng
Copy link
Author

jngbng commented Feb 22, 2016

Sorry, I hope you may access the file via this link:
http://211.41.234.53:18888/not_working_torrent.zip

2016-02-22 23:41 GMT+09:00 Matt Joiner notifications@github.com:

@jngbng https://github.com/jngbng There's no attachment. I don't think
you can do that on github. Can you provide a link?


Reply to this email directly or view it on GitHub
#65 (comment).

@anacrolix
Copy link
Owner

Thanks. I'll ensure it's parsed correctly, and that the nodes are incorporated into the DHT with commits in the next few days.

@jngbng
Copy link
Author

jngbng commented Feb 22, 2016

Thanks a lot!
2016. 2. 22. 오후 11:54에 "Matt Joiner" notifications@github.com님이 작성:

Thanks. I'll ensure it's parsed correctly, and that the nodes are
incorporated into the DHT with commits in the next few days.


Reply to this email directly or view it on GitHub
#65 (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

2 participants