Merge pull request #37 from tflori/2.0

initialize saml.slo session with empty array
codegreencreative · Jan 7, 2021 · 050035d · 050035d
2 parents 650cf30 + 68d048d
commit 050035d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/Http/Controllers/LogoutController.php b/src/Http/Controllers/LogoutController.php
@@ -21,6 +21,10 @@ public function index(Request $request)
             $slo_redirect = $request->session()->get('saml.slo_redirect');
         }
 
+        if (null === $request->session()->get('saml.slo')) {
+            $request->session()->put('saml.slo', []);
+        }
+
         // Need to broadcast to our other SAML apps to log out!
         // Loop through our service providers and "touch" the logout URL's
         foreach (config('samlidp.sp') as $key => $sp) {

diff --git a/src/Listeners/SamlLogout.php b/src/Listeners/SamlLogout.php
@@ -16,7 +16,7 @@ class SamlLogout
     public function handle(Logout $event)
     {
         // Make sure we are not in the process of SLO when handling the redirect
-        if (!session('saml.slo')) {
+        if (null === session('saml.slo')) {
             abort(redirect('saml/logout'), 200);
         }
     }