[Console] Remove exec from console component

maxbeckers · maxbeckers · commit 1805f29d197b · 2023-05-19T10:18:54.000+02:00
diff --git a/src/Symfony/Component/Console/Helper/QuestionHelper.php b/src/Symfony/Component/Console/Helper/QuestionHelper.php
@@ -500,13 +500,11 @@ private function isInteractiveInput($inputStream): bool
             return self::$stdinIsInteractive = @posix_isatty(fopen('php://stdin', 'r'));
         }
 
-        if (!\function_exists('exec')) {
+        if (!\function_exists('shell_exec')) {
             return self::$stdinIsInteractive = true;
         }
 
-        exec('stty 2> /dev/null', $output, $status);
-
-        return self::$stdinIsInteractive = 1 !== $status;
+        return self::$stdinIsInteractive = !is_string(shell_exec('stty 2> /dev/null'));
     }
 
     /**
diff --git a/src/Symfony/Component/Console/Terminal.php b/src/Symfony/Component/Console/Terminal.php
@@ -64,14 +64,12 @@ public static function hasSttyAvailable(): bool
             return self::$stty;
         }
 
-        // skip check if exec function is disabled
-        if (!\function_exists('exec')) {
+        // skip check if shell_exec function is disabled
+        if (!\function_exists('shell_exec')) {
             return false;
         }
 
-        exec('stty 2>&1', $output, $exitcode);
-
-        return self::$stty = 0 === $exitcode;
+        return self::$stty = is_string(shell_exec('stty 2>&1'));
     }
 
     private static function initDimensions()
diff --git a/src/Symfony/Component/Console/Tests/Helper/QuestionHelperTest.php b/src/Symfony/Component/Console/Tests/Helper/QuestionHelperTest.php
@@ -430,7 +430,7 @@ public function testAskHiddenResponse()
         $this->assertEquals('8AM', $dialog->ask($this->createStreamableInputInterfaceMock($this->getInputStream("8AM\n")), $this->createOutputInterface(), $question));
     }
 
-    public function testAskHiddenResponseTrimmed()
+    public function testAskHiddenResponseNotTrimmed()
     {
         if ('\\' === \DIRECTORY_SEPARATOR) {
             $this->markTestSkipped('This test is not supported on Windows');
@@ -442,7 +442,7 @@ public function testAskHiddenResponseTrimmed()
         $question->setHidden(true);
         $question->setTrimmable(false);
 
-        $this->assertEquals(' 8AM', $dialog->ask($this->createStreamableInputInterfaceMock($this->getInputStream(' 8AM')), $this->createOutputInterface(), $question));
+        $this->assertEquals(' 8AM'.\PHP_EOL, $dialog->ask($this->createStreamableInputInterfaceMock($this->getInputStream(' 8AM'.\PHP_EOL)), $this->createOutputInterface(), $question));
     }
 
     public function testAskMultilineResponseWithEOF()
diff --git a/src/Symfony/Component/Console/Tests/TerminalTest.php b/src/Symfony/Component/Console/Tests/TerminalTest.php
@@ -77,8 +77,8 @@ public function testSttyOnWindows()
             $this->markTestSkipped('Must be on windows');
         }
 
-        $sttyString = exec('(stty -a | grep columns) 2>&1', $output, $exitcode);
-        if (0 !== $exitcode) {
+        $sttyString = shell_exec('(stty -a | grep columns) 2>&1');
+        if (!is_string($sttyString)) {
             $this->markTestSkipped('Must have stty support');
         }
 

Original file line number	Diff line number	Diff line change
`@@ -500,13 +500,11 @@ private function isInteractiveInput($inputStream): bool`
`500`	`500`	`return self::$stdinIsInteractive = @posix_isatty(fopen('php://stdin', 'r'));`
`501`	`501`	`}`
`502`	`502`
`503`		`- if (!\function_exists('exec')) {`
	`503`	`+ if (!\function_exists('shell_exec')) {`
`504`	`504`	`return self::$stdinIsInteractive = true;`
`505`	`505`	`}`
`506`	`506`
`507`		`- exec('stty 2> /dev/null', $output, $status);`
`508`		`-`
`509`		`- return self::$stdinIsInteractive = 1 !== $status;`
	`507`	`+ return self::$stdinIsInteractive = !is_string(shell_exec('stty 2> /dev/null'));`
`510`	`508`	`}`
`511`	`509`
`512`	`510`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -64,14 +64,12 @@ public static function hasSttyAvailable(): bool`
`64`	`64`	`return self::$stty;`
`65`	`65`	`}`
`66`	`66`
`67`		`- // skip check if exec function is disabled`
`68`		`- if (!\function_exists('exec')) {`
	`67`	`+ // skip check if shell_exec function is disabled`
	`68`	`+ if (!\function_exists('shell_exec')) {`
`69`	`69`	`return false;`
`70`	`70`	`}`
`71`	`71`
`72`		`- exec('stty 2>&1', $output, $exitcode);`
`73`		`-`
`74`		`- return self::$stty = 0 === $exitcode;`
	`72`	`+ return self::$stty = is_string(shell_exec('stty 2>&1'));`
`75`	`73`	`}`
`76`	`74`
`77`	`75`	`private static function initDimensions()`
Original file line number	Diff line number	Diff line change
`@@ -430,7 +430,7 @@ public function testAskHiddenResponse()`
`430`	`430`	`$this->assertEquals('8AM', $dialog->ask($this->createStreamableInputInterfaceMock($this->getInputStream("8AM\n")), $this->createOutputInterface(), $question));`
`431`	`431`	`}`
`432`	`432`
`433`		`- public function testAskHiddenResponseTrimmed()`
	`433`	`+ public function testAskHiddenResponseNotTrimmed()`
`434`	`434`	`{`
`435`	`435`	`if ('\\' === \DIRECTORY_SEPARATOR) {`
`436`	`436`	`$this->markTestSkipped('This test is not supported on Windows');`
`@@ -442,7 +442,7 @@ public function testAskHiddenResponseTrimmed()`
`442`	`442`	`$question->setHidden(true);`
`443`	`443`	`$question->setTrimmable(false);`
`444`	`444`
`445`		`- $this->assertEquals(' 8AM', $dialog->ask($this->createStreamableInputInterfaceMock($this->getInputStream(' 8AM')), $this->createOutputInterface(), $question));`
	`445`	`+ $this->assertEquals(' 8AM'.\PHP_EOL, $dialog->ask($this->createStreamableInputInterfaceMock($this->getInputStream(' 8AM'.\PHP_EOL)), $this->createOutputInterface(), $question));`
`446`	`446`	`}`
`447`	`447`
`448`	`448`	`public function testAskMultilineResponseWithEOF()`
Original file line number	Diff line number	Diff line change
`@@ -77,8 +77,8 @@ public function testSttyOnWindows()`
`77`	`77`	`$this->markTestSkipped('Must be on windows');`
`78`	`78`	`}`
`79`	`79`
`80`		`- $sttyString = exec('(stty -a \| grep columns) 2>&1', $output, $exitcode);`
`81`		`- if (0 !== $exitcode) {`
	`80`	`+ $sttyString = shell_exec('(stty -a \| grep columns) 2>&1');`
	`81`	`+ if (!is_string($sttyString)) {`
`82`	`82`	`$this->markTestSkipped('Must have stty support');`
`83`	`83`	`}`
`84`	`84`