erickok / transdroid

Manage your torrents from your Android device
GNU General Public License v3.0
1.28k stars 201 forks source link

Latest Deluge broke transdrone #355

Closed yeyeoke closed 7 years ago

yeyeoke commented 7 years ago

As stated by the title, lastest update to Deluge (1.3.14) had some added security that broke transdrone.

Here is a link to Deluge forum that discusses this: http://forum.deluge-torrent.org/viewtopic.php?f=7&t=54428

marchom commented 7 years ago

As far as I can tell, the guys from Deluge dev added a check for the content type of the http request on the web interface.

This should be easily fixed by adding

httppost.setHeader("Content-Type", "application/json"); httppost.setHeader("Accept", "application/json");

to

app/src/main/java/org/transdroid/daemon/Deluge/DelugeAdapter.java line:525;556

Perhaps someone can test if this fixes it and is backwards compatible. It would be great to get this compatibility issue sorted out.

AnnieTheEagle commented 7 years ago

@marchom I actually did this change yesterday but only got around to testing it on my Android Device (Samsung Galaxy S7 Edge - 6.0.1). It works and Deluge sprang back to life! 😄

Just adding the httppost.setHeader("content-type", "application/json"); was sufficient.

See Deluge code change http://git.deluge-torrent.org/deluge/commit/?h=1.3-stable&id=318ab179865e0707d7945edc3a13a464a108d583.

marchom commented 7 years ago

Yeah, that makes sense. Deluge only checks on the content-type. Good stuff @AnnieTheEagle. I'll wait until F-Droid picks up the new version to try it out.

benskyio commented 7 years ago

Hi All,

Was this commit added to 2.5.7 ? I'm still seeing these all over my transdroid logs:

38808 -- Fri Mar 10 15:42:21 PST 2017 -- 3 -- Error: AuthenticationFailure exception: Password error? Server time difference? No (valid) cookie in response and JSON was:

Thanks in advance for your help. bmo

AnnieTheEagle commented 7 years ago

I still get alot of failures while trying to refresh Deluge. It doesn't happen with rTorrent based servers.

benskyio commented 7 years ago

I haven't been able to connect to deluge at all for 2 days, not just refresh errors.

On Mar 10, 2017 4:19 PM, "Annie the Eagle" notifications@github.com wrote:

I still get alot of failures while trying to refresh Deluge. It doesn't happen with rTorrent based servers.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/erickok/transdroid/issues/355#issuecomment-285820576, or mute the thread https://github.com/notifications/unsubscribe-auth/AAdU2dxcCYvwMCYAegEcb-YgnFH9FKNGks5rkehygaJpZM4MU8xZ .

boekhold commented 7 years ago

What are the chances of a 2.5.8 release that includes this fix? I can only assume this affects a lot of users.

benskyio commented 7 years ago

Here's a re-compiled version of transdroid with the above fix, for anyone not smart enough to use SDKs, Gradle or Android studio. (I was BARELY smart enough / it took me 2 hours to get all the tools working and get a successfully build).

https://www.dropbox.com/s/lwnu7wswze8zbwk/transdroid_3-11-2017.apk?dl=0

paulpoco commented 7 years ago

Yes I tried to get Android Studio working on Ubuntu but gave up. I have made Android apps before(Icon Packs) but that was on Windows 7.

Thanks for making a temp version, will try it out.

paulpoco commented 7 years ago

You have to export your settings first, then uninstall, then install the temp version, then import your settings.

bicodegas commented 7 years ago

This temp version seems to be working fine. I suppose the new version will end up on f-droid, right?

erickok commented 7 years ago

Yes and I'll release an official 2.5.8 version too. Sorry for the slowdowns I'm away for the weekend.

Ansem93 commented 7 years ago

what's f-droid?

AnnieTheEagle commented 7 years ago

@erickok - Will the official 2.5.8 have any fixes for the issues around deluge sometimes not refreshing.

benskyio commented 7 years ago

@erickok @marchom @AnnieTheEagle - Thanks for addressing. We all appreciate Transdroid, a very convenient app that you provide for free.

@Ansem93 - F-droid is an android package/application manager, similar to the Google App store but specifically for FOSS. https://f-droid.org/

lucifurious commented 7 years ago

ETA on 2.5.8?

ksample commented 7 years ago

Thanks for a great app! I use it every day. I built the latest commit and it connects to Deluge again w/no issues I can see (Nougat on LG V-20).

Thanks again!

SweAdelpha commented 7 years ago

The Dropbox link doesn't work anymore. Can some one be so kind to upload the temp version again? Would really appreciate it!

benskyio commented 7 years ago

Apologies. I restored the download link.

On Thu, Mar 16, 2017 at 7:29 AM, SweAdelpha notifications@github.com wrote:

The Dropbox link doesn't work anymore. Can some one be so kind to upload the temp version again? Would really appreciate it!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/erickok/transdroid/issues/355#issuecomment-287074679, or mute the thread https://github.com/notifications/unsubscribe-auth/AAdU2fkQ8HfOvY4mGgFpXgpit7BrSj6Jks5rmUc5gaJpZM4MU8xZ .

SweAdelpha commented 7 years ago

Thank you so much!

erickok commented 7 years ago

2.5.8 is released.