[Execute] Escape call parameters for Windows

[Calum]

From: ​http://forum.deluge-torrent.org/viewtopic.php?f=9&t=48203&p=221405

To expand on my prior post, I'm talking about escaping required by the Windows OS command line. Unless an ampersand is escaped, Windows interprets it as separating 2 commands, so it runs everything before the ampersand as one command and everything after it as another command (ref: ​http://www.robvanderwoude.com/condexec.php). So when the execute plugin tries to run a command and the filename has an ampersand in it, Windows tries to run it as 2 separate and partial commands.

I have tested my script by calling it manually, and if it is called with the ampersand properly escaped, my script works fine. So the problem isn't my script as you suggested. And it's not an encoding issue. It's a Windows-specific command line escaping issue with the plugin.

​http://stackoverflow.com/questions/14020944/how-do-i-escape-ampersands-in-batch-file-function-parameter

[Calum]

So the answer here is to escape the ampersand with a tripe-caret e.g. ^^^& and then quote any echo output in the batch script. This requires more testing and a better understanding of how getProcessOutputAndValue works so leaving this one for now.

from twisted.internet.utils import getProcessOutputAndValue
from twisted.internet import defer
from twisted.internet import reactor
import os
import traceback

@defer.inlineCallbacks
def runProgram():
    torrent_id = "torrhash"
    torrent_name = "test&name".replace('&', '^^^&')
    download_location = "C:/dir"
    command = "C:/Documents and Settings/Administrator/Desktop/test.bat"
    args = (torrent_id, torrent_name, download_location)
    try:
        result = yield getProcessOutputAndValue(command, args, env=os.environ)
        print result
        reactor.stop()
    except Exception:
        traceback.print_exc()
        reactor.stop()

reactor.callLater(0, runProgram)
reactor.run()

import subprocess
torrent_id = 'torrhash'
torrent_name = 'test^^^&name'
download_location = 'C:/dir'
command = 'C:/Documents and Settings/Administrator/Desktop/test.bat'
result = subprocess.call([command, torrent_id, torrent_name, download_location])

test.bat

@echo off
set torrentid=%1
set torrentname=%2
set torrentpath=%3

@echo "Torrent Details:  %torrentname% %torrentpath% %torrentid%"  >> "%userprofile%\Desktop\execute_script.log"

[Calum]

Fixed the use-case for ampersand as other chars unlikely to be in torrent name (libtorrent fixes invalid chars on windows) or directories.

1.3-stable: [5f92810f761e7df7], develop: [abf90f1dd647a9b46]

[dashbad]

I have come across this bug because my execute script is failing on Linux, because Ampersands are being escaped to

^^^&

Steps to reproduce:

1. Write bash script:

   #!/bin/bash
   exec 1> >(logger -s -t $(basename $0)) 2>&1
   export name="$1"
   echo $name
   

2. Set script as execute script
3. Trigger script for a torrent with an ampersand in the name

The syslog shows that the ampersand in the name has been converted to

^^^&

Can this change be rolled back/modified so it doesn't impact non-Windows users?

[Calum]

This was fixed in 1.3.14: [df88c82265f7b]