[SOLUCIONADO] Al dejar los campos vacios no aparece nada

diegoj9999 · #1 (**permalink**) 04/12/2015, 16:04

Como dice el titulo , cuando dejo el campo vacio no me aparece nada, cuando en el usuario pongo algo si me aparece pero si no pongo el usuario no me aparece nada, osea la pagina en blanco, vacia , aca dejo el codigo

Código PHP:

Ver originalif ($_POST['username']) { 
//Comprobacion del envio del nombre de usuario y password 
$username=$_POST['username']; 
$password=$_POST['password']; 
if ( $username==NULL || $password == NULL ) { 
echo "La password o el usuario  no fueron enviados"; 
}else{ 
$query = mysql_query("SELECT username,password FROM users WHERE username = '$username'") or die(mysql_error()); 
$row = mysql_fetch_array($query); 
$logeado = 0;
$pass = crypt($password,"$2y$");
if ($row['password']==$pass){
echo "Te has logeado exitosamente";
$logeado++;
echo "<br>Contrasenia encriptada[".$pass."]";
if($logeado == 1){
echo "<br>ESTADO: En linea";
 
 
?>
<html>
<head>
<title>Redireccion</title>
<meta http-equiv="Refresh" content="5;URL=http://localhost/TEST/LoginSystem/Hola.php
">
</head>
<body>
Sera redirigido en unos segundos...
</body>
</html>
<?php
}
}
else
{
 
 
echo "La clave que has ingresado no es la correcta.";
echo "<br>En crypt devolvio [".$pass."]";
}
}

pateketrueke · #2 (**permalink**) 04/12/2015, 16:26

Cita:

Como dice el titulo , cuando dejo el campo vacio no me aparece nada, cuando en el usuario pongo algo si me aparece pero si no pongo el usuario no me aparece nada, osea la pagina en blanco, vacia , aca dejo el codigo

Claro, y seguramente piensas que somos adivinos o brujos como para ejecutar el código mentalmente y decirte exactamente qué ha fallado ¿no?

¿Y si mejor revisas el log de errores en busca de algo más útil?

herzbazi · #3 (**permalink**) 04/12/2015, 16:46

no entiendo exactamente que quieres lograr con todo el codigo que tienes , si quieres hacer un login algo como este te funciona de maravilla, aclarandote que falta seguridad en tu consulta

Código PHP:

Ver original<?php
    if (isset($_POST['username']) && !empty($_POST['username']) &&
        isset($_POST['password']) && !empty($_POST['password'])){
            $username=$_POST['username'];
            $password=$_POST['password'];
 
            $query = mysql_query("SELECT username,password FROM users WHERE username = '$username'") or die(mysql_error());
            $row = mysql_fetch_array($query);
            $NumReg= mysql_num_rows($row);
            if($NumReg == 1){
                $_SESSION['usuario'] = $row['tipo_usario'];
                header("Location: pagina.php");
            }elseif ($NumReg == 0) {
                header("Location: index.php?m=1");
            }
 
        }else{
            echo "La password o el usuario  no fueron enviados"; 
        }
    }
?>

busca sprintf y mysqli_real_escape_string para darle seguridad pues asi como la tienes facil de entrar a tu sistema sin contar un user y pass

diegoj9999 · #4 (**permalink**) 04/12/2015, 19:53

Cita:

Iniciado por herzbazi

no entiendo exactamente que quieres lograr con todo el codigo que tienes , si quieres hacer un login algo como este te funciona de maravilla, aclarandote que falta seguridad en tu consulta

Código PHP:

Ver original<?php
    if (isset($_POST['username']) && !empty($_POST['username']) &&
        isset($_POST['password']) && !empty($_POST['password'])){
            $username=$_POST['username'];
            $password=$_POST['password'];
 
            $query = mysql_query("SELECT username,password FROM users WHERE username = '$username'") or die(mysql_error());
            $row = mysql_fetch_array($query);
            $NumReg= mysql_num_rows($row);
            if($NumReg == 1){
                $_SESSION['usuario'] = $row['tipo_usario'];
                header("Location: pagina.php");
            }elseif ($NumReg == 0) {
                header("Location: index.php?m=1");
            }
 
        }else{
            echo "La password o el usuario  no fueron enviados"; 
        }
    }
?>

busca sprintf y mysqli_real_escape_string para darle seguridad pues asi como la tienes facil de entrar a tu sistema sin contar un user y pass

Hola, no entendí mucho del real escape string, no me quedo claro que es y para que se usa, en el manual entendi como se usa pero no entendi para que se usa, casi yendome del tema este codigo lo pase bien a mysqli? , estaba en mysql

