]> git.mxchange.org Git - mailer.git/blobdiff - inc/modules/admin/admin-inc.php
Further fixes for non-working admin login and debug lines commented out
[mailer.git] / inc / modules / admin / admin-inc.php
index 8b136e04a7204f21d9c43b8242aaee45519840b7..6a726a464d0147e080a948ddb609fa64719f7b6b 100644 (file)
@@ -42,22 +42,22 @@ if (!defined('__SECURITY')) {
 } // END - if
 
 // Register an administrator account
-function addAdminAccount ($user, $md5, $email) {
+function addAdminAccount ($adminLogin, $passHash, $adminEmail) {
        // Login does already exist
        $ret = 'already';
 
        // Lookup the admin
        $result = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_admins` WHERE `login`='%s' LIMIT 1",
-               array($user), __FUNCTION__, __LINE__);
+               array($adminLogin), __FUNCTION__, __LINE__);
 
        // Is the entry there?
        if (SQL_NUMROWS($result) == '0') {
                // Ok, let's create the admin login
                SQL_QUERY_ESC("INSERT INTO `{?_MYSQL_PREFIX?}_admins` (`login`, `password`, `email`) VALUES ('%s', '%s', '%s')",
                        array(
-                               $user,
-                               $md5,
-                               $email
+                               $adminLogin,
+                               $passHash,
+                               $adminEmail
                        ), __FUNCTION__, __LINE__);
 
                // All done
@@ -71,170 +71,72 @@ function addAdminAccount ($user, $md5, $email) {
        return $ret;
 }
 
-// Only be executed on login procedure!
-function ifAdminLoginDataIsValid ($admin, $password) {
-       // By default no admin is found
+// This function will be executed when the admin is not logged in and has submitted his login data
+function ifAdminLoginDataIsValid ($adminLogin, $adminPassword) {
+       // First of all, no admin login is found
        $ret = '404';
 
-       // Get admin id
-       $adminId = getAdminId($admin);
-
-       // Init array with admin id by default
-       $data = array('admin_id' => $adminId);
-
-       // Is the cache valid?
-       if (isAdminHashSet($admin)) {
-               // Get password from cache
-               $data['password'] = getAdminHash($admin);
-               $ret = 'pass';
-               incrementStatsEntry('cache_hits');
-
-               // Include more admins data?
-               if ((isExtensionInstalledAndNewer('admins', '0.7.2')) && (isset($GLOBALS['cache_array']['admin']['login_failures'][$adminId]))) {
-                       // Load them here
-                       $data['login_failures'] = $GLOBALS['cache_array']['admin']['login_failures'][$adminId];
-                       $data['last_failure']   = $GLOBALS['cache_array']['admin']['last_failure'][$adminId];
-               } // END - if
-       } elseif (!isExtensionActive('cache')) {
-               // Add extra data via filter now
-               $add = runFilterChain('sql_admin_extra_data');
+       // Then we need to lookup the login name by getting the admin hash
+       $adminHash = getAdminHash($adminLogin);
 
-               // Get password from DB
-               $result = SQL_QUERY_ESC("SELECT `password`" . $add . " FROM `{?_MYSQL_PREFIX?}_admins` WHERE `id`=%s LIMIT 1",
-                       array($adminId), __FUNCTION__, __LINE__);
-
-               // Entry found?
-               if (SQL_NUMROWS($result) == 1) {
-                       // Login password found
-                       $ret = 'pass';
+       // If this is fine, we can continue
+       if ($adminHash != '-1') {
+               // Get admin id and set it as current
+               setCurrentAdminId(getAdminId($adminLogin));
 
-                       // Fetch data
-                       $data = SQL_FETCHARRAY($result);
-               } // END - if
+               // Now, we need to encode the password in the same way the one is encoded in database
+               $testHash = generateHash($adminPassword, $adminHash);
 
-               // Free result
-               SQL_FREERESULT($result);
-       }
-
-       //* DEBUG: */ outputHtml('*' . $data['password'] . '/' . md5($password) .'/' . $ret . '*<br />');
-       if ((isset($data['password'])) && (strlen($data['password']) == 32) && ($data['password'] == md5($password))) {
-               // Generate new hash
-               $data['password'] = generateHash($password);
-
-               // Is the sql_patches not installed, than we cannot have a valid hashed password here!
-               //* DEBUG: */ outputHtml($ret . ',' . intval(isExtensionInstalledAndOlder('sql_patches', '0.3.6')) . '/' . intval(!isExtensionInstalled('sql_patches')).'<br />');
-               if (($ret == 'pass') && ((isExtensionInstalledAndOlder('sql_patches', '0.3.6')) || (!isExtensionInstalled('sql_patches')))) $ret = 'done';
-       } elseif ((isExtensionInstalledAndOlder('sql_patches', '0.3.6')) || (!isExtensionInstalled('sql_patches'))) {
-               // Old hashing way
-               return $ret;
-       } elseif (!isset($data['password'])) {
-               // Password not found, so no valid login!
-               return $ret;
-       }
-
-       // Generate salt of password
-       $salt = substr($data['password'], 0, -40);
-
-       // Check if password is same
-       //* DEBUG: */ outputHtml('*' . $ret . ',' . $data['password'] . ',' . $password . ',' . $salt . '*<br />');
-       if (($ret == 'pass') && ($data['password'] == generateHash($password, $salt)) && ((!empty($salt))) || ($data['password'] == md5($password))) {
-               // Re-hash the plain passord with new random salt
-               $data['password'] = generateHash($password);
-
-               // Do we have 0.7.0 of admins or later?
-               // Remmeber login failures if available
-               if ((isExtensionInstalledAndNewer('admins', '0.7.2')) && (isset($data['login_failures']))) {
-                       // Store it in session
-                       setSession('mxchange_admin_failures', $data['login_failures']);
-                       setSession('mxchange_admin_last_fail', $data['last_failure']);
-
-                       // Update password and reset login failures
-                       SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_admins` SET `password`='%s',`login_failures`=0,`last_failure`='0000-00-00 00:00:00' WHERE `id`=%s LIMIT 1",
-                               array($data['password'], $adminId), __FUNCTION__, __LINE__);
+               // If they both match, the login data is valid
+               if ($testHash == $adminHash) {
+                       // All fine
+                       $ret = 'done';
                } else {
-                       // Update password
-                       SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_admins` SET `password`='%s' WHERE `id`=%s LIMIT 1",
-                               array($data['password'], $adminId), __FUNCTION__, __LINE__);
+                       // Set status
+                       $ret = 'pass';
                }
-
-               // Rebuild cache
-               rebuildCacheFile('admin', 'admin');
-
-               // Login has failed by default... ;-)
-               $ret = 'failed1';
-
-               // Password matches so login here
-               if (doAdminLogin($admin, $data['password'])) {
-                       // All done now
-                       $ret = 'done';
-               } // END - if
-       } elseif ((empty($salt)) && ($ret == 'pass')) {
-               // Something bad went wrong
-               $ret = 'failed_salt';
-       } elseif ($ret == 'done') {
-               // Try to login here if we have the old hashing way (sql_patches not installed?)
-               if (!doAdminLogin($admin, $data['password'])) {
-                       // Something went wrong
-                       $ret = 'failed2';
-               } // END - if
-       }
-
-       // Count login failure if admins extension version is 0.7.0+
-       if (($ret == 'pass') && (getExtensionVersion('admins') >= '0.7.0')) {
-               // Update counter
-               SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_admins` SET login_failures`=`login_failures`+1,`last_failure`=NOW() WHERE `id`=%s LIMIT 1",
-                       array($adminId), __FUNCTION__, __LINE__);
-
-               // Rebuild cache
-               rebuildCacheFile('admin', 'admin');
        } // END - if
 
-       // Return the result
-       //* DEBUG: */ die('RETURN=' . $ret);
-       return $ret;
-}
-
-// Try to login the admin by setting some session/cookie variables
-function doAdminLogin ($adminLogin, $passHash) {
-       // Reset failure counter on matching admins version
-       if ((isExtensionInstalledAndNewer('admins', '0.7.0')) && ((isExtensionInstalledAndOlder('sql_patches', '0.3.6')) || (!isExtensionInstalled('sql_patches')))) {
-               // Reset counter on out-dated sql_patches version
-               SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_admins` SET `login_failures`=0, `last_failure`='0000-00-00 00:00:00' WHERE `login`='%s' LIMIT 1",
-                       array($adminLogin), __FUNCTION__, __LINE__);
+       // Prepare data array
+       $data = array(
+               'login'      => $adminLogin,
+               'plain_pass' => $adminPassword,
+               'pass_hash'  => $adminHash
+       );
 
-               // Rebuild cache
-               rebuildCacheFile('admin', 'admin');
-       } // END - if
+       // Run a special filter
+       runFilterChain('do_admin_login_' . $ret, $data);
 
