2

Quisiera saber si pueden ustedes identificar cuál es el inconveniente que presento en este código. La línea del error es: "header("location: index.php");". No sé si se debe a que está mal cerrada con los ";". Aún no he podido solucionar el problema. Estoy usando XAMPP para la realización de las pruebas. Es un código simple para saber si logra conectar con la base de datos y después ir trabajándolo. Gracias por la ayuda!

<?php
    session_start();
    include_once "config/config.php";
     
    function verificar_login($dni,$password,&$result) {
        $sql = "SELECT dni, password FROM members WHERE dni = '$dni' and password = '$password'";
        $rec = mysql_query($sql);
        $count = 0;
     
        while($row = mysql_fetch_object($rec))
        {
            $count++;
            $result = $row;
        }
     
        if($count == 1)
        {
            return 1;
        }
     
        else
        {
            return 0;
        }
    }
     
    if(!isset($_SESSION['dniid']))
    {
        if(isset($_POST['login']))
        {
            if(verificar_login($_POST['dni'],$_POST['password'],$result) == 1)
            {
                $_SESSION['dniid'] = $result->dniid
                header("location: index.php");
            }
            else
            {
                echo "<div class="error">Su DNI es incorrecto, intente nuevamente.</div>";
            }
        }
    ?>
     
    <form action="" method="post" class="login">
        <div><label>DNI</label><input name="dni" type="text" ></div>
        <div><label>Password</label><input name="password" type="password"></div>
        <div><input name="login" type="submit" value="login"></div>
    </form>
    <?php
    } else {
        echo 'Su usuario ingreso correctamente.';
        echo '<a href="logout.php">Logout</a>';
    }

También adjunto el código de conexion.php que está programado en formato PDO. La idea es que en base al código de conexión en PDO se pudiera adaptar el código de login.php que está hecho en mysqli.

<?php
//Archivo config para Producción
ob_start();
session_start();

//set timezone
date_default_timezone_set('America/Argentina/Buenos_Aires');

//database credentials
##para conectarse a la BBDD del servidor de produccion##
define('DBHOST','localhost');
define('DBPORT','3306');
define('DBUSER','root');
define('DBPASS','123');
define('DBNAME','siriodb');

//application address
##para uso en el servidor de desarrollo local##
define('DIR','https://www.test.com.ar/');
define('SITETITLE','Login');
define('SITEEMAIL','[email protected]');
define('SITECUSTOMER','Pagina test'); // Ej Nombre de empresa

//Conexion con setAttribute(PDO::ATTR_EMULATE_PREPARES, false); utilizada para la mayoria de las consultas

try {
    //create PDO connection
    $db = new PDO("mysql:host=".DBHOST.";port=".DBPORT.";charset=utf8mb4;dbname=".DBNAME, DBUSER, DBPASS);
    //$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_SILENT);//Suggested to uncomment on production websites
    $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);//Suggested to comment on production websites
    $db->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
    //$db->setAttribute(PDO::ATTR_USE_BUFFERED_QUERY, true);

} catch(PDOException $e) {
    //show error
    //echo '<p class="bg-danger">'.$e->getMessage().'</p>';
    echo "Fallo al conectar a la BBDD Remota (db1)";
    exit;
}
10
  • Te falta cerrar la línea anterior con punto y coma:$_SESSION['dniid'] = $result->dniid;
    – A. Cedano
    Commented el 7 jul. 2021 a las 12:41
  • Sí, lo he hecho pero me tira otro error en el código, en este caso: syntax error, unexpected identifier "error", expecting "," or ";" En la linea echo "<div class="error">Su DNI es incorrecto, intente nuevamente.</div>";
    – Francisco
    Commented el 7 jul. 2021 a las 12:45
  • 1
    mysql_query esta obsoleto, deberías usar PDO o mysqli_
    – rpaillao
    Commented el 7 jul. 2021 a las 12:49
  • Sí, tenes mucha razón. Estoy trabajando con PHP 7 y debería comenzar a usar mysqli, lo corregiré. Muchas gracias :)
    – Francisco
    Commented el 7 jul. 2021 a las 12:52
  • Muestra lo que tienes en config.php, pues no se sabe cómo obtienes la conexión, si es mediante un método, si es directamente, etc. En los comentarios pusiste un bloque try para conectar, pero falta contexto.
    – A. Cedano
    Commented el 8 jul. 2021 a las 17:10

1 respuesta 1

0

Hay dos errores en tu código:

  1. Te falta cerrar la línea anterior al header location con punto y coma: $_SESSION['dniid'] = $result->dniid;
  2. En las cadenas con dobles comillas, debes escapar la comillas interiores con \, aquí no lo estás haciendo: echo "<div class="error">Su DNI es incorrecto, intente nuevamente.</div>"; Así el error desaparecería: echo "<div class=\"error\">Su DNI es incorrecto, intente nuevamente.</div>"; porque escapas las comillas que hay para class="error".

Por otra parte, si estás usando PDO, el código debe ser adaptado a PDO. De igual modo, el archivo donde conectas a PDO tiene que tener alguna forma de acceder a la conexión. Yo he creado en dicho archivo una función getConnection() que te retorna sea el objeto $db (que es la conexión) o null en caso de fallo.

El archivo sería así:

