简体   繁体   中英

php login don't work

I'm working on a login system and I want, In case the user entered wrong username & password to return him to the login page and if its ok redirect to admin/index.php it's not working and no errors .

Function code And the html code

    if(isset($_POST['submit_form'])){
        $username = mysqli_real_escape_string($_POST['username']);
        $password = mysqli_real_escape_string($_POST['password']);
        $query = mysqli_query("SELECT * FROM users WHERE username = '$username' AND   password ='{$password'");
        if($query==false){
            set_message(" Your password or username are wrong ");
            redirect("login.php");
        }else{
            redirect("admin");
        }
    }

The Html code

<form class="form-inline" role="form" action="" method="post" enctype="multipart/form-data">
    <div class="form-group">        
        <label class="form-group" for="username">username</label>
        <input type="text" name="username" class="form-control" id="username" placeholder="username">
    </div>
    <div class="form-group">
        <label class="sr-only" for="password">Password</label>
        <input type="password" name="password" class="form-control" id="password" placeholder="password">
    </div>
    <button type="submit" name="submit_form" class="btn btn-black">
    <input type="hidden" name="submit_form" value="<?php echo session_id()"/>
        Login
    </button>
</form>

The functions

function redirect($location) {
    header("Location : $location"); 
}

function query($sql){
    global $connection ;
    return mysqli_query($connection, $sql);
}

function confirm($result){
    global $connection;
    if(!$result){
        die("QUERY FAILED " . mysqli_error($connection));
    } 
}

function escape_string($string){
    global $connection ;
    return mysqli_real_escape_string($connection, $string);
} 

Yes, sorry i am just getting used to this thing. Thanks for pointing out the mistake.

    //this will eliminate the BOT hitting up the forms
    if(isset($_POST['submit_form']) && $_POST['submit_form']==session_id()){
    $username = mysql_real_escape_string($_POST['username']);
    $password = mysql_real_escape_string($_POST['password']);
    $query = mysqli_query("SELECT * FROM users WHERE username = '$username' AND   password ='$password'");
    if($query==false){
        set_message(" Your password or username are wrong ");
        redirect("login.php");
    }else{
        redirect("admin");
    }
}

And I updated the form as well.

  <form class="form-inline" role="form" action="" method="post" enctype="multipart/form-data">
<div class="form-group">        
    <label class="form-group" for="username">username</label>
    <input type="text" name="username" class="form-control" id="username" placeholder="username">
</div>
<div class="form-group">
    <label class="sr-only" for="password">Password</label>
    <input type="password" name="password" class="form-control" id="password" placeholder="password">
</div>
<button type="submit" name="submit_form" class="btn btn-black">

    Login
</button> <input type="hidden" name="submit_form" value="<?php echo session_id()"/>

I would use sprintf when parsing up the query. It eliminates SQL Injections. You can do something like this:

      $query = sprintf('SELECT * FROM TABLE WHERE username = "%s" AND password = "%s"',
      mysql_real_escape_string($username),
      mysql_real_escape_string($password));

Hopefully this works for you.

The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM