[1.x] Avoids usage of SWOOLE_SSL when openssl is not configured o…

…n swoole (#357) * Only uses SWOOLE_SSL if exists * Adds guard Co-Authored-By: 沈唁 <[email protected]> Co-authored-by: 沈唁 <[email protected]>
laravel · Aug 5, 2021 · 64a1c3c · 64a1c3c
1 parent 0a3968e
commit 64a1c3c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/bin/createSwooleServer.php b/bin/createSwooleServer.php
@@ -7,7 +7,9 @@
         $serverState['host'] ?? '127.0.0.1',
         $serverState['port'] ?? '8080',
         SWOOLE_PROCESS,
-        SWOOLE_SOCK_TCP | ((bool) ($config['swoole']['ssl'] ?? 0)) * SWOOLE_SSL,
+        ($config['swoole']['ssl'] ?? false)
+            ? SWOOLE_SOCK_TCP | SWOOLE_SSL
+            : SWOOLE_SOCK_TCP,
     );
 } catch (Throwable $e) {
     Laravel\Octane\Stream::shutdown($e);

diff --git a/src/Commands/StartSwooleCommand.php b/src/Commands/StartSwooleCommand.php
@@ -66,6 +66,12 @@ public function handle(
             return 1;
         }
 
+        if (config('octane.swoole.ssl', false) === true && ! defined('SWOOLE_SSL')) {
+            $this->error('You must configure Swoole with `--enable-openssl` to support ssl.');
+
+            return 1;
+        }
+
         $this->writeServerStateFile($serverStateFile, $extension);
 
         $this->forgetEnvironmentVariables();