我的登录页面始终显示数据不正确。 找不到错误

Question

Here's my code:

<?php

try {
$connection = new PDO("mysql:host=localhost;dbname=tp1-2015_groep2", "utp1-2015_groep2", "bB5HvUC7MV7mqpWp");
$connection->query("SET NAMES utf8");
    echo "<p id='popupg'>Heeft u al een account? U kunt nu inloggen!</p>";
} catch (PDOException $exception) {
    echo "<p id='popupr'>U bent NIET verbonden! Probeer alstublieft opnieuw te verbinden!</p>";
    exit;
}



function hashedPassword($username, $password) {
    return hash ("sha512", $username.$password);
}


//zodra het login knop ingedrukt wordt..
if(isset($_POST['login'])){

    //Verkrijg de info binnen van onze formulier
    $username = $_POST['username'];
    $password = $_POST['password'];

    //Haal de geselecteerde velden op
    $sql = $connection->prepare ("SELECT id, username, password FROM gebruikersgegevens WHERE username = :username AND password = :password");


    $passwordHash = hashedPassword($username, $password);

    //Koppel de onderstaande waarden aan elkaar
    $sql->bindValue(':username', $username, PDO::PARAM_STR);
    $sql->bindValue(':password', $passwordHash, PDO::PARAM_STR);

    //Voer het voorbereidde query uit
    $sql->execute();
    if ($sql->execute()) {

    //Haal rij op..
    $sql->fetch();

    //kijken of de rijen kloppen..
    $count = $sql->rowCount();

            if ($count !== 0) {

        //Zodra count "true" geeft, vergelijk wachtwoorden..
            $validPassword = password_verify($password, $user['password']);
                if ($validPassword) {
                    //Bied de gebruiker een login sessie aan.
                        $_SESSION['user_id'] = $user['id'];
                        $_SESSION['logged_in'] = time();
                        $_SESSION['username'] = $_POST['username'];
                            //Stuurt de gebruiker door naar het onderstaande link..        
                            header('Location: index.php');
                            exit;
                    } else {
                        die('<p id="popupr">Het verifiëren is fout gegaan!</p>');
                }

        } else {
            //een tijdelijke error oplossing!
            echo '<p id="popupr">De combinatie van wachtwoord en gebruikersnaam klopt niet!</p>';

        }
    }
}

I have to make a blog for school where you need to be able to register and login. But the login page doesn't really work. Can't find the bug. I tried so many things, but nothing worked. It always says the combination to login is wrong. I already think where it goes wrong, but I can't find what's wrong. It's where the password is getting verified.

Answer 1

You're doing double work. For one you're retrieving rows from the table which match both the username and password and then later you're comparing the password which is hashed using a different hashing method to the hashed password. Later you're accessing $user which hasn't been set yet (You're using $sql->fetch() without assigning the result into a variable).

You also had a double $sql->execute() . Which (if it returns false) means something went wrong with the query, not that there aren't any rows.

Try the following,

<?php
session_start();
if(isset($_POST['login'])){

    //Verkrijg de info binnen van onze formulier
    $username = $_POST['username'];
    $password = $_POST['password'];

    //Haal de geselecteerde velden op
    $sql = $connection->prepare ("SELECT id, username, password FROM gebruikersgegevens WHERE username = :username AND password = :password");


    $passwordHash = hashedPassword($username, $password);

    //Koppel de onderstaande waarden aan elkaar
    $sql->bindValue(':username', $username, PDO::PARAM_STR);
    $sql->bindValue(':password', $passwordHash, PDO::PARAM_STR);

    //Voer het voorbereidde query uit
    if ($sql->execute()) {

        //Haal rij op.. (En sla op in $user)
        $user = $sql->fetch();

        //kijken of de rijen kloppen..
        $count = $sql->rowCount();

        if ($count > 0) {

            //Bied de gebruiker een login sessie aan.
            $_SESSION['user_id'] = $user['id'];
            $_SESSION['logged_in'] = time();
            $_SESSION['username'] = $_POST['username'];

            //Stuurt de gebruiker door naar het onderstaande link..        
            header('Location: index.php');
            exit;

        } else {
            die('<p id="popupr">Het verifiëren is fout gegaan!</p>');
        }

    } else {
        //een tijdelijke error oplossing!
        // Je komt alleen hier als de query mislukt is (fout in de query of database).
        echo '<p id="popupr">De combinatie van wachtwoord en gebruikersnaam klopt niet!</p>';

    }
}

Answer 2

You use password_verify() to verifiy password but you generate hash with hash() function. Try to change hash() to password_hash() . See password_verify() documentation . password_hash() using crypt() to generate password see password_hash documentation .