Home / Questions / 192534
Accepted
Karla
Asked: 2020-08-30 10:02:06 +0800 CST 2020-08-30 10:02:06 +0800 CST

Avoid sending password reset email to a non-active user?

  • 772

I tried resetting the password of an inactive user, who still hasn't verified his email, and when I send that form, it tells me to check the email to reset the password, but it shouldn't send the email because the user is inactive.

So how can I add another personalized message, if the user is not active, tell him, please activate your account, if you have trouble activating, resend the verification in the following form (a link).

if (empty($_POST['email'])) {
    $email_require = 'Por favor, ingrese su correo electrónico';
} else {
    if (preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", $_POST['email'])) {
        $email = $_POST['email'];

        $active = 1;
        $stmt = $con->prepare("SELECT email, first_name, email_code FROM users WHERE email=? OR username=? AND active=?");
        $stmt->bind_param("ssi",$email,$username,$active);
        $stmt->execute();
        $stmt->store_result();
        //como los datos que pasas en los inputs concuerdan con la Base de datos te tare 1 fila con lo datos que pediste
        //es decir la condicion es verdadera
        if ($stmt->num_rows>0) {
          //aqui debes hacer el proceso de recuperacion 
            $stmt->bind_result($email_user, $first_name, $email_code);
            $stmt->fetch();

            echo '¡Por favor, revise su correo electrónico '.$email_user.' para obtener un enlace de confirmación para completar su restablecimiento de contraseña!';

            $to = $email_user;
            $subject = "Proceso para restablecer su contraseña";
            $message_body = 'Hola '.$first_name.',
            ¡Has solicitado restablecimiento de contraseña!

            Por favor, haga clic en este enlace para restablecer su contraseña.

            http://example.com/login-system/reset.php?email='.urlencode($email).'&key='.$email_code.'';
            mail($to, $subject, $message_body, 'From: [email protected]');
        } else {
          //como los campos que envio estan vacion, entonces la base de datos nunca retornara ninguna fila
          //y es aqui donde captura el error 
            echo "¡Usuario con ese correo electrónico no existe!";
        }
    } else {
        echo 'Tu dirección de correo electrónico no es válida';
    }
}
}
php

2 Answers

  1. Best Answer
    • We must modify the statement SELECTto retrieve the column [active].
    • Once the value of the column is retrieved [active], evaluate its value.

    Here is the implemented solution:

    <?

if (empty($_POST['email'])) {
    $email_require = 'Por favor, ingrese su correo electrónico';
} else {
    if (preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", $_POST['email'])) {
        $email = $_POST['email'];

        $stmt = $con->prepare("SELECT email, first_name, email_code, active FROM users WHERE email=? OR username=?");
        $stmt->bind_param("ssi",$email,$username);
        $stmt->execute();
        $stmt->store_result();
        //como los datos que pasas en los inputs concuerdan con la Base de datos te tare 1 fila con lo datos que pediste
        //es decir la condicion es verdadera
        if ($stmt->num_rows>0) {
          //aqui debes hacer el proceso de recuperacion 
            $stmt->bind_result($email_user, $first_name, $email_code, $active);
            $stmt->fetch();

            // Usuario no activo?
            if (!$active) {
               echo 'Por favor active su cuenta. Si tiene problema en activar reenvié la verificación en el siguiente formulario. http://example.com/login-system/activate.php?email='.urlencode($email);
            }
            // Recuperar contraseña
            else
            {
                    echo '¡Por favor, revise su correo electrónico '.$email_user.' para obtener un enlace de confirmación para completar su restablecimiento de contraseña!';

                    $to = $email_user;
                    $subject = "Proceso para restablecer su contraseña";
                    $message_body = 'Hola '.$first_name.',
                    ¡Has solicitado restablecimiento de contraseña!

                    Por favor, haga clic en este enlace para restablecer su contraseña.

                    http://example.com/login-system/reset.php?email='.urlencode($email).'&key='.$email_code.'';
                    mail($to, $subject, $message_body, 'From: [email protected]');
            }

        } else {
          //como los campos que envio estan vacion, entonces la base de datos nunca retornara ninguna fila
          //y es aqui donde captura el error 
            echo "¡Usuario con ese correo electrónico no existe!";
        }
    } else {
        echo 'Tu dirección de correo electrónico no es válida';
    }
}
}
    • 1

  2. In my experience I would reserve three fields in the users table for this case:

    • (boolean) active - To mark if a user is active/inactive (ie can login).
    • (boolean) validate - To mark if a user has verified their email.
    • (text) token - To store hash of reset password/validate email.

    Then:

    1. When registering a user a hash is saved in token . Set active =1 and validate =0. (The user can already login but his email has not been validated).
    2. The user clicks on the email validation link. Mark token =null, activate =1 and validate =1.
    3. User forgets his password: A token is saved , activate =1 and validate =1. (If the case of token!=null and validate=1 occurs, we can already identify this case in a query)
    4. User login must be prohibited: mark active =0.
    5. The user edits/changes his email account in his profile: a token is saved , activate =1 and validate =0.

    I hope it works for you, greetings.

    • 1