Estimados buenas noches. Estoy realizando un sistema de login con PHP y MYSQL. El objetivo principal es que se valide el número de intentos fallidos cuando el usuario ingresa mal la contraseña, a partir del intento numero 3 se debe mostrar un captcha que debe resolver el usuario. Al intento 5 se bloquea al usuario por sesión y por el momento a modo de prueba se le da un tiempo de bloqueo en la base de datos de 3 minutos. Por el momento solo muestro mensajes para verificar que se estén realizando las validaciones.
Mi problema resulta ser que después del intento numero 3 pongo el captcha en true para que este se muestre, pero al realizar la validación sigue estando en false, entonces no se en que línea debo poner el captcha en false por defecto, para que solo cambie a true dependiendo del número de intentos. Segundo problema, resulta ser que estoy repitiendo muchas veces la parte del switch, donde se verifica si la cuenta esta activa, inactiva, bloqueada (solo si ya expiro el tiempo de bloqueo) o deshabilitada. Ya que realizo esa validación tanto si el captcha es requerido o si no es requerido.
Y, por último, si la contraseña es incorrecta debo igual validar el captcha si es requerido o no, ¿y si este está bien ingresado?
<?php
session_set_save_handler(new \SessionHandler());
session_start();
include dirname(__FILE__).'/logica/conexion/conexion.php';
date_default_timezone_set("America/Santiago");
function intentoFallido($con, $usuario, $intentos) {
$mensaje = ['mensaje' => 'Usuario o contraseña inválidos'];
$tiempobloqueo = 0;
if($intentos < 2){
$numero_intentos = $con->actualizar("intentos = intentos + 1","usuarios","correo = ?",[$usuario]);
}
if($intentos >= 2) {
$numero_intentos = $con->actualizar("intentos = intentos + 1","usuarios","correo = ?",[$usuario]);
$captcha = true;
$mensaje = ['mensaje' => 'Mostrar captcha' , 'error' => 'Tras varios intentos de acceso fallidos, debe resolver el captcha correctamente'];
}
if($intentos >= 5) {
$tiempobloqueo = time() + 180; // Hora actual + 3 minutos
$_SESSION['tiempobloqueo'] = $tiempobloqueo;
$fechabloqueo = date('Y-m-d H:i:s', $tiempobloqueo);
$captcha = true;
$numero_intentos = $con->actualizar("intentos = intentos + 1, tiempobloqueo = ?, estado = ?", "usuarios","correo = ?",[$fechabloqueo, 'Bloqueada', $usuario]);
/* $mensaje = ' Su cuenta ha sido bloqueada temporalmente; podrá volver a intentar hasta: ' . date('d-m-Y H:i', $_SESSION['tiempobloqueo']);*/
$mensaje = ['mensaje' => 'Ocultar formulario'];
}
return $mensaje;
}
$con = new Conexion();
$mensaje = ['mensaje' => ''];
if(isset($_POST['submit'])){
if(empty($_POST['usuario']) || empty($_POST['clave'])){
$mensaje = ['mensaje' => "Por favor ingrese un usuario y contraseña validos"];
}else{
if(isset($_SESSION['tiempobloqueo']) && $_SESSION['tiempobloqueo'] > time()) {
$mensaje = ['mensaje' => 'Ocultar formulario'];
}else{
$usuario = filter_var(filter_input(INPUT_POST,'usuario'),FILTER_SANITIZE_STRING);
$clave = filter_var(filter_input(INPUT_POST,'clave'),FILTER_SANITIZE_STRING);
$datos = $con->obtenerdatos("correo,clave,nombre,apellido,sid,idusuario,intentos,tiempobloqueo,estado","usuarios","correo = ?",array($usuario));
$mensaje = ['mensaje' => ''];
$captcha = false;
if(count($datos) > 0){
$fila = $datos[0];
$db_clave = $fila['clave'];
$db_intentos = $fila['intentos'];
$db_bloqueo = strtotime($fila['tiempobloqueo']);
if($db_intentos > 5 && $db_bloqueo > time()){
$mensaje = ['mensaje' => 'Su cuenta sigue bloqueda'];
}else{
if($mensaje['mensaje'] == '' && password_verify($clave,$db_clave)){
if($captcha == true){
$mensaje = ['mensaje' => 'Se debe validar captcha'];
$valor_captcha = $_SESSION['captcha_text'];
$captcha_ingresado = filter_var(filter_input(INPUT_POST,'text-captcha'),FILTER_SANITIZE_STRING);
if($valor_captcha == $captcha_ingresado){
switch ($fila['estado']){
case 'Activa':
$mensaje = ['mensaje' => 'Login correcto'];
break;
case 'Bloqueada':
$mensaje = ['mensaje' => 'Cuenta bloqueda, pero puede iniciar sesion'];
break;
case 'Inactiva':
$mensaje = ['mensaje' => 'Su cuenta aún no a sido activada, revise su bandeja de correo electrónico donde encontrará el link de activación'];
break;
case 'Deshabilitada':
$mensaje = ['mensaje' => 'Su cuenta se encuentra deshabilitada por haber infringido las normas de nuestro sitio, para más detalles póngase en contacto con soporte@ecuservicechile.cl'];
break;
}
}else{
$mensaje = ['mensaje' => 'Captcha incorrecto'];
}
}else{
switch ($fila['estado']){
case 'Activa':
$mensaje = ['mensaje' => 'Login correcto'];
break;
case 'Bloqueada':
$mensaje = ['mensaje' => 'Cuenta bloqueda, pero puede iniciar sesion'];
break;
case 'Inactiva':
$mensaje = ['mensaje' => 'Su cuenta aún no a sido activada, revise su bandeja de correo electrónico donde encontrará el link de activación'];
break;
case 'Deshabilitada':
$mensaje = ['mensaje' => 'Su cuenta se encuentra deshabilitada por haber infringido las normas de nuestro sitio, para más detalles póngase en contacto con soporte@ecuservicechile.cl'];
break;
}
}
}else{
$mensaje = intentoFallido($con, $usuario, $db_intentos);
}
}
}else{
$mensaje = ['mensaje' => 'Usuario o contraseña inválidos'];
}
}
}
}
Adjunto el layout del formulario
<form action="" method="post" autocomplete="new-password">
<div class="form-row justify-content-center mt-3">
<?php
if($mensaje['mensaje'] != 'Ocultar formulario'){
?>
<div class="col-lg-10 col-md-9 mb-3">
<label class="label-log" for="usuario">USUARIO</label>
<input type="text" class="form-control form-control-log" name="usuario" id="usuario" style="text-transform:upperca;" onkeyup="javascript:this.value=this.value.toLowerCase();" spellcheck="false" autocomplete="new-password">
</div>
<div class="col-lg-10 col-md-9 mb-3">
<label class="label-log" for="clave">CONTRASEÑA</label>
<input type="password" class="form-control form-control-log" name="clave" id="clave" spellcheck="false" autocomplete="new-password">
<i class="icon-eye mostrar-clave ver-clave"></i>
</div>
<div class="col-lg-10 col-md-9 mb-3">
<input type="submit" class="btn btn-success w-100" value="Ingresar" name="submit">
</div>
<?php
}
switch($mensaje['mensaje']){
case 'Mostrar captcha':
?>
<div class="col-lg-5">
<img src="prueba7.php" alt="CAPTCHA" class="w-100">
</div>
<div class="col-lg-5">
<label class="label-log" for="text-captcha">INGRESE EL CAPTCHA</label>
<input type="text" class="form-control form-control-log" name="text-captcha" id="text-captcha" style="text-transform:uppercase;" onkeyup="javascript:this.value=this.value.toUpperCase();" spellcheck="false" autocomplete="new-password">
</div>
<div class="col-lg-10">
<p class="text-white mt-2" id="error"><?php echo $mensaje['error'];?></p>
</div>
<?php
break;
default:
?>
<div class="col-lg-10">
<p class="text-white mt-2" id="error"><?php echo $mensaje['mensaje'];?></p>
</div>
<?php
break;
}
?>
</div>
</form>
$captcha = ($intentos >= 3) ? true : false;
, si los intentos son igual a 3 o superior se pone en true, mientras seria false. Una alternativa seria usar sesiones para mostrar/ocultar el captcha.$captcha = ($db_intentos >= 3) ? true : false;
después de$db_bloqueo
de esta manera la variable $captcha se pone en true o false según los intentos fallidos obtenido desde la BD, y ya debería valorarte si hay 3 intentos fallidos o másif($captcha == true){