Merge pull request #44 from gquemener/bugfix/stderr

Rely on last command exit status to raise error

Author: Herzult

src/Ssh/Exec.php

@@ -9,6 +9,7 @@
  *
  * @author Cam Spiers <[email protected]>
  * @author Greg Militello <[email protected]>
+ * @author Gildas Quéméner <[email protected]>
  */
 class Exec extends Subsystem
 {
@@ -19,14 +20,18 @@ protected function createResource()
 
     public function run($cmd, $pty = null, array $env = array(), $width = 80, $height = 25, $width_height_type = SSH2_TERM_UNIT_CHARS)
     {
+        $cmd .= ';echo -ne "[return_code:$?]"';
         $stdout = ssh2_exec($this->getResource(), $cmd, $pty, $env, $width, $height, $width_height_type);
         $stderr = ssh2_fetch_stream($stdout, SSH2_STREAM_STDERR);
         stream_set_blocking($stderr, true);
         stream_set_blocking($stdout, true);
-        $error = stream_get_contents($stderr);
-        if ($error !== '') {
-            throw new RuntimeException($error);
+
+        $output = stream_get_contents($stdout);
+        preg_match('/\[return_code:(.*?)\]/', $output, $match);
+        if ((int) $match[1] !== 0) {
+            throw new RuntimeException(stream_get_contents($stderr), (int) $match[1]);
         }
-        return stream_get_contents($stdout);
+
+        return preg_replace('/\[return_code:(.*?)\]/', '', $output);
     }
 }

tests/Ssh/FunctionalTests/ExecTest.php

@@ -45,7 +45,7 @@ public function testExecuteMultilineOutput()
     }
 
     /**
-     * @expectedException \RuntimeException MessageOnStdErr
+     * @expectedException \RuntimeException
      */
     public function testExecuteErrorOutput()
     {
@@ -54,9 +54,9 @@ public function testExecuteErrorOutput()
         $session = new Session($configuration, $authentication);
 
         $exec = $session->getExec();
-        $output = $exec->run('echo "MessageOnStdErr" > /dev/stderr');
+        $output = $exec->run('false');
 
         $this->assertEquals('', trim($output));
     }
 }
- 
+