patric99 / remote-torrent-adder

Automatically exported from code.google.com/p/remote-torrent-adder
0 stars 1 forks source link

Sending torrents to different servers doesn't work properly (ruTorrent) #120

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What version of Chrome are you using? On what operating system?
Version 39.0.2171.62 (64-bit) on Windows 7 SP1 64-bit.

What client (version?) are you connecting to? Are you using SSL?
Version 1.2.2, SSL is on.

What steps will reproduce the problem?
1. Add multiple servers to RTA (set port to 443 and SSL is on)
2. Try different servers from context menu.

What is the expected output?
RTA should send torrents to selected server from context menu.

What do you see instead?
It always uses the first added server.

Original issue reported on code.google.com by off...@gmail.com on 13 Nov 2014 at 1:20

GoogleCodeExporter commented 9 years ago
can you supply some more information on your settings, in particular: do you 
use the label/directory selection dialog?

Original comment by jul...@gmail.com on 18 Nov 2014 at 7:34

GoogleCodeExporter commented 9 years ago
Yes, i do. "Label/Directory interactivity" is checked.

###

servers = [{"name":"Subb","client":"ruTorrent 
WebUI","host":"Subb.bsite.com","port":"443","hostsecure":true,"login":"username"
,"password":"passwordx","ruTorrentrelativepath":"/webui","rutorrentlabel":"","ru
torrentdirectory":"","rutorrentdirlabelask":true,"rutorrentaddpaused":false,"dir
list":"[\"moviepacks",\"tvpacks\"]","labellist":"[\"moviepacks",\"tvpacks\"]"},{
"name":"Subc","client":"ruTorrent 
WebUI","host":"Subc.bsite.com","port":"443","hostsecure":true,"login":"username"
,"password":"passwordx","ruTorrentrelativepath":"/webui","rutorrentlabel":"","ru
torrentdirectory":"","rutorrentdirlabelask":true,"rutorrentaddpaused":false}]
linksfoundindicator = true
popupduration = 2000
showpopups = true
catchfromcontextmenu = true
catchfrompage = false
linkmatches = 
([\]\[]|\b)\.torrent\b([^\-]|$)~torrents\.php\?action=download~\?p=torrents&pid=
[0-9]+&action=download&tid=[0-9]+~asite\.net\/.*-d-[0-9]+\.ts
catchfromnewtab = false

Original comment by off...@gmail.com on 18 Nov 2014 at 8:04

GoogleCodeExporter commented 9 years ago
It works if i disable it.

Original comment by off...@gmail.com on 18 Nov 2014 at 8:22

GoogleCodeExporter commented 9 years ago
okay, so two things:

1. for clarification: if you just left-click a torrent link, it'll always 
select the first server. there's just no reasonable way to implement choosing 
the server better than what the context menu already does.

2. i believe i may have found the issue with the context menu, and it's 
directly related to the context menu and label/dir chooser dialog, so it's 
pretty isolated. i've added a bit of code to try to fix this. if you could 
check whether the attached zip file produces the desired behaviour, that'd 
bring us closer to a solution. instructions to add it are in TryNewCode, with 
BackupRestoreSettings you can import the settings from your webstore version 
(disable that while you test the local one).

Original comment by jul...@gmail.com on 18 Nov 2014 at 8:24

Attachments:

GoogleCodeExporter commented 9 years ago
1. I don't use left-click for downloading. So no problem.

2. Tested the new code but it still sends torrent to the first added server.

Original comment by off...@gmail.com on 18 Nov 2014 at 8:49

GoogleCodeExporter commented 9 years ago
hey, please take a look at this one. missed one call in the earlier version.

Original comment by jul...@gmail.com on 18 Nov 2014 at 11:00

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks, this version works.

Original comment by off...@gmail.com on 18 Nov 2014 at 11:11

GoogleCodeExporter commented 9 years ago
ok, i'll look at some other things and will publish it to the webstore today or 
tomorrow. thanks for reporting and bearing with me.

Original comment by jul...@gmail.com on 18 Nov 2014 at 11:11

GoogleCodeExporter commented 9 years ago

Original comment by jul...@gmail.com on 19 Nov 2014 at 2:11