PHP Login Failed Attempt Lockout

JordanReich

Here is the session_inc.php I'm using

<?php
session_start();
$MM_authorizedUsers = "1,2,3,4,5";
$MM_donotCheckaccess = "false";

// *** Restrict Access To Page: Grant or deny access to this page
function isAuthorized($strUsers, $strGroups, $UserName, $UserGroup) {
// For security, start by assuming the visitor is NOT authorized.
$isValid = False;

// When a visitor has logged into this site, the Session variable MM_Username set equal to their username.
// Therefore, we know that a user is NOT logged in if that Session variable is blank.
if (!empty($UserName)) {
// Besides being logged in, you may restrict access to only certain users based on an ID established when they login.
// Parse the strings into arrays.
$arrUsers = Explode(",", $strUsers);
$arrGroups = Explode(",", $strGroups);
if (in_array($UserName, $arrUsers)) {
$isValid = true;
}
// Or, you may restrict access to only certain users based on their username.
if (in_array($UserGroup, $arrGroups)) {
$isValid = true;
}
if (($strUsers == "") && false) {
$isValid = true;
}
}
return $isValid;
}
$MM_restrictGoTo = "login.php";
if (!((isset($SESSION['MM_Username'])) && (isAuthorized("",$MM_authorizedUsers, $SESSION['MM_Username'], $SESSION['MM_AdminMainUserGroup'])))) {
$MM_qsChar = "?";
$MM_referrer = $SERVER['PHP_SELF'];
if (strpos($MM_restrictGoTo, "?")) $MM_qsChar = "&";
if (isset($QUERY_STRING) && strlen($QUERY_STRING) > 0)
$MM_referrer .= "?" . $QUERY_STRING;
$MM_restrictGoTo = $MM_restrictGoTo. $MM_qsChar . "accesscheck=" . urlencode($MM_referrer);
header("Location: ". $MM_restrictGoTo);
exit;
}
// Logout the current user.
$logoutAction = $SERVER['PHP_SELF']."?doLogout=true";
if ((isset($SERVER['QUERY_STRING'])) && ($SERVER['QUERY_STRING'] != "")){
$logoutAction .="&". htmlentities($SERVER['QUERY_STRING']);
}

if ((isset($GET['doLogout'])) &&($GET['doLogout']=="true")){
//to fully log out a visitor we need to clear the session varialbles
session_unregister('MM_Username');
session_unregister('MM_AdminMainUserGroup');

$logoutGoTo = "http://www.ktreich.com";
if ($logoutGoTo) {
header("Location: $logoutGoTo");
exit;
}
}
$duhdumduh = $_SESSION['MM_Username'];
$log_file = "log_users.txt";
$ip = getenv('REMOTE_ADDR');
$fp = fopen("$log_file", "a");
fputs($fp, "$duhdumduh - $ip\n");
flock($fp, 3);
fclose($fp);
//PRINT("<center>Your Ip was logged $duhdumduh .....$ip</center>");
?>

What is the easiest way and if you could provide a coded example I'd appreciate it as I've been messing with forever with no luck so far. I have added two additional lines to my MYSQL file.

last_failed_login_attempt
number_login_attempts

Basically just want something that counts with a username each time they login incorrect and ++ (addons one) until a certain number lets say 3. Than once you arrive at 3 than the limit is met and the account is just locked out. I can take care of the admin side as far as clearing the lockout. But for whatever reason this is stumping me.

Thanks!

anakadote

What I would do is store all of their login attempts and if they try to login unsuccessfully three times within a certain window, say 30 minutes, then they have to wait until their first login attempt goes beyond that first 30 min login window to try again.

So, have a database table, say "logins" with a primary key of "arbitrary_id", that stores login attempts. Then query the database for all login attempts within the 30 min window. If the number of results equals 3 or more then deny access.

SELECT COUNT(arbitrary_id) FROM logins WHERE user_id=$user_id AND DATE_SUB(CURDATE(), INTERVAL 30 MINUTE) <= last_login;

"last_login" is the DATETIME value of when they tried to login