<?php
  //Archivo config para Producción
  function getConnection()
  {
    ob_start();
    session_start();

    //set timezone
    date_default_timezone_set('America/Argentina/Buenos_Aires');

    //database credentials
    ##para conectarse a la BBDD del servidor de produccion##
    define('DBHOST','localhost');
    define('DBPORT','3306');
    define('DBUSER','root');
    define('DBPASS','123');
    define('DBNAME','siriodb');

    //application address
    ##para uso en el servidor de desarrollo local##
    define('DIR','https://www.test.com.ar/');
    define('SITETITLE','Login');
    define('SITEEMAIL','[email protected]');
    define('SITECUSTOMER','Pagina test'); // Ej Nombre de empresa

    //Conexion con setAttribute(PDO::ATTR_EMULATE_PREPARES, false); utilizada para la mayoria de las consultas

    try {
        //create PDO connection
        $db = new PDO("mysql:host=".DBHOST.";port=".DBPORT.";charset=utf8mb4;dbname=".DBNAME, DBUSER, DBPASS);
        //$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_SILENT);//Suggested to uncomment on production websites
        $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);//Suggested to comment on production websites
        $db->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
        //$db->setAttribute(PDO::ATTR_USE_BUFFERED_QUERY, true);
        return $db;

    } catch(PDOException $e) {
        //show error
        //echo '<p class="bg-danger">'.$e->getMessage().'</p>';
        echo "Fallo al conectar a la BBDD Remota (db1)";
        return NULL;
        exit;
    }
  }

Ahora adaptamos el código a PDO y de paso usamos consultas preparadas pues tu código no es seguro por el simple hecho de usar PDO. La seguridad consiste en no pasar datos externos directamente en las consultas.

<?php
    session_start();
    include_once "config/config.php"; 
    function verificar_login($dni,$password,&$result) 
    {
        $pdo=getConnection();
        if ($pdo) 
        {
          $sql = "SELECT COUNT(*) FROM members WHERE dni = :dni and password = :password";
          $stmt = $pdo->prepare($sql);
          $stmt->bindParam(":dni", $dni, PDO::PARAM_STR);
          $stmt->bindParam(":password", $password, PDO::PARAM_STR);
          $stmt->execute();
          $count = $stmt->fetchColumn();
          return ($count > 0);
        } 
        else 
        {
          echo "Error en la conexión";
          return false;
        }
        
    }
     
    if(!isset($_SESSION['dniid']))
    {
        if(isset($_POST['login']))
        {
            if(verificar_login($_POST['dni'],$_POST['password'],$result) > 0)
            {
                $_SESSION['dniid'] = $result->dniid;
                header("location: index.php");
            }
            else
            {
                echo "<div class=\"error\">Su DNI es incorrecto, intente nuevamente.</div>";
            }
        }
    }
    ?>     
    <form action="" method="post" class="login">
        <div><label>DNI</label><input name="dni" type="text" ></div>
        <div><label>Password</label><input name="password" type="password"></div>
        <div><input name="login" type="submit" value="login"></div>
    </form>

Edit

He simplificado el código en los siguientes aspectos:

  1. Para verificar si hay registros es mejor hacer una consulta del tipo SELECT COUNT(*) ... la cual traerá la cantidad de filas que existan con ese criterio.
  2. Dado que este tipo de consultas arroja una sola fila, no hace falta recorrer las filas e ir incrementando un contador como hacías en tu código original. Simplemente puedes obtener los resultados de la columna con fetchColumn().
  3. Luego puedes retornar true o false con algo así: return ($count > 0). Esto retornará true si la cantidad de filas es mayor que cero o false en caso contrario.
  4. En la llamada a la función hemos cambiado esto: if(verificar_login($_POST['dni'],$_POST['password'],$result) == 1) por esto: if(verificar_login($_POST['dni'],$_POST['password'],$result)), porque hemos hecho que verificar_login() devuelva un booleano true o false sin más. Si la idea es comprobar que existen registros, no debes comparar contra 1, porque el código no funcionará si por algún motivo hay más de una fila y eso podría ocurrir en algunos contextos.
  5. Finalmente, los echo que hay después del form no tienen ningún sentido. En caso de ingresar correctamente tú estás haciendo un location más arriba, eso redireccionará a otra página, por lo que no estarás ya en el contexto actual para mostrar ese contenido. Si quieres mostrar mensajes o lo que sea tendrás que pasarlos a la página a la que redireccionas o replantear la forma de organizar el código en cuanto a las redirecciones.

Nota sobre la verificación de claves.

En los comentarios salió el tema de password_hash() y password_verify(), pero no has aclarado nada al respecto. Si al guardar el password creaste un hash y lo guardaste en la BD, ahora debes usar password_verify() para comprobar. Si no sabes cómo funciona consulta la documentación o revisa esta respuesta.

17
  • Increíble, esto funcionó al 100%. Agradezco enormemente tu ayuda :).
    – Francisco
    Commented el 7 jul. 2021 a las 12:51
  • @Francisco, me alegro de que lo hayas resuelto. Considera también simplificar tu código, hay muchas cosas repetitivas e innecesarias en el mismo.
    – A. Cedano
    Commented el 7 jul. 2021 a las 16:02
  • Gracias por tu consejo. Honestamente estoy teniendo conflictos en el login. Cuando pongo el DNI y contraseña me salta DNI incorrecto cuando todas sus credenciales están bien colocadas.
    – Francisco
    Commented el 8 jul. 2021 a las 11:02
  • Debe ser por lo que me tratas de decir, que hay cosas innecesarias y repetitivas.
    – Francisco
    Commented el 8 jul. 2021 a las 11:02
  • @Francisco revisa la respuesta con una propuesta de código mejorada. Al final he agregado una sección Edit con algunas cosas.
    – A. Cedano
    Commented el 8 jul. 2021 a las 13:08

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.