Allow stdin to be redirected to ssh command (#6275)

* Allow stdin to be redirected to ssh command

As described in #6274, stdin is currently not redirected to ssh command.

With this change the command is adapted to be consistent with `sql:cli` and stdin is redirected.

* Add usage info / test todo for stdin redirection (#6274)

* Update src/Commands/core/SshCommands.php

---------

Co-authored-by: Leo von Klenze <[email protected]>
Co-authored-by: Moshe Weitzman <[email protected]>

Author: 3 people

src/Commands/core/SshCommands.php

@@ -4,6 +4,8 @@
 
 namespace Drush\Commands\core;
 
+use Consolidation\AnnotatedCommand\Input\StdinAwareInterface;
+use Consolidation\AnnotatedCommand\Input\StdinAwareTrait;
 use Consolidation\SiteAlias\SiteAliasManagerInterface;
 use Consolidation\SiteProcess\Util\Shell;
 use Consolidation\SiteProcess\Util\Tty;
@@ -13,9 +15,10 @@
 use Drush\Commands\DrushCommands;
 
 #[CLI\Bootstrap(DrupalBootLevels::NONE)]
-final class SshCommands extends DrushCommands
+final class SshCommands extends DrushCommands implements StdinAwareInterface
 {
     use AutowireTrait;
+    use StdinAwareTrait;
 
     const SSH = 'site:ssh';
 
@@ -37,6 +40,7 @@ public function __construct(
     #[CLI\Usage(name: 'drush @prod ssh "ls /tmp"', description: 'Run <info>ls /tmp</info> on <info>@prod</info> site.')]
     #[CLI\Usage(name: 'drush @prod ssh "git pull"', description: 'Run <info>git pull</info> on the Drupal root directory on the <info>@prod</info> site.')]
     #[CLI\Usage(name: 'drush ssh "git pull"', description: 'Run <info>git pull</info> on the local Drupal root directory.')]
+    #[CLI\Usage(name: 'echo \'Deny from all\' | drush @prod ssh "tee > do-not-expose-dir/.htaccess"', description: 'Protect directory <info>do-not-expose-dir</info> from being served by Apache httpd.')]
     #[CLI\Topics(topics: [DocsCommands::ALIASES])]
     public function ssh(array $code, $options = ['cd' => self::REQ]): void
     {
@@ -56,8 +60,10 @@ public function ssh(array $code, $options = ['cd' => self::REQ]): void
         }
 
         $process = $this->processManager()->siteProcess($alias, $code);
-        if (Tty::isTtySupported()) {
-            $process->setTty($options['tty']);
+        if (!Tty::isTtySupported()) {
+            $process->setInput($this->stdin()->getStream());
+        } else {
+            $process->setTty($process->setTty($options['tty']));
         }
         // The transport handles the chdir during processArgs().
         $fallback = $alias->hasRoot() ? $alias->root() : null;

tests/functional/SiteSshTest.php

@@ -72,4 +72,8 @@ public function testSshSingleArgs()
         $expected = "[notice] Simulating: ssh -o PasswordAuthentication=no user@server 'cd /path/to/drupal && ls /path1 /path2'";
         $this->assertStringContainsString($expected, $this->getSimplifiedErrorOutput());
     }
+
+    /**
+     * @todo If this becomes an integration test, add test for stdin handling.
+     */
 }