Merge pull request #39 from clue-labs/php8

Socket address of closed socket should be null (support PHP 8)

Author: jsor

.github/workflows/ci.yml

@@ -11,6 +11,7 @@ jobs:
     strategy:
       matrix:
         php:
+          - 8.0
           - 7.4
           - 7.3
           - 7.2

README.md

@@ -46,7 +46,7 @@ $ composer require react/datagram:^1.5
 See also the [CHANGELOG](CHANGELOG.md) for details about version upgrades.
 
 This project aims to run on any platform and thus does not require any PHP
-extensions and supports running on legacy PHP 5.3 through current PHP 7+ and
+extensions and supports running on legacy PHP 5.3 through current PHP 8+ and
 HHVM.
 It's *highly recommended to use PHP 7+* for this project.
 

src/Socket.php

@@ -36,11 +36,19 @@ public function __construct(LoopInterface $loop, $socket, Buffer $buffer = null)
 
     public function getLocalAddress()
     {
+        if ($this->socket === false) {
+            return null;
+        }
+
         return $this->sanitizeAddress(@\stream_socket_get_name($this->socket, false));
     }
 
     public function getRemoteAddress()
     {
+        if ($this->socket === false) {
+            return null;
+        }
+
         return $this->sanitizeAddress(@\stream_socket_get_name($this->socket, true));
     }