requête SQL en php

Marsh Posté le 07-04-2020 à 15:57:22

Bonjour j'ai réussi à faire une requête INSERT pour s'inscrire mais j'essaie de faire un système de changement de mdp et je n'y arrive pas j'aimerais votre aide merci.
Ma BDD appelée users est sur phpmyadmin et comporte plusieurs colonnes dont id pseudo email et password.
Le problème vient de ma requête SQL

Code :

$q= $db->prepare("UPDATE users SET password = :new_password WHERE email = :email" );

Dans ce code ci-dessous

Code :

<?php session_start(); //ouverture de la session
//Récuperation du pseudo
$pseudo=$_SESSION["pseudo"];
$email=$_SESSION["lemail"];
$password=$_SESSION["lpassword"];
//Récuperation du dossier usilisateur correspondant au pseudo
?>
<!DOCTYPE html>
<html>
<head>
<meta charset='uft-8'>
<title>Paramètres</title>
<link rel='stylesheet' href='css/main.css'>
<link rel='stylesheet' href='css/index.css'>
</head>
<nav>
<img src=/images/cloud.png title='icone' id='icone'>
</nav>
<body>
<div id="wrapper">
<form method="post">
<input type="password" name="new_password" id="new_password" placeholder="Votre Nouveau Mot de Passe"><br/>
<input type="password" name="newc_password" id="newc_password" placeholder="Confirmer Votre Nouveau Mot de Passe"><br/>
<input type="submit" name="formsend" id="formsend" value="Envoyer"><br/>
</form>
<?php
if(isset($_POST['formsend'])){
extract($_POST);
if(!empty($new_password) && !empty($newc_password)){
if($new_password == $newc_password){
include 'database.php';
global $db;
$q= $db->prepare("UPDATE users SET password = :new_password WHERE email = :email" );
$q->execute([
'password'=> $new_password
]);
echo "<font style=\"font family: courrier new;\"><strong>Le mdp a été changé merci de vous reconnecter</strong></font>";
}else{
echo "<font style=\"font family: courrier new;\"><strong>Les mot de passes ne correspondent pas</strong></font>";
}
}else{
echo "<font style=\"font family: courrier new;\"><strong>Les champs ne sont pas tous remplis</strong></font>";
}
}
?>
</div>
</body>
</html>

J'ai cette erreur :

Code :

Uncaught PDOException: SQLSTATE[HY093]: Invalid parameter number: parameter was not defined in

Reply

Marsh Posté le 07-04-2020 à 15:57:22

Reply

Marsh Posté le 07-04-2020 à 16:12:20

Code :

$q= $db->prepare("UPDATE users SET password = :new_password WHERE email = :email" );
$q->execute([
'password'=> $new_password
]);

Tu ne crois pas qu'il te manque un paramètre quelque part ?

Spoiler :

A moins que le but soit d'appliquer le nouveau mot de passe à tous les utilisateurs mais alors c'est ta requête qui n'est pas bonne...

---------------
D3

Reply

Marsh Posté le 07-04-2020 à 16:24:00

problème résolu :

Code :

$q= $db->prepare("UPDATE users SET password = :password WHERE pseudo = '$pseudo' " );
$q->execute([
'password'=> $new_password
]);

J'ai un nouveau problème est qu'avec le nouveau mot de passe il est impossible de me connecter avec le nouveau mdp pourtant changé dans la BDD
C'est surement du à ma page de connexion :

Code :

<?php session_start(); //ouverture de la session
if (isset($_SESSION["pseudo"])){
header('Location: dashboard.php');
}
?>
<?php
if(isset($_POST['formlogin'])){
extract($_POST);
if(!empty($lemail) && !empty($lpassword)){
$q = $db->prepare("SELECT * FROM users WHERE email = :email" );
$q->execute(['email' => $lemail]);
$result = $q->fetch();
if($result == true){
//le compte existe bien
if(password_verify($lpassword, $result['password'])){
echo "Le mot de passe est bon , connexion";
$_SESSION["pseudo"]=$result['pseudo'];
$_SESSION["lemail"]=$result['email'];
$_SESSION["lpassword"]=$result['password'];
$_SESSION['fileUpload']=False;
header('Location: dashboard.php');
exit();
}else{
echo "Le mot de passe est incorrect";
}
}
else{
echo "Le compte portant l'email " . $lemail . "n'existe pas";
}
}
else{
echo "Veuillez completer l'ensemble des champs";
}
}
?>
<script type="text/javascript" src="js/index.js"></script>
</body>
</html>

Pour raccourcir le code j'ai enlevé la partie avec la déclaration des boutons

Message édité par Nestati le 07-04-2020 à 16:25:00

Reply

Marsh Posté le 07-04-2020 à 17:17:25

Lu,

à quel moment tu haches ce nouveau mot de passe avec password_hash ?

Code :

$q= $db->prepare("UPDATE users SET password = :password WHERE pseudo = '$pseudo' " );

Et te voilà avec une injection SQL via $pseudo ! Pourquoi n'en as-tu pas fait un paramètre/marqueur comme tu l'as fait pour le (hash du) mot de passe ?!?

Code :

if($result == true){

La comparaison (non stricte) à true est inutile car implicite avec une condition (if/else if/for/while/etc)

Message édité par pluj le 07-04-2020 à 17:19:41

Reply

requête SQL en php

Sujets relatifs:

Leave a Replay