DRCIT/snipe-it
1
0
Fork 0
mirror of https://github.com/snipe/snipe-it.git synced 2025-02-02 08:21:09 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Few more log lines

Browse source 
Signed-off-by: snipe <snipe@snipe.net>
This commit is contained in:
snipe 2022-05-10 12:27:42 -07:00
parent 646166b2b6
commit 4fccf4ddc4
1 changed files with 17 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
app/Http/Controllers/Auth/LoginController.php
View file

@ -120,7 +120,8 @@ class LoginController extends Controller
try { try {
Log::debug("Attempting to log user in by SAML authentication."); Log::debug("Attempting to log user in by SAML authentication.");
$user = $saml->samlLogin($samlData); $user = $saml->samlLogin($samlData);
if(!is_null($user)) {
if (!is_null($user)) {
Auth::login($user); Auth::login($user);
} else { } else {
$username = $saml->getUsername(); $username = $saml->getUsername();
@ -133,11 +134,26 @@ class LoginController extends Controller
$user->last_login = \Carbon::now(); $user->last_login = \Carbon::now();
$user->save(); $user->save();
} }
} catch (\Exception $e) { } catch (\Exception $e) {
\Log::warning("There was an error authenticating the SAML user: " . $e->getMessage()); \Log::warning("There was an error authenticating the SAML user: " . $e->getMessage());
throw new \Exception($e->getMessage()); throw new \Exception($e->getMessage());
} }
// Fallthrough with better logging
} else {
// Better logging
if (!$saml->isEnabled()) {
\Log::warning("SAML page requested, but SAML does not seem to enabled.");
} else {
\Log::warning("SAML page requested, but samlData seems empty.");
}
} }
\Log::warning("Something else went wrong while trying to login as SAML user");
} }
/** /**