do not evaluate SAML cfg every time app is loaded

- it tests only the first configuration, others were not taken into
  account
- the configuration check is also only needed when SAML auth is actually
  happening

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
This commit is contained in:
Arthur Schiwon 2022-01-12 23:55:40 +01:00 committed by blizzz (Rebase PR Action)
parent c51048b566
commit 0000691857

View file

@ -68,11 +68,6 @@ $returnScript = false;
$type = '';
switch ($config->getAppValue('user_saml', 'type')) {
case 'saml':
try {
$oneLoginSettings = new \OneLogin\Saml2\Settings($samlSettings->getOneLoginSettingsArray(1));
} catch (\OneLogin\SAML2\Error $e) {
$returnScript = true;
}
$type = 'saml';
break;
case 'environment-variable':