[Fix] Fixes escaping issues on remote commands

andres-montanez · Feb 11, 2017 · 28396c1 · 28396c1
1 parent eddb88b
commit 28396c1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/Runtime/Runtime.php b/src/Runtime/Runtime.php
@@ -424,8 +424,8 @@ public function runRemoteCommand($cmd, $jail, $timeout = 120)
             $cmdDelegate = sprintf('cd %s && %s', $hostPath, $cmdDelegate);
         }
 
-        $cmdRemote = str_replace(['"', '&', ';', '|'], ['\"', '\&', '\;', '\|'], $cmdDelegate);
-        $cmdLocal = sprintf('ssh -p %d %s %s@%s sh -c \"%s\"', $sshConfig['port'], $sshConfig['flags'], $user, $host, $cmdRemote);
+        $cmdRemote = str_replace('"', '\"', $cmdDelegate);
+        $cmdLocal = sprintf('ssh -p %d %s %s@%s "%s"', $sshConfig['port'], $sshConfig['flags'], $user, $host, $cmdRemote);
 
         return $this->runLocalCommand($cmdLocal, $timeout);
     }