problema con logeo de contraseña y usuario

[rufus]

hola

que estoy haciendo un logeo en una aplicacion con nombre y contraseña. la contraseña la encripto con sha1 y en el script de login.php meto los usuarios con su contraseña que he creado pero me dice . me da el error usuario/contraseña no coinciden. es por que tengo que poner algun codigo extra por lo del "sha1" de encriptacion? os dejo el codigo.

necesito ayuda :P

login.php:

Código PHP:

  <?php 
require_once("../../includes/initialize.php"); 
if($sesion->esta_logueado()) 
{ 
    redireccionar_a("admin.php"); 
} 
?> 
<?php 
if(isset($_POST["submit"])) 
{ 
    $username = $_POST["username"]; 
    $password = $_POST["password"]; 
     
    $usuario= Usuario::autenticar($username,$password); 
     
     
 
    if($usuario) 
    { 
        global $sesion; 
        $sesion->loguearse($usuario); 
        grabar_acciones("logueo", "el usuario ".$usuario->id . " se ha logueado."); 
        redireccionar_a("admin.php"); 
    } 
    else 
    { 
        $mensaje = "usuario/clave no coinciden."; 
         
    } 
} 
?> 
<?php include("admin_header.php"); ?> 
 <table id="estructura"> 
    <tr> 
      <td id="menu"> 
       
      </td> 
      <td id="pagina"> 
          <h2>Administración</h2> 
   <p>    <?php echo isset($mensaje) ? $mensaje : "" ; ?> </p> 
        <form action="login.php" method="post"> 
        <table> 
            <tr> 
                <td>Nombre de usuario:</td> 
                <td><input type="text" name="username" /></td> 
            </tr> 
            <tr> 
                <td>Contraseña:</td> 
                <td><input type="password" name="password" /></td> 
            </tr> 
        </table> 
        <input type="submit" value="Ingresar" name="submit"/> 
        </form> 
     </td> 
    </tr> 
  </table> 
  
<?php include("admin_footer.php"); ?>   

session.php

Código PHP:

  <?php 
 
class Sesion 
{ 
    public $id; 
    public $usuario; 
    private $logueado = false; 
     
    function __construct() 
    { 
        session_start(); 
        $this->verificar_logueo(); 
    } 
     
    public function esta_logueado() 
    { 
         
        return $this->logueado; 
         
    } 
     
    public function loguearse($usuario) 
    { 
        if($id) 
        { 
            $this->id = $_SESSION["id"] = $usuario->id; 
            $this->usuario = $_SESSION["usuario"] = $usuario->usuario; 
            $this->logueado = true; 
        } 
    } 
     
    public function desloguearse() 
    { 
        unset($this->id); 
        unset($_SESSION["id"]); 
        $this->logueado = false; 
         
    } 
    private function verificar_logueo() 
    { 
        if(isset($_SESSION["id"])) 
        { 
            $this->id =  $_SESSION["id"]; 
            $this->usuario = $_SESSION["usuario"]; 
            $this->logueado = true; 
        } 
        else 
        { 
            unset($this->id); 
            $this->logueado = false; 
        } 
    } 
} 
 
$sesion = new Sesion(); 
?>   

[rufus]

Para crear el usuario:

create.php

Código PHP:

  <?php require("../../includes/session.php");?> 
<?php //verificar_sesion(); ?> 
 
<?php require_once("../../includes/initialize.php"); ?> 
 
 
<?php 
    if(isset($_POST["username"])) 
    { 
        $errores = array(); 
        $errores = array_merge($errores, validar_campos_obligatorios(array("username","password"))); 
        $max_caracteres = array("username" => 50,"password" => 40); 
        foreach($max_caracteres as $campo => $max) 
        { 
            if(strlen($_POST[$campo])>$max) 
            { 
                $errores[] = $campo;     
            }             
        } 
         
        $username =trim(preparar_consulta($_POST["username"])); 
        $password = sha1(trim(preparar_consulta($_POST["password"]))); 
        //$nombre = trim(preparar_consulta($_POST["nombre"])); 
//        $apellido = trim(preparar_consulta($_POST["apellido"])); 
         
        if(empty($errores)) 
        { 
            $consulta = "INSERT INTO usuarios ( 
                            username, password) VALUES ( 
                            '{$username}','{$password}')"; 
            $resultado = mysql_query($consulta); 
            if($resultado) 
            { 
                $mensaje = "El usuario ha sido creado."; 
            } 
            else 
            { 
                $mensaje = "No se ha podido crear el usuario: " . mysql_error(); 
            } 
        } 
        else 
        { 
            $mensaje = "Se han encontrado " . count($errores) . " errores"; 
        } 
    } 
?> 
<?php include("admin_header.php"); ?> 
  <table id="estructura"> 
    <tr> 
      <td id="menu"> 
      <a href="admin.php">Regresar al menú principal</a> 
      </td> 
      <td id="pagina"> 
          <h2>Crear nuevo usuario</h2> 
        <?php if(isset($mensaje)) { echo "<p>" . $mensaje . "</p>"; } ?> 
        <form action="create.php" method="post"> 
        <table> 
            <tr> 
                <td>Nombre de usuario:</td> 
                <td><input type="text" name="username" /></td> 
            </tr> 
            <tr> 
                <td>Contraseña:</td> 
                <td><input type="password" name="password" /></td> 
            </tr>            
        </table> 
        <input type="submit" value="Crear usuario" /> 
        </form> 
     </td> 
    </tr> 
  </table> 
<?php require_once("admin_footer.php"); ?>   

user.php

Código PHP:

  <?php 
require_once(LIB_DIR.SD."database.php"); 
 
