Merge pull request #94 from clue-labs/compat

WyriHaximus · web-flow · commit 6e43a558fe83 · 2017-04-23T20:15:06.000+02:00
Work around latest PHP 7.0.18 and 7.1.4 no longer accepting full URIs
diff --git a/src/TcpConnector.php b/src/TcpConnector.php
@@ -66,14 +66,12 @@ public function connect($uri)
             }
         }
 
-        // HHVM fails to parse URIs with a query but no path, so let's add a dummy path
-        // See also https://3v4l.org/jEhLF
-        if (defined('HHVM_VERSION') && isset($parts['query']) && !isset($parts['path'])) {
-            $uri = str_replace('?', '/?', $uri); // @codeCoverageIgnore
-        }
+        // latest versions of PHP no longer accept any other URI components and
+        // HHVM fails to parse URIs with a query but no path, so let's simplify our URI here
+        $remote = 'tcp://' . $parts['host'] . ':' . $parts['port'];
 
         $socket = @stream_socket_client(
-            $uri,
+            $remote,
             $errno,
             $errstr,
             0,
