xwmario / rutorrent

Automatically exported from code.google.com/p/rutorrent
0 stars 0 forks source link

Escaping in rXMLRPCRequest breaks move plugin #800

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Have a torrent containing a file with a " or ' in it.
2. Attempt to use the move plugin to move it to another directory.
3. That file fails to move to the new directory.

What is the expected output? What do you see instead?

I would expect the file to be moved by the move plugin. "Testing'.zip" attempts 
to move the file "Testing\'.zip" including the blackslash in the path.

What environment are you using?
1. My ruTorrent version is 3.5
2. My rTorrent version is 0.9.2

From my perspective this issue appears to be with rXMLRPCRequest and how it 
escapes the data, rather than the move plugin.

Original issue reported on code.google.com by anthonyr...@gmail.com on 9 Feb 2013 at 10:04

GoogleCodeExporter commented 8 years ago
What is your mean under the words "the move plugin"? What about the *real* 
plugin's name?

Original comment by novik65 on 9 Feb 2013 at 10:45

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
datadir

Original comment by anthonyr...@gmail.com on 9 Feb 2013 at 10:47

GoogleCodeExporter commented 8 years ago

Original comment by novik65 on 9 Feb 2013 at 12:30

GoogleCodeExporter commented 8 years ago
1) Can't reproduce this.
2) Check permissions. If you set $datadir_runmode in plugin's conf.php to 
'webserver' then webserver and rtorrent users must have rights to write to 
destination directory. Otherwise - rtorrent user only.
3) Turn on debug mode in the plugin's conf.php and attach here /tmp/errors.log.

Original comment by novik65 on 15 Mar 2013 at 1:16

GoogleCodeExporter commented 8 years ago
This can be closed as invalid.

It appears this was fixed before rutorrent 3.5 was released and my duplication 
of this issue in 3.5 was a case of a faulty upgrade from an older rutorrent 
version.

Sorry for wasting your time.

Original comment by anthonyr...@gmail.com on 11 Apr 2013 at 3:19

GoogleCodeExporter commented 8 years ago

Original comment by novik65 on 11 Apr 2013 at 4:19