This question already has an answer here:
I have some code to register that works, but I don't know how to hash the password. I want to use sha512.
$con=mysqli_connect("localhost","root","","users");
// Check connection
if (mysqli_connect_errno())
{
echo "Failed to connect to MySQL: " . mysqli_connect_error();
}
$sql="INSERT INTO users (username, password, email)
VALUES
('$_POST[username]','$_POST[password]','$_POST[email]')";
if (!mysqli_query($con,$sql))
{
die('Error: ' . mysqli_error($con));
}
echo "Thank you.";
mysqli_close($con);
I am aware of my mysql login having no password. This is a local mysql server just used for tests.
Sanitize your input data first. $password = filter_input(INPUT_POST, 'password', FILTER_SANITIZE_STRING);Hash your password string
A better way to hash the password would be to use the new password hash API
Please escape your data before entering into your database. They are open to attacks.
There are a lot of problems with what you're doing here.
First, you are vulnerable to SQL injections because you are not sanitizing your inputs in the SQL.
Second, you should avoid using a fast hash like SHA512 for this. It's not considered secure anymore. Take a look at this question. You basically want to use an adaptive hash function like bcrypt.
You can use the
hash()
function to hash your password:Then modify your insert statement to insert your hashed password into the database:
Be aware that your SQL statement is vulnerable to SQL injection since you are using unsanitized user input. For improved security and to protect the integrity of your data, please consider escaping and validating the input before using it in an SQL statement. One way to accomplish this is via
mysqli_real_escape_string()
:Here's an example of how to hash the password:
I recommend salting the password. Read more about this here:
http://www.aspheute.com/english/20040105.asp
Also read about "mysqli_real_escape_string"
…and Prepared Statements.