Skip to content

Commit 8ce7eef

Browse files
committed
Rewrite process.php using PDO instead of mysqli.
Uses parameter binding to help reduce the risk of SQL injection.
1 parent 294f26b commit 8ce7eef

File tree

1 file changed

+15
-16
lines changed

1 file changed

+15
-16
lines changed

process.php

Lines changed: 15 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,18 @@
11
<?php
2-
error_reporting( E_ALL & ~E_DEPRECATED & ~E_NOTICE );
3-
// Establishing connection with server by passing "server_name", "user_id", "password" ,"database_name"
4-
$conn = mysqli_connect("server_name", "user_id", "password","database_name");
2+
error_reporting(E_ALL & ~E_DEPRECATED & ~E_NOTICE);
3+
try {
4+
$PDO = new \PDO('mysql:dbname=database_name;host=server_name;port=3306', 'user_id', 'password');
5+
} catch (\PDOException $Exception) {
6+
echo $Exception->getMessage();
7+
die;
8+
}
59

6-
//Fetching Values from URL
7-
$name =$_POST['Name'];
8-
$msg =$_POST['Message'];
10+
$Prepared = $PDO->prepare('INSERT INTO `formsub` (`name`, `msg`) values (:name, :msg)');
11+
if ($Prepared !== false) {
12+
$Query = $Prepared->execute([
13+
'name' => $_POST['Name'],
14+
'msg' => $_POST['Message']
15+
]);
16+
}
917

10-
//Insert query
11-
$sql = "insert into formsub (name,msg) values ('$name','$msg')";
12-
$query = mysqli_query( $conn,$sql);
13-
if($query){
14-
echo "Working good";
15-
}else echo "error";
16-
17-
// clossing connection
18-
mysqli_close($conn);
19-
?>
18+
echo $Query ? 'Working good' : 'Error';

0 commit comments

Comments
 (0)