Merge pull request 24Slides#35 from dmyers/patch-2

Improve sls handling of errors to match acs action

Author: Artem Brezhnev

src/Http/Controllers/Saml2Controller.php

@@ -86,10 +86,16 @@ public function acs(Auth $auth)
      */
     public function sls(Auth $auth)
     {
-        $error = $auth->sls(config('saml2.retrieveParametersFromServer'));
+        $errors = $auth->sls(config('saml2.retrieveParametersFromServer'));
 
-        if (!empty($error)) {
-            throw new \Exception("Could not log out");
+        if (!empty($errors)) {
+            logger()->error('saml2.error_detail', ['error' => $auth->getLastErrorReason()]);
+            session()->flash('saml2.error_detail', [$auth->getLastErrorReason()]);
+
+            logger()->error('saml2.error', $errors);
+            session()->flash('saml2.error', $errors);
+
+            return redirect(config('saml2.errorRoute'));
         }
 
         return redirect(config('saml2.logoutRoute')); //may be set a configurable default