class Usuario 
{ 
    public $id; 
    public $username; 
    public $password; 
     
     
 
 
    function validar_campos_obligatorios($campos_obligatorios) 
    { 
        $errores = array(); 
        foreach($campos_obligatorios as $campo) 
        {             
            if(!isset($_POST[$campo]) || (empty($_POST[$campo]) && !is_numeric($_POST[$campo]))) 
            { 
                $errores[] = $campo; 
            }     
        } 
        return $errores; 
    } 
     
    public static function buscar_por_id($id) 
    { 
        global $bd; 
        $matriz_usuarios = self::buscar_por_sql("SELECT * FROM    usuarios WHERE id={$id}"); 
        return (!empty($matriz_usuarios))? array_shift($matriz_usuarios) : false; 
    } 
     
     
    public static function buscar_todos() 
    { 
        return self::buscar_por_sql(" SELECT * FROM    usuarios"); 
         
     
         
    } 
     
    public static function buscar_por_sql($sql) 
    { 
        global $bd; 
        $resultado = $bd->enviar_consulta($sql); 
        $matriz_usuarios = array(); 
        while($registro = $bd->fetch_array($resultado)) 
        { 
            array_push($matriz_usuarios,Usuario::instanciar($registro)); 
        } 
        return $matriz_usuarios; 
    } 
     
    public static function instanciar($registro) 
    { 
        $usuario = new Usuario(); 
     
        foreach ($registro as $propiedad => $valor) 
        { 
            if ($usuario->propiedad_existe($propiedad)) 
            { 
                $usuario->$propiedad = $valor; 
            } 
        } 
        return $usuario; 
    } 
     
    public function nombre_completo() 
    { 
        if(isset($this->nombre) && isset($this->apellido)) 
        { 
            return $this->nombre . " " . $this->apellido; 
        } 
        else 
        { 
            return ""; 
        } 
    } 
     
     
    public function propiedad_existe($propiedad) 
    { 
        $propiedades = get_object_vars($this); 
        return array_key_exists($propiedad, $propiedades); 
    } 
     
    public static function autenticar($username = "", $password= "") 
    { 
        global $bd; 
        $username = $bd->preparar_consulta($username); 
        $password = $bd->preparar_consulta($password); 
         
        $sql = "SELECT * FROM usuarios "; 
        $sql .= " WHERE username='{$username}' "; 
        $sql .=" AND password='{$password}' "; 
        $sql .=" LIMIT 1"; 
         
        $matriz_usuarios = Usuario::buscar_por_sql($sql); 
        return  (!empty($matriz_usuarios)) ? array_shift($matriz_usuarios) : false ; 
    } 
     
} 
     
 
?>   

[rufus]

estara el problema en session.php

en la funcion

Código PHP:

      private function verificar_logueo() 
    { 
        if(isset($_SESSION["id"])) 
        { 
            $this->id =  $_SESSION["id"]; 
            $this->username = $_SESSION["username"]; 
            $this->logueado = true; 
        } 
        else 
        { 
            unset($this->id); 
            $this->logueado = false; 
        } 
    }  
   

nota: he cambiado las variables $_SESSION usuario a username

[rufus]

o el fallo esta en la encriptacion sha1 que en login.php tengo que poner algun codigo para descifrar la contraseña?

[Uncontroled_Duck]

Lo he mirado así un poco rápido. Al guardar la pass lo guardas mediante

Código PHP:

  $password = sha1(trim(preparar_consulta($_POST["password"])  
   

Y al realizar la comparación, estás comparando la pass que viene por POST (sin sha1), con la que está guardada en la DB que la has guardado con sha1.

Código PHP:

  //archivo login 
$username = $_POST["username"];  
$password = $_POST["password"];  
      
$usuario= Usuario::autenticar($username,$password);  
   

En alguna parte de aquí debes igualar el proceso de hasheado antes de pasarlo a la consulta para que compare las dos contraseñas exactamente iguales.

Código PHP:

  //class Usuario 
    public static function autenticar($username = "", $password= "")  
    {  
        global $bd;  
        $username = $bd->preparar_consulta($username);  
        $password = $bd->preparar_consulta($password);  
          
        $sql = "SELECT * FROM usuarios ";  
        $sql .= " WHERE username='{$username}' ";  
        $sql .=" AND password='{$password}' ";  
        $sql .=" LIMIT 1";  
          
        $matriz_usuarios = Usuario::buscar_por_sql($sql);  
        return  (!empty($matriz_usuarios)) ? array_shift($matriz_usuarios) : false ;  
    }  
   

Prueba y nos cuentas,

[SetheR]

Yo un fallo que veo así mirando por encima está aquí.

Código PHP:

  public function loguearse($usuario)  
    {  
        if($id)  
        {  
            $this->id = $_SESSION["id"] = $usuario->id;  
            $this->usuario = $_SESSION["usuario"] = $usuario->usuario;  
            $this->logueado = true;  
        }  
    }  
   

donde $id no existe,con lo cual el método no hace nada

[rufus]

creo que lo e solucionado. (creo). estos son los cambios que le he hecho al codigo con los consejos que me habeis dado:

login.php

Código PHP:

  $username = $_POST["username"]; 
//en la linea de abajo e metido sha1() 
$password = sha1($_POST["password"]);     
    $usuario= Usuario::autenticar($username,$password);  
   

session.php

Código PHP:

  public function loguearse($username) 
    { 
        if($id) 
        { 
            // esta es la linea que he modificado poniendo $id  $this->id = $_SESSION["id"] = $id->id; 
            $this->usuario = $_SESSION["username"] = $usuario->username; 
            $this->logueado = true; 
        } 
    }