Question

我的Codeigniter（3.0版）登录模块中有这个方法。它工作正常，但这样安全吗？使用PHP password_verify检查登录名和密码是否有更好的解决方案？（PHP 5.6， MySQL 5.0）。

        $user = $this->input->post('username');
        $password = $this->input->post('password');
        $myquery = $this->db->query("SELECT * FROM users WHERE user = '$user'");
        $row = $myquery->row();

        if (isset($row))
        {
            //Using hashed password - PASSWORD_BCRYPT method - from database
            $hash = $row->password;


            if (password_verify($password, $hash)) {

                echo 'Password is valid!';


            } else {

                echo 'Invalid password.';

            }


        } else{

            echo 'Wrong user!';
        }

Answer 1

你的代码看起来很好，但你可以用CI方式做得更多一点，更清洁，在这种情况下，你受到sql注入的保护，你有更多的封装

尝试这样的事情：

public function checkLogin()
{
    $this->load->library("form_validation");

    $arrLoginRules = array
    (
        array(
            "field" => "username",
            "label" => "Benutzername",
            "rules" => "trim|required"
        ),
        array(
            "field" => "password",
            "label" => "Passwort",
            "rules" => "trim|required"
        )

    );

    $this->form_validation->set_rules($arrLoginRules);

    try
    {
        if (!$this->form_validation->run()) throw new UnexpectedValueException(validation_errors());

        $user = $this->input->post('username');
        $password = $this->input->post('password');
        $query = $this->db
            ->select("*")
            ->from("users")
            ->where("user", $user)
            ->get();

        if ($query->num_rows() != 1)    throw new UnexpectedValueException("Wrong user!");

        $row = $query->row();
        if (!password_verify($password, $row->hash)) throw new UnexpectedValueException("Invalid password!");

        echo "valid user";

    }
    catch(Excecption $e)
    {
        echo $e->getMessage();
    }
}

有关更多信息，请查看Form validation 和Query Builder文档。

Codeigniter password_verify方法

1 个答案: