* [MOD] Improved login when the webserver auth is used. Related #180

nuxsmin · Dec 24, 2015 · 6ce7ade · 6ce7ade
1 parent 03c7ebe
commit 6ce7ade
Show file tree

Hide file tree

Showing 5 changed files with 162 additions and 102 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,5 +1,6 @@
 === ** v1.2.0.10 ===
 
+* [MOD] Improved login when the webserver auth is used.
 * [MOD] Updated Russian translation. Thanks to @TitovLab
 * [FIX] Fixed user id checking instead login when using webserver auth. Thanks to @TitovLab
 * [FIX] Integer casting for boolean values on users management. Thanks to @maseht

diff --git a/CHANGELOG-ES b/CHANGELOG-ES
@@ -1,6 +1,7 @@
 === ** v1.2.0.10 ===
 
-* [ADD] Actualizada traducción al ruso. Gracias a @TitovLab
+* [MOD] Mejorado el login cuando la autentificación del servidor web es usada.
+* [MOD] Actualizada traducción al ruso. Gracias a @TitovLab
 * [FIX] Corregida la comprobación del usuario cuando se usa la autentificación del servidor. Gracias a @TitovLab
 * [FIX] Casting a entero desde booleano en gestion de usuarios. Gracias a @maseht
 * [FIX] Corregido truncado de cadenas multibyte. Gracias a @TitovLab

diff --git a/ajax/ajax_doLogin.php b/ajax/ajax_doLogin.php
@@ -23,6 +23,7 @@
  *
  */
 
+use SP\Auth;
 use SP\CryptMasterPass;
 use SP\Request;
 use SP\SessionUtil;
@@ -63,7 +64,7 @@
 // Autentificamos por LDAP
 if ($resLdap === true) {
     $Log->addDescription('(LDAP)');
-    $Log->addDescription(sprintf('%s : %s', _('Servidor Login'), \SP\Ldap::getLdapServer()));
+    $Log->addDescription(sprintf('%s: %s', _('Servidor Login'), \SP\Ldap::getLdapServer()));
 
     // Verificamos si el usuario existe en la BBDD
     if (!UserLdap::checkLDAPUserInDB($userLogin)) {
@@ -86,21 +87,21 @@
 } else if ($resLdap == 49) {
     $Log->addDescription('(LDAP)');
     $Log->addDescription(_('Login incorrecto'));
-    $Log->addDescription(_('Usuario') . ": " . $userLogin);
+    $Log->addDescription(sprintf('%s: %s', _('Usuario'), $userLogin));
     $Log->writeLog();
 
     SP\Response::printJSON(_('Usuario/Clave incorrectos'));
 } else if ($resLdap === 701) {
     $Log->addDescription('(LDAP)');
     $Log->addDescription(_('Cuenta expirada'));
-    $Log->addDescription(_('Usuario') . ": " . $userLogin);
+    $Log->addDescription(sprintf('%s: %s', _('Usuario'), $userLogin));
     $Log->writeLog();
 
     SP\Response::printJSON(_('Cuenta expirada'));
 } else if ($resLdap === 702) {
     $Log->addDescription('(LDAP)');
     $Log->addDescription(_('El usuario no tiene grupos asociados'));
-    $Log->addDescription(_('Usuario') . ": " . $userLogin);
+    $Log->addDescription(sprintf('%s: %s', _('Usuario'), $userLogin));
     $Log->writeLog();
 
     SP\Response::printJSON(_('Usuario/Clave incorrectos'));
@@ -111,17 +112,27 @@
     // Autentificamos con la BBDD
     if (!SP\Auth::authUserMySQL($userLogin, $userPass)) {
         $Log->addDescription(_('Login incorrecto'));
-        $Log->addDescription(_('Usuario') . ": " . $userLogin);
+        $Log->addDescription(sprintf('%s: %s', _('Usuario'), $userLogin));
         $Log->writeLog();
 
         SP\Response::printJSON(_('Usuario/Clave incorrectos'));
     }
 }
 
+// Comprobar si concide el login con la autentificación del servidor web
+if (!Auth::checkServerAuthUser($userLogin)){
+    $Log->addDescription(_('Login incorrecto'));
+    $Log->addDescription(sprintf('%s: %s', _('Usuario'), $userLogin));
+    $Log->addDescription(sprintf('%s: %s (%s)', _('Autentificación'), Auth::getServerAuthType(), Auth::getServerAuthUser()));
+    $Log->writeLog();
+
+    SP\Response::printJSON(_('Usuario/Clave incorrectos'));
+}
+
 // Comprobar si el usuario está deshabilitado
 if (UserUtil::checkUserIsDisabled($userLogin)) {
     $Log->addDescription(_('Usuario deshabilitado'));
-    $Log->addDescription(_('Usuario') . ": " . $userLogin);
+    $Log->addDescription(sprintf('%s: %s', _('Usuario'), $userLogin));
     $Log->writeLog();
 
     SP\Response::printJSON(_('Usuario deshabilitado'));
@@ -171,9 +182,9 @@
     // Cargar las variables de sesión del usuario
     SessionUtil::loadUserSession($User);
 
-    $Log->addDescription(sprintf('%s : %s', _('Usuario'), $userLogin));
-    $Log->addDescription(sprintf('%s : %s', _('Perfil'), SP\Profile::getProfileNameById($User->getUserProfileId())));
-    $Log->addDescription(sprintf('%s : %s', _('Grupo'), SP\Groups::getGroupNameById($User->getUserGroupId())));
+    $Log->addDescription(sprintf('%s: %s', _('Usuario'), $userLogin));
+    $Log->addDescription(sprintf('%s: %s', _('Perfil'), SP\Profile::getProfileNameById($User->getUserProfileId())));
+    $Log->addDescription(sprintf('%s: %s', _('Grupo'), SP\Groups::getGroupNameById($User->getUserGroupId())));
     $Log->writeLog();
 } else {
     SP\Response::printJSON(_('Error interno'));

diff --git a/inc/Auth.class.php b/inc/Auth.class.php
@@ -220,4 +220,44 @@ public static function checkAuthToken($actionId, $token)
 
         return (DB::$lastNumRows === 1);
     }
+
+    /**
+     * Comprobar si el usuario es autentificado por el servidor web
+     *
+     * @param $login string El login del usuario a comprobar
+     * @return bool
+     */
+    public static function checkServerAuthUser($login)
+    {
+        if (isset($_SERVER['PHP_AUTH_USER']) || isset($_SERVER['REMOTE_USER'])) {
+            return self::getServerAuthUser() == $login;
+        }
+
+        return true;
+    }
+
+    /**
+     * Devuelve el typo de autentificación del servidor web
+     * @return string
+     */
+    public static function getServerAuthType()
+    {
+        return strtoupper($_SERVER['AUTH_TYPE']);
+    }
+
+    /**
+     * Devolver el nombre del usuario autentificado por el servidor web
+     *
+     * @return string
+     */
+    public static function getServerAuthUser()
+    {
+        if (isset($_SERVER['PHP_AUTH_USER'])) {
+            return $_SERVER['PHP_AUTH_USER'];
+        } elseif (isset($_SERVER['REMOTE_USER'])) {
+            return $_SERVER['REMOTE_USER'];
+        }
+
+        return '';
+    }
 }