Merge pull request swooletw#368 from lilianjin/patch-3

albertcht · web-flow · commit 6b3732d77ba8 · 2020-03-08T15:46:45.000+08:00
There is no session option on 5.8, so fix it
diff --git a/src/Websocket/Middleware/StartSession.php b/src/Websocket/Middleware/StartSession.php
@@ -86,6 +86,6 @@ public function getSession(Request $request)
      */
     protected function sessionConfigured()
     {
-        return Arr::get($this->manager->getSessionConfig(), 'session') !== null;
+        return ! is_null($this->manager->getSessionConfig()['driver'] ?? null);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,6 @@ public function getSession(Request $request)`
`86`	`86`	`*/`
`87`	`87`	`protected function sessionConfigured()`
`88`	`88`	`{`
`89`		`- return Arr::get($this->manager->getSessionConfig(), 'session') !== null;`
	`89`	`+ return ! is_null($this->manager->getSessionConfig()['driver'] ?? null);`
`90`	`90`	`}`
`91`	`91`	`}`