Skip to main content
se añadieron 382 caracteres en el cuerpo
Origen Enlace
Xerif
  • 7.4k
  • 3
  • 19
  • 42

Dado que guardas la conraseña encriptada, al buscar el usuario en la base de datos no debes hacerlo por la conraseña, ya que no van a coincidir la pass en texto plano con la pass encriptada debes hacerlo solo por el usuario, luego validar la pass. También le añadiria un LIMIT 1 al sqlSQL ya que el usuario debe ser único y no hay necesidad de seguir escaneando la tabla si ya se encontró un registro:

    public function obtenerUsuario($nombre, $clave){
        $db = Db::conectar();
        $usuario = new Usuario();

        // Ajustamos el SQL
        $select = $db->prepare('SELECT * FROM USUARIOS WHERE usuario=:usuario LIMIT 1');
        $select->bindValue('usuario', $nombre);
        // Esto sobra
        // $select->bindValue('pass',$clave);
        $select->execute();
        $registro = $select->fetch();

        // aquí es donde verificas la pass no en el SQL
        if (password_verify($clave, $registro['pass'])) { 
            //si es correcta, asigna los valores que trae desde la base de datos
            /*$usuario->setCodusua($registro['CodUsua']);*/
            $usuario->setNombre($registro['nombre']);
            $usuario->setUsuario($registro['usuario']);
            $usuario->setPass($registro['pass']); /*pass en mi bd*/
        }           
        return $usuario;
    }

También veo que en algunos lados utilizas el campo usuario=:usuario y en otros nombre=:nombre no se si tienes los 2 campos pero deberias de elegir cual de ellos se utilizará para iniciar sesion y debería ser un campo único si no lo es ya.

Dado que guardas la conraseña encriptada, al buscar el usuario en la base de datos no debes hacerlo por la conraseña, ya que no van a coincidir la pass en texto plano con la pass encriptada. También le añadiria un LIMIT 1 al sql ya que el usuario debe ser único:

    public function obtenerUsuario($nombre, $clave){
        $db = Db::conectar();
        $usuario = new Usuario();

        // Ajustamos el SQL
        $select = $db->prepare('SELECT * FROM USUARIOS WHERE usuario=:usuario LIMIT 1');
        $select->bindValue('usuario', $nombre);
        // Esto sobra
        // $select->bindValue('pass',$clave);
        $select->execute();
        $registro = $select->fetch();

        // aquí es donde verificas la pass no en el SQL
        if (password_verify($clave, $registro['pass'])) { 
            //si es correcta, asigna los valores que trae desde la base de datos
            /*$usuario->setCodusua($registro['CodUsua']);*/
            $usuario->setNombre($registro['nombre']);
            $usuario->setUsuario($registro['usuario']);
            $usuario->setPass($registro['pass']); /*pass en mi bd*/
        }           
        return $usuario;
    }

Dado que guardas la conraseña encriptada, al buscar el usuario en la base de datos no debes hacerlo por la conraseña, ya que no van a coincidir la pass en texto plano con la pass encriptada debes hacerlo solo por el usuario, luego validar la pass. También le añadiria un LIMIT 1 al SQL ya que el usuario debe ser único y no hay necesidad de seguir escaneando la tabla si ya se encontró un registro:

    public function obtenerUsuario($nombre, $clave){
        $db = Db::conectar();
        $usuario = new Usuario();

        // Ajustamos el SQL
        $select = $db->prepare('SELECT * FROM USUARIOS WHERE usuario=:usuario LIMIT 1');
        $select->bindValue('usuario', $nombre);
        // Esto sobra
        // $select->bindValue('pass',$clave);
        $select->execute();
        $registro = $select->fetch();

        // aquí es donde verificas la pass no en el SQL
        if (password_verify($clave, $registro['pass'])) { 
            //si es correcta, asigna los valores que trae desde la base de datos
            /*$usuario->setCodusua($registro['CodUsua']);*/
            $usuario->setNombre($registro['nombre']);
            $usuario->setUsuario($registro['usuario']);
            $usuario->setPass($registro['pass']); /*pass en mi bd*/
        }           
        return $usuario;
    }

También veo que en algunos lados utilizas el campo usuario=:usuario y en otros nombre=:nombre no se si tienes los 2 campos pero deberias de elegir cual de ellos se utilizará para iniciar sesion y debería ser un campo único si no lo es ya.

Origen Enlace
Xerif
  • 7.4k
  • 3
  • 19
  • 42

Dado que guardas la conraseña encriptada, al buscar el usuario en la base de datos no debes hacerlo por la conraseña, ya que no van a coincidir la pass en texto plano con la pass encriptada. También le añadiria un LIMIT 1 al sql ya que el usuario debe ser único:

    public function obtenerUsuario($nombre, $clave){
        $db = Db::conectar();
        $usuario = new Usuario();

        // Ajustamos el SQL
        $select = $db->prepare('SELECT * FROM USUARIOS WHERE usuario=:usuario LIMIT 1');
        $select->bindValue('usuario', $nombre);
        // Esto sobra
        // $select->bindValue('pass',$clave);
        $select->execute();
        $registro = $select->fetch();

        // aquí es donde verificas la pass no en el SQL
        if (password_verify($clave, $registro['pass'])) { 
            //si es correcta, asigna los valores que trae desde la base de datos
            /*$usuario->setCodusua($registro['CodUsua']);*/
            $usuario->setNombre($registro['nombre']);
            $usuario->setUsuario($registro['usuario']);
            $usuario->setPass($registro['pass']); /*pass en mi bd*/
        }           
        return $usuario;
    }