diff --git a/application/loginbasic.class.inc.php b/application/loginbasic.class.inc.php index 06210cfa4..68f8eeaf6 100644 --- a/application/loginbasic.class.inc.php +++ b/application/loginbasic.class.inc.php @@ -62,6 +62,7 @@ class LoginBasic extends AbstractLoginFSMExtension $iErrorCode = LoginWebPage::EXIT_CODE_WRONGCREDENTIALS; return LoginWebPage::LOGIN_FSM_ERROR; } + Session::Set('auth_user', $sAuthUser); } return LoginWebPage::LOGIN_FSM_CONTINUE; } @@ -70,8 +71,7 @@ class LoginBasic extends AbstractLoginFSMExtension { if (Session::Get('login_mode') == 'basic') { - list($sAuthUser) = $this->GetAuthUserAndPassword(); - LoginWebPage::OnLoginSuccess($sAuthUser, 'internal', $_SESSION['login_mode']); + LoginWebPage::OnLoginSuccess(Session::Get('auth_user'), 'internal', Session::Get('login_mode')); } return LoginWebPage::LOGIN_FSM_CONTINUE; } diff --git a/application/loginexternal.class.inc.php b/application/loginexternal.class.inc.php index 759b7482b..a7055a14b 100644 --- a/application/loginexternal.class.inc.php +++ b/application/loginexternal.class.inc.php @@ -45,6 +45,7 @@ class LoginExternal extends AbstractLoginFSMExtension $iErrorCode = LoginWebPage::EXIT_CODE_WRONGCREDENTIALS; return LoginWebPage::LOGIN_FSM_ERROR; } + Session::Set('auth_user', $sAuthUser); } return LoginWebPage::LOGIN_FSM_CONTINUE; } @@ -53,8 +54,7 @@ class LoginExternal extends AbstractLoginFSMExtension { if (Session::Get('login_mode') == 'external') { - $sAuthUser = $this->GetAuthUser(); - LoginWebPage::OnLoginSuccess($sAuthUser, 'external', $_SESSION['login_mode']); + LoginWebPage::OnLoginSuccess(Session::Get('auth_user'), 'external', Session::Get('login_mode')); } return LoginWebPage::LOGIN_FSM_CONTINUE; } diff --git a/application/loginform.class.inc.php b/application/loginform.class.inc.php index f10b2263c..d8e5bc8ee 100644 --- a/application/loginform.class.inc.php +++ b/application/loginform.class.inc.php @@ -71,6 +71,7 @@ class LoginForm extends AbstractLoginFSMExtension implements iLoginUIExtension $iErrorCode = LoginWebPage::EXIT_CODE_WRONGCREDENTIALS; return LoginWebPage::LOGIN_FSM_ERROR; } + Session::Set('auth_user', $sAuthUser); } return LoginWebPage::LOGIN_FSM_CONTINUE; } @@ -82,17 +83,8 @@ class LoginForm extends AbstractLoginFSMExtension implements iLoginUIExtension { if (Session::Get('login_mode') == 'form') { - if (isset($_SESSION['auth_user'])) - { - // If FSM reenter this state (example 2FA) then the auth_user is not resubmitted - $sAuthUser = $_SESSION['auth_user']; - } - else - { - $sAuthUser = utils::ReadPostedParam('auth_user', '', 'raw_data'); - } // Store 'auth_user' in session for further use - LoginWebPage::OnLoginSuccess($sAuthUser, 'internal', Session::Get('login_mode')); + LoginWebPage::OnLoginSuccess(Session::Get('auth_user'), 'internal', Session::Get('login_mode')); } return LoginWebPage::LOGIN_FSM_CONTINUE; } diff --git a/application/loginurl.class.inc.php b/application/loginurl.class.inc.php index 2aab9bd22..554058481 100644 --- a/application/loginurl.class.inc.php +++ b/application/loginurl.class.inc.php @@ -60,6 +60,7 @@ class LoginURL extends AbstractLoginFSMExtension $iErrorCode = LoginWebPage::EXIT_CODE_WRONGCREDENTIALS; return LoginWebPage::LOGIN_FSM_ERROR; } + Session::Set('auth_user', $sAuthUser); } return LoginWebPage::LOGIN_FSM_CONTINUE; } @@ -68,8 +69,7 @@ class LoginURL extends AbstractLoginFSMExtension { if (Session::Get('login_mode') == 'url') { - $sAuthUser = utils::ReadParam('auth_user', '', false, 'raw_data'); - LoginWebPage::OnLoginSuccess($sAuthUser, 'internal', $_SESSION['login_mode']); + LoginWebPage::OnLoginSuccess(Session::Get('auth_user'), 'internal', Session::Get('login_mode')); } return LoginWebPage::LOGIN_FSM_CONTINUE; } diff --git a/application/loginwebpage.class.inc.php b/application/loginwebpage.class.inc.php index 10abc3cf3..0360d3ee5 100644 --- a/application/loginwebpage.class.inc.php +++ b/application/loginwebpage.class.inc.php @@ -112,7 +112,7 @@ class LoginWebPage extends NiceWebPage */ public static function SynchronizeProfiles(&$oUser, array $aProfiles, $sOrigin) { - $oProfilesSet = $oUser->Get(‘profile_list’); + $oProfilesSet = $oUser->Get('profile_list'); //delete old profiles $aExistingProfiles = []; while ($oProfile = $oProfilesSet->Fetch()) diff --git a/datamodels/2.x/itop-portal-base/portal/public/css/variables.scss b/datamodels/2.x/itop-portal-base/portal/public/css/variables.scss index 5bcce1d65..72d0a21aa 100644 --- a/datamodels/2.x/itop-portal-base/portal/public/css/variables.scss +++ b/datamodels/2.x/itop-portal-base/portal/public/css/variables.scss @@ -612,7 +612,7 @@ $popover-arrow-color: $popover-bg !default; //** Popover outer arrow width $popover-arrow-outer-width: ($popover-arrow-width + 1) !default; //** Popover outer arrow color -$popover-arrow-outer-color: fadein($popover-border-color, 5%) !default; +$popover-arrow-outer-color: fade-in($popover-border-color, 0.05) !default; //** Popover outer arrow fallback color $popover-arrow-outer-fallback-color: darken($popover-fallback-border-color, 20%) !default;