Código PHP:

  
<?php

// Configura los datos de tu cuenta 
$dbhost='localhost'; 
$dbusername='root'; 
$dbuserpass=''; 
$dbname='database'; 
// Conexión a la base de datos 
$conexion = mysqli_connect($dbhost, $dbusername, $dbuserpass, $dbname); 

if (mysqli_connect_errno())
  {
  echo "Failed to connect to MySQL: " . mysqli_connect_error();
  exit();
  }

// Preguntaremos si se han enviado ya las variables necesarias 
if (isset($_POST["username"])) { 
$username = $_POST["username"]; 
$password = $_POST["password"]; 
$cpassword = $_POST["cpassword"]; 
$email = $_POST["email"]; 



// Hay campos en blanco 
if($username==NULL|$password==NULL|$cpassword==NULL|$email==NULL) { 
echo "un campo está vacio."; 
}else{ 
// ¿Coinciden las contraseñas? 
if($password!=$cpassword) { 
echo "Las contraseñas no coinciden"; 
}else{
$pass = crypt($password,"$2y$");

// Comprobamos si el nombre de usuario o la cuenta de correo ya existían 
$checkuser = mysqli_query($conexion,"SELECT username FROM users WHERE username='$username'"); 
$username_exist = mysqli_num_rows($checkuser); 
$checkemail = mysqli_query($conexion,"SELECT email FROM users WHERE email='$email'"); 
$email_exist = mysqli_num_rows($checkemail); 

if ($email_exist>0|$username_exist>0) { 
echo "EL nombre de usuario o la cuenta de correo estan ya en uso"; 
}else{ 

$user = mysqli_real_escape_string($conexion,$username);
$pass1 = mysqli_real_escape_string($conexion,$pass);


//Todo parece correcto procedemos con la inserccion 
$query = mysqli_query($conexion,"INSERT INTO users (username, password, email) VALUES('$user','$pass1','$email')"); 
echo "El usuario $username ha sido registrado de manera satisfactoria."; 
} 
} 
} 
}

mysqli_close($conexion);
  

?>

Y lo de sprintf: Daría mas seguridad usarlo?

herzbazi · #5 (**permalink**) 04/12/2015, 20:13

si colocas esto en tu login

en usuario Pegaso y en contraseña ' OR ''='

veras de lo que hablo te pongo (IGUAL TE LOGUEA)

ejemplo de la seguridad de tu select

Código PHP:

Ver original$checkusere = sprintf("SELECT  username   FROM users WHERE username= '%s' 
            mysql_real_escape_string($conexion,$_POST['user']),
            mysql_real_escape_string($conexion,$_POST['pass']));
$result = mysql_query($checkusere);
$username_exist = mysqli_num_rows($result);

y continuas con el resto de tu codigo.

me cuentas

diegoj9999 · #6 (**permalink**) 04/12/2015, 21:22

Cita:

Iniciado por herzbazi

si colocas esto en tu login

en usuario Pegaso y en contraseña ' OR ''='

veras de lo que hablo te pongo (IGUAL TE LOGUEA)

ejemplo de la seguridad de tu select

Código PHP:

Ver original$checkusere = sprintf("SELECT  username   FROM users WHERE username= '%s' 
            mysql_real_escape_string($conexion,$_POST['user']),
            mysql_real_escape_string($conexion,$_POST['pass']));
$result = mysql_query($checkusere);
$username_exist = mysqli_num_rows($result);

y continuas con el resto de tu codigo.

me cuentas

Hola, Gracias , ya me funcionó, dejo el codigo para los que los necesitan y si por si lo quieres ver tambien

Código PHP:

    if (isset($_POST['username']) && !empty($_POST['username']) && isset($_POST['password']) && !empty($_POST['password'])){

            $username=$_POST['username'];

            $password=$_POST['password'];

            $user = mysqli_real_escape_string($conexion,$username);

            $query = sprintf("SELECT username,password FROM users WHERE username = '%s'",$user);

            $result = mysqli_query($conexion,$query);

            $username_exist = mysqli_num_rows($result);

            $row = mysqli_fetch_array($result, MYSQLI_ASSOC);

            $pass = mysqli_real_escape_string($conexion,$_POST['password']);

            $pass1 = crypt($pass,"$2y$");

            if($username_exist == 1 && $row['password']==$pass1){

               echo "Bienvenido";

            }elseif ($username_exist == 0) {

                echo "El usuario no existe";

            }

            elseif($row['password']!=$pass)

            {

            echo "La contraseña es erronea";

            }

 

        }else{

            echo "La password o el usuario  no fueron enviados"; 

        }