Problemas con funcion login

[Skorge]

Buenas a todos.

Gente tengo una function login que quiero implementar por que me parecio bastante interesante, pero ya la he implementado la he volteado de pies a cabeza y no quiere funcionar. Lo curioso es que no me muestra un error de Ejecucion ni de sintaxis si no de validacion echo "solicitud invalida" Pero ya lo he cambiado y descambiado he leido miles de veces los codigos y todo esta igual,*Lo se es una muy mala practica tomar codigos, pero quiero aprender de un modo en que Pruebo y aprendo* y no se ya voy 3 dias en esto y no doy con el error Gracias Chicos quien me pueda hechar un Cable

*Codigos*
Funcion

Código PHP:

function login($username, $password, $mysqli){
    // usar delacaraciones preparadas significa que la inyeccion de SQL no sera posible
    if ($stmt = $mysqli->prepare("SELECT Nombre, Apellidos, password FROM usuario WHERE username = ? LIMIT 1")){
        $stmt->bind_param('s', $username); // Une $email a el parametro.
        $stmt->execute();
        $stmt->store_result();
        
        // Obitne las variables del resultado
        $stmt->bind_result($username, $Nombre, $Apellidos, $password);
        $stmt->fetch();
        
        //Hace le HASH de la contraseña con una sal unica.
        $password = hash('sha512', $password);
        if ($stmt->num_rows == 1){
            // Si el usuario existe, revisa si la cuenta esta bloqueada
            // Por muchos intentos de conexion
            
            if (checkbrute($username, $mysqli) == true) {
                // La cuenta est bloqueada
                // Obten el agente de usuario del usuario
                $user_browser = $_SERVER['HTTP_USER_AGENT'];
                // Proteccion XSS ya que podriamos imprimir este valor.
                $username = preg_replace("/[^0-9]+/", "", $username);
                $_SESION['username'] = $username;
                // Proteccion XSS ya que podriamos imprimir este valor.
                $username = preg_replace("/[^a-zA-Z0-9_\-]+", "", $username);
                $_SESION['username'] = $username;
                $_SESSION['login_string'] = hash('sha512', $password . $user_browser);
                // inicio de sesion exitoso
                return true;
            } else {
                // La contraseña no es correcta 
                // Se graba este intento en la BASE DE DATOS.
                $now = time();
                $mysqli->query("INSERT INTO inyeccion(username, time) VALUE ('$username','$now)");
                return false;
                }
            }
        } else {
        // Vos no xistis boludo
        return false;
        }
    }
 

Login

Código PHP:

<?php
// muestra los errores de ejecucion que tengo
error_reporting(E_ERROR | E_WARNING | E_PARSE);
 
error_reporting(E_ERROR | E_WARNING | E_PARSE | E_NOTICE);
?>
<?php
include_once 'db_connect.php';
include_once 'function.php';
 
sec_session_start(); 
 
if (isset($_POST['username'], $_POST['p'])){
    $username = $_POST['username'];
    $password = $_POST['p']; // Contraseña con el HASH
 
if(login($username, $password, $mysqli) == true) {
    // Inicio de session exitosa
    header('Location: ../protected_page.php');
    exit();
    } else {
    header('Location: ../index.php?error=1');
    exit();
    }
} else {
    // Las variables POST correctas no se enviaron a esta pagina.
    echo 'Solicitud invalida';
}
?>
 

[xerifandtomas]

Y donde se valida la contraseña exactamente? por que no lo veo por ningun lado, no se si en la funcion checkbrute() puede ser que este el problema, pero no muestras lo que hace lo que hace.

[Skorge]

Gracias amigo por responder, no subi la funcion del Checkbrute por que no queria saturar esto con tanto codigo y me mandaran a el cuerno :/

Código PHP:

if (checkbrute($username, $mysqli) == true) {
                // La cuenta est bloqueada
                // Obten el agente de usuario del usuario
                $user_browser = $_SERVER['HTTP_USER_AGENT'];
                // Proteccion XSS ya que podriamos imprimir este valor.
                $username = preg_replace("/[^0-9]+/", "", $username);
                $_SESION['username'] = $username;
                // Proteccion XSS ya que podriamos imprimir este valor.
                $username = preg_replace("/[^a-zA-Z0-9_\-]+", "", $username);
                $_SESION['username'] = $username;
                $_SESSION['login_string'] = hash('sha512', $password . $user_browser);
                // inicio de sesion exitoso
                return true;
            } else {
                // La contraseña no es correcta 
                // Se graba este intento en la BASE DE DATOS.
                $now = time();
                $mysqli->query("INSERT INTO inyeccion(username, time) VALUE ('$username','$now)");
                return false;
                }
            }
        } else {
        // Vos no xistis boludo
        return false;
        }
    }
 

Esta es la parte que sigue y donde se valida el Password es hay mismo en esta funcion esta en la parte del Login_string Gracias por responder amigo

[xerifandtomas]

Pues sigo sin verlo

Si te refieres a esto

Código PHP:

$_SESSION['login_string'] = hash('sha512', $password . $user_browser);
 

No veo ahí ninguna comparación, solo una asignacion.

Intenta poner echo en cada uno de los if, para saber que partes se ejecutan y cuales no.


Ejemplo

Código PHP:

if (checkbrute($username, $mysqli) == true) {
/* resto de codigo */
                echo 'dentro de: if checkbrute()';
                return true;
            } else {
/* resto de codigo */
             echo 'dentro de: else checkbrute()';
             return false;
                }
 

[Skorge]

No muestra nada amigo :/, es mas no muestra ni si quiera un mensaje personaliazado .... Esta da que pensar

Lo casual es que este codigo funciona pero dirigido hacia otra base de datos, :/ y obvio con otras variables que no sean las de username