Improve PHP 8.4+ support by avoiding implicitly nullable types

Author: clue

.github/workflows/ci.yml

@@ -7,10 +7,11 @@ on:
 jobs:
   PHPUnit:
     name: PHPUnit (PHP ${{ matrix.php }})
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     strategy:
       matrix:
         php:
+          - 8.4
           - 8.3
           - 8.2
           - 8.1
@@ -35,7 +36,7 @@ jobs:
 
   PHPUnit-hhvm:
     name: PHPUnit (HHVM)
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-24.04
     continue-on-error: true
     steps:
       - uses: actions/checkout@v4

composer.json

@@ -12,8 +12,8 @@
     ],
     "require": {
         "php": ">=5.3",
-        "react/promise": "^3 || ^2.1 || ^1.2",
-        "react/socket": "^1.12"
+        "react/promise": "^3.2 || ^2.1 || ^1.2",
+        "react/socket": "^1.16"
     },
     "require-dev": {
         "clue/block-react": "^1.5",

src/Client.php

@@ -34,8 +34,12 @@ final class Client implements ConnectorInterface
     public function __construct(
         #[\SensitiveParameter]
         $socksUri,
-        ConnectorInterface $connector = null
+        $connector = null
     ) {
+        if ($connector !== null && !$connector instanceof ConnectorInterface) { // manual type check to support legacy PHP < 7.1
+            throw new InvalidArgumentException('Argument #2 ($connector) expected null|React\Socket\ConnectorInterface');
+        }
+
         // support `sockss://` scheme for SOCKS over TLS
         // support `socks+unix://` scheme for Unix domain socket (UDS) paths
         if (preg_match('/^(socks(?:5|4)?)(s|\+unix):\/\/(.*?@)?(.+?)$/', $socksUri, $match)) {

src/Server.php

@@ -58,11 +58,18 @@ final class Server
      * @param null|array|callable $auth
      */
     public function __construct(
-        LoopInterface $loop = null,
-        ConnectorInterface $connector = null,
+        $loop = null,
+        $connector = null,
         #[\SensitiveParameter]
         $auth = null
     ) {
+        if ($loop !== null && !$loop instanceof LoopInterface) { // manual type check to support legacy PHP < 7.1
+            throw new \InvalidArgumentException('Argument #1 ($loop) expected null|React\EventLoop\LoopInterface');
+        }
+        if ($connector !== null && !$connector instanceof ConnectorInterface) { // manual type check to support legacy PHP < 7.1
+            throw new \InvalidArgumentException('Argument #2 ($connector) expected null|React\Socket\ConnectorInterface');
+        }
+
         if (\is_array($auth)) {
             // wrap authentication array in authentication callback
             $this->auth = function (

tests/ClientTest.php

@@ -83,6 +83,12 @@ public function testCtorAcceptsUriWithSocks5UnixScheme()
         $this->assertTrue(true);
     }
 
+    public function testCtorThrowsForInvalidConnector()
+    {
+        $this->setExpectedException('InvalidArgumentException', 'Argument #2 ($connector) expected null|React\Socket\ConnectorInterface');
+        new Client('127.0.0.1:1080', 'connector');
+    }
+
     public function testCtorThrowsForInvalidUri()
     {
         $this->setExpectedException("InvalidArgumentException");

tests/ServerTest.php

@@ -68,6 +68,18 @@ public function testConstructorWithStaticAuthArray()
         ));
     }
 
+    public function testConstructorThrowsForInvalidLoop()
+    {
+        $this->setExpectedException('InvalidArgumentException', 'Argument #1 ($loop) expected null|React\EventLoop\LoopInterface');
+        new Server('loop');
+    }
+
+    public function testConstructorThrowsForInvalidConnector()
+    {
+        $this->setExpectedException('InvalidArgumentException', 'Argument #2 ($connector) expected null|React\Socket\ConnectorInterface');
+        new Server(null, 'connector');
+    }
+
     public function testConstructorWithInvalidAuthenticatorThrows()
     {
         $this->setExpectedException("InvalidArgumentException");