fix: specifically look if it's the local port that's already in use

carlalexander · carlalexander · commit cc63f0ffcce8 · 2022-04-19T18:33:32.000-04:00
diff --git a/src/Tool/Ssh.php b/src/Tool/Ssh.php
@@ -49,8 +49,8 @@ public static function tunnelBastionHost(array $bastionHost, int $localPort, str
                 return;
             }
 
-            if (false !== stripos($buffer, 'address already in use')) {
-                throw new RuntimeException(sprintf('Unable to create SSH tunnel. Port "%s" is already in use.', $localPort));
+            if (false !== stripos($buffer, sprintf('%s: address already in use', $localPort))) {
+                throw new RuntimeException(sprintf('Unable to create SSH tunnel. Local port "%s" is already in use.', $localPort));
             }
         });
 

Original file line number	Diff line number	Diff line change
`@@ -49,8 +49,8 @@ public static function tunnelBastionHost(array $bastionHost, int $localPort, str`
`49`	`49`	`return;`
`50`	`50`	`}`
`51`	`51`
`52`		`- if (false !== stripos($buffer, 'address already in use')) {`
`53`		`- throw new RuntimeException(sprintf('Unable to create SSH tunnel. Port "%s" is already in use.', $localPort));`
	`52`	`+ if (false !== stripos($buffer, sprintf('%s: address already in use', $localPort))) {`
	`53`	`+ throw new RuntimeException(sprintf('Unable to create SSH tunnel. Local port "%s" is already in use.', $localPort));`
`54`	`54`	`}`
`55`	`55`	`});`
`56`	`56`