[Solved] uploaded.net: %-Zeichen im Passwort führt zu IllegalArgumentException von URLDecoder

Beim letzten Passwortwechsel des uploaded.net-Benutzerkontos ist mir aufgefallen, dass Passwörter mit enthaltenen %-Zeichen immer zu einer `invalid account`-Meldung führen. Im Log findet sich in diesem Fall eine aus URLDecoder.decode() geworfene IllegalArgumentException mit dem folgenden Backtrace:

25 27.03.13 11:21:22 - SEVERE [java_downloader] -> AccountUpdate: uploaded.to failed!
EXCEPTION   25 27.03.13 11:21:22 - SEVERE [java_downloader] -> SEVERE Exception occurred
java.lang.IllegalArgumentException: URLDecoder: Incomplete trailing escape (%) pattern
	at java.net.URLDecoder.decode(URLDecoder.java:168)
	at jd.plugins.hoster.Uploadedto.api_getAccessToken(Uploadedto.java:748)
	at jd.plugins.hoster.Uploadedto.fetchAccountInfo_API(Uploadedto.java:315)
	at jd.plugins.hoster.Uploadedto.fetchAccountInfo(Uploadedto.java:364)
[...]

Der Grund dafür findet sich im uploaded.net-Plugin (src/jd/plugins/Uploadedto.java), api_getAccessToken(), Zeile 748, ausgecheckt aus dem SVN - revision 20299:

br.postPage("**External links are only visible to Support Staff**, "name=" + Encoding.urlEncode(account.getUser()) + "&pass=" + JDHash.getSHA1(URLDecoder.decode(account.getPass(), "UTF-8").toLowerCase(Locale.ENGLISH)) + "&ishash=1&app=JDownloader");

Hier wird versucht, das Passwort vor dem Bilden des SHA1-Hashes durch URLDecoder.decode() Prozent-dekodieren zu lassen. Bei Passwörtern ohne enthaltene %-Zeichen ist das wirkungslos, sind %-Zeichen enthalten, führt das zu Problemen - entweder durch die Verfälschung des Passworts, sofern die Zeichen nach dem %-Zeichen eine gültige %-Escape-Sequenz bilden, oder durch die besagte Exception.

Das Problem lässt sich leicht auch unabhängig von JD reproduzieren, ein Minimalbeispiel wie

System.out.println(URLDecoder.decode("foo%K"));

führt zu exakt derselben Exception:

Exception in thread "main" java.lang.IllegalArgumentException: URLDecoder: Incomplete trailing escape (%) pattern
	at java.net.URLDecoder.decode(URLDecoder.java:168)
	at java.net.URLDecoder.decode(URLDecoder.java:83)
	at Foo.main(Foo.java:6)

Die Lösung dürfte aber mutmasslich einfach sein: URLDecoder.decode() entfernen.

[pspzockerscene]

Hi
Ich hab die Änderungen mal so übernommen.
Danke fürs Feedback und sorry für meine späte Antwort.

GreeZ psp

Kein Problem, danke für das Beheben des Problems. Sobald ich das Plugin-Update erhalte, werde ich erneut ein Passwort mit %-Zeichen testen und melden, ob nun das korrekt funktioniert.

[pspzockerscene]

Danke

GreeZ pspzockerscene