Merge pull request #1061 from akotulu/master

Fixed onWebsocketConnected isn't called with SSL connection
walkor · Oct 10, 2024 · 7d9d8b7 · 7d9d8b7
2 parents 43f9aad + a3bf4be
commit 7d9d8b7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/Protocols/Websocket.php b/src/Protocols/Websocket.php
@@ -492,13 +492,13 @@ public static function dealHandshake(string $buffer, TcpConnection $connection):
             }
             $handshakeMessage .= "\r\n";
             // Send handshake response.
-            $status = $connection->send($handshakeMessage, true);
+            $connection->send($handshakeMessage, true);
             // Mark handshake complete.
             $connection->context->websocketHandshake = true;
 
             // Try to emit onWebSocketConnected callback.
             $onWebsocketConnected = $connection->onWebSocketConnected ?? $connection->worker->onWebSocketConnected ?? false;
-            if ($status && $onWebsocketConnected) {
+            if ($onWebsocketConnected) {
                 try {
                     $onWebsocketConnected($connection, $request);
                 } catch (Throwable $e) {