-       // Now set all session variables and return the result
-       return ((
-               setSession('admin_md5', generatePassString(generateHash($passHash, '', false)))
-       ) && (
-               setSession('admin_login', $adminLogin)
-       ) && (
-               setSession('admin_last', time())
-       ));
+       // Return status
+       return $ret;
 }
 
 // Only be executed on cookie checking
-function ifAdminCookiesAreValid ($admin, $password) {
-       // By default no admin cookies are found
-       $ret  = '404';
-       $pass = '';
+function ifAdminCookiesAreValid ($adminLogin, $passHash) {
+       // First of all, no admin login is found
+       $ret = '404';
 
-       // Get hash
-       $pass = getAdminHash($admin);
-       if ($pass != '-1') $ret = 'pass';
+       // Then we need to lookup the login name by getting the admin hash
+       $adminHash = getAdminHash($adminLogin);
 
-       //* DEBUG: */ outputHtml(__FUNCTION__ . '(<font color="#0000aa">' . __LINE__."</font>):".generatePassString($pass).'('.strlen($pass).")/".$password.'('.strlen($password).")<br />");
+       // If this is fine, we can continue
+       if ($adminHash != '-1') {
+               // Now, we need to encode the password in the same way the one is encoded in database
+               $testHash = encodeHashForCookie($adminHash);
+               //* DEBUG: */ outputHtml('adminLogin='.$adminLogin.',<br />passHash='.$passHash.',<br />adminHash='.$adminHash.',<br />testHash='.$testHash.'<br />');
 
-       // Check if password matches
-       if (($ret == 'pass') && ((generatePassString($pass) == $password) || ($pass == $password) || ((strlen($pass) == 32) && (md5($password) == $pass))) && (isAdmin())) {
-               // Passwords matches!
-               $ret = 'done';
+               // If they both match, the login data is valid
+               if ($testHash == $passHash) {
+                       // All fine
+                       $ret = 'done';
+               } else {
+                       // Set status
+                       $ret = 'pass';
+               }
        } // END - if
 
-       // Return result
+       // Return status
+       //* DEBUG: */ outputHtml('ret='.$ret);
        return $ret;
 }
 
@@ -243,7 +145,7 @@ function doAdminAction () {
        // Get default what
        $what = getWhat();
 
-       //* DEBUG: */ outputHtml(__LINE__."*".$what.'/'.getModule().'/'.getAction().'/'.getWhat()."*<br />");
+       //* DEBUG: */ outputHtml(__LINE__.'*'.$what.'/'.getModule().'/'.getAction().'/'.getWhat().'*<br />');
 
        // Remove any spaces from variable
        if (empty($what)) {
@@ -659,7 +561,7 @@ function adminSaveSettings (&$postData, $tableName = '_config', $whereStatement
        $affected = SQL_AFFECTEDROWS();
 
        // Rebuild cache
-       rebuildCacheFile('config', 'config');
+       rebuildCache('config', 'config');
 
        // Settings saved, so display message?
        if ($displayMessage === true) loadTemplate('admin_settings_saved', false, getMessage('SETTINGS_SAVED'));
@@ -734,7 +636,7 @@ function generateUserProfileLink ($userid, $title = '', $what = 'list_user') {
 
 // Check "logical-area-mode"
 function adminGetMenuMode () {
-       // Set the global mode as the mode for all admins
+       // Set the default menu mode as the mode for all admins
        $mode = getConfig('admin_menu');
        $adminMode = $mode;
 
@@ -761,7 +663,7 @@ function adminGetMenuMode () {
                SQL_FREERESULT($result);
        }
 
-       // Check what the admin wants and set it when it's not the global mode
+       // Check what the admin wants and set it when it's not the default mode
        if ($adminMode != 'global') $mode = $adminMode;
 
        // Return admin-menu's mode
@@ -1243,17 +1145,8 @@ function adminResetValidateHashLogin ($hash, $login) {
 
 // Reset the password for the login. Do NOT call this function without calling above function first!
 function doResetAdminPassword ($login, $password) {
-       // Init hash
-       $passHash = '';
-
-       // Now check if we have sql_patches installed
-       if (isExtensionInstalledAndNewer('sql_patches', '0.3.6')) {
-               // Use new way of hashing
-               $passHash = generateHash($password);
-       } else {
-               // Old MD5 method
-               $passHash = md5($password);
-       }
+       // Generate hash (we already check for sql_patches in generateHash())
+       $passHash = generateHash($password);
 
        // Update database
        SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_admins` SET `password`='%s' WHERE `login`='%s' LIMIT 1",