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

Error for all authkey tracker #171

Open
bingyu50 opened this issue Sep 10, 2022 · 11 comments
Open

Error for all authkey tracker #171

bingyu50 opened this issue Sep 10, 2022 · 11 comments

Comments

@bingyu50
Copy link

got below error for this kind of tracker: https://aaa.bbb.cccc/announce.php?authkey=9207|885|AJ2uIk80]

"TORRENT WAS REJECTED BY SERVER: JAVA.NET.URISYNTAXEXCEPTION: ILLEGAL CHARACTER IN QUERY AT INDEX 48: https://aaa.bbb.cccc/announce.php?authkey=9207|885|AJ2uIk80]"

@bingyu50
Copy link
Author

image

@laur89
Copy link
Contributor

laur89 commented Sep 10, 2022

@anthonyraymond is this likely thrown from com.turn.ttorrent.common.Torrent, possibly line #176 (URI uri = new URI(tracker.getString());)? Sounds like they should encode the uri before feeding it to URI constructor.

@google2013
Copy link

@anthonyraymond is this likely thrown from com.turn.ttorrent.common.Torrent, possibly line #176 (URI uri = new URI(tracker.getString());)? Sounds like they should encode the uri before feeding it to URI constructor.

Is there any pull requests? Thank you very much !

@laur89
Copy link
Contributor

laur89 commented Nov 15, 2022

No PRs. What needs be done:
a) verify it's solved in turn.ttorrent project's upstream; if not, fix it there first;
b) upgrade joal to use latest version of ttorrent (currently used version is quite a bit behind)

I might take it on, but only if/when #172 is merged.

@cwyalpha
Copy link

Caused by: java.net.URISyntaxException: Illegal character in query at index 53: https://xxx/announce.php?authkey=12345|67890|abcxyz
at java.base/java.net.URI$Parser.fail(URI.java:2973)
at java.base/java.net.URI$Parser.checkChars(URI.java:3144)
at java.base/java.net.URI$Parser.parseHierarchical(URI.java:3232)
at java.base/java.net.URI$Parser.parse(URI.java:3174)
at java.base/java.net.URI.(URI.java:623)
at com.turn.ttorrent.common.Torrent.(Torrent.java:191)
... 8 more

I have the same issue, but an earlier version of joal is OK for this site

@anthonyraymond
Copy link
Owner

Hello there, i'm aware of this issue, but i'm having a lot of work atm and i don't have much time to look at it to eventually fix the issue on my spare time.

But i will as soon as possible. I'll keep you updated

@HmgvPy4Q
Copy link

Temporary solution: encode '|' to '%7C' (edit the tracker url in the torrent file).

@anthonyraymond
Copy link
Owner

Note to me:
Should be possible to update to latest version of ttorent and replace parsing with : com/turn/ttorrent/common/TorrentParser.java

Joal is using an old version, but i don't think that it'm depending of ttorrent too much except for some constants or parsing.

Maybe @laur89 will happen to have some time before i do ^^.

Place your bets.

@butts136
Copy link

Hi, Any news about this problem ? Editing ('|' to '%7C') the trackers is not that easy when you download a couple torrents a day.

Just asking, Thanks ;)

@laur89
Copy link
Contributor

laur89 commented Jun 18, 2023

@butts136 been away from joal for a while. Will try to make some time coming week, but no promises.

@skywalkerscott
Copy link

@butts136 been away from joal for a while. Will try to make some time coming week, but no promises.

Are there any other tools similar to JOAL? The authkey problem is still there.

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

8 participants