Problème authentification PHP/MYSQL

Version imprimable

Bonjour j'essaie de créer une simple authentification par login et mot de passe ainsi qu'un ajout d'utilisateur et tout lié avec une base de données sou MYSQL. Cependant au moment ou j'essaie d'ajouter un utilisateur lorsque je vais verifier dans ma table s'il est bien ajouté elle reste vidé voici mon code. Voyez quelque chose d'anormal ou que j'ai oublié?

fichier authentification.php
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 <? include("conf.site"); include("fonctions.php"); $bdd = new PDO('mysql:host='.$BDD_hote.';dbname='.$BDD_nmDB, $BDD_user, $BDD_pass); if(isset($_REQUEST['login']) && isset($_REQUEST['password'])) { $requete="INSERT INTO `monsite`.`users` (`id`, `login`, `password`) VALUES (NULL, '". $_REQUEST['login']."', '".md5($_REQUEST['password'])."');"; sql($requete); echo "Compte ajouté a la base"; } else { header("location : add_user.html"); // Avec simplement ma présentation en html faisant appel ce fichier php } ?>
Merci d'avance :)

21/03/2012, 10h07
stealth35

t'as pas mis de gestion des erreurs
http://php.net/manual/fr/pdo.error-handling.php

Salam;
aussi faut pensé au try catch .
exemple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
<?php
// On se connecte à la bdd
try
{
			$PARAM_hote='localhost'; // le chemin vers le serveur
			$PARAM_nom_bd='grip'; // le nom de votre base de données
			$PARAM_utilisateur='root'; // nom d'utilisateur pour se connecter
			$PARAM_mot_passe=''; // mot de passe de l'utilisateur pour se connecter
			// connexion marche trés bien avec affichage de msg erreure
 
		// Options de connection
			$options = array
						(
							PDO::MYSQL_ATTR_INIT_COMMAND    => "SET NAMES utf8" ,  // indiquer à MySQL que echanger nos données en UTF8.
							PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION , 
							PDO::ATTR_PERSISTENT => true
						);
 
 
        $cbd = new PDO('mysql:host='.$PARAM_hote.';dbname='.$PARAM_nom_bd, $PARAM_utilisateur, $PARAM_mot_passe , $options  );
}
 
catch(PDOException $e)
{
        echo 'Une erreur de connexion est survenue !'; $e->getMessage();
        die();
}
$cnx = null; // Fermeture de la connexion
?>

ensuite l'insertion avec des requêtes préparées c'est très efficace pour la sécurité. :ccool:

Citation:

Envoyé par redoran

Salam;
aussi faut pensé au try catch .
exemple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
<?php
// On se connecte à la bdd
try
{
			$PARAM_hote='localhost'; // le chemin vers le serveur
			$PARAM_nom_bd='grip'; // le nom de votre base de données
			$PARAM_utilisateur='root'; // nom d'utilisateur pour se connecter
			$PARAM_mot_passe=''; // mot de passe de l'utilisateur pour se connecter
			// connexion marche trés bien avec affichage de msg erreure
 
		// Options de connection
			$options = array
						(
							PDO::MYSQL_ATTR_INIT_COMMAND    => "SET NAMES utf8" ,  // indiquer à MySQL que echanger nos données en UTF8.
							PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION , 
							PDO::ATTR_PERSISTENT => true
						);
 
 
        $cbd = new PDO('mysql:host='.$PARAM_hote.';dbname='.$PARAM_nom_bd, $PARAM_utilisateur, $PARAM_mot_passe , $options  );
}
 
catch(PDOException $e)
{
        echo 'Une erreur de connexion est survenue !'; $e->getMessage();
        die();
}
$cnx = null; // Fermeture de la connexion
?>

ensuite l'insertion avec des requêtes préparées c'est très efficace pour la sécurité. :ccool:

pourquoi SET NAMES utf8 ? y'a charset dans le dsn
pour mettre en PDO::ATTR_PERSISTENT ?
c'est pas die mais exit, si c'est pour arrêter le code autant laisser l'exception

21/03/2012, 10h32
redoran
Re;

Citation:

pourquoi SET NAMES utf8 ?

pour échangé les données en utf8 dans mon cas.
Code:

1 2 3 pour mettre en PDO::ATTR_PERSISTENT ? c'est pas die mais exit, si c'est pour arrêter le code autant laisser l'exception
là j'ai pas fait attention plus tôt dire j'avais aucune idée. merci pour la correction.
21/03/2012, 10h46
stealth35

Citation:

Envoyé par redoran

Re; pour échangé les données en utf8 dans mon cas.

fait juste : mysql:host:localhost;dbname:truc;charset:utf8 (mais dispo a partir de 5.3.6), a savoir que fait un SET NAMES c'est différent, le résultat est le même c'est le comportement de échappement des donnée risque de ne pas être pareil ;)

Je suis un peu perdu (débutant de chez débutant) il me semblait que cela suffisait de mettre dans mon fichier site.conf:
Code:

1 2 3 4 5 6 <? $BDD_hote="localhost"; $BDD_nmDB="monsite"; $BDD_user="***"; $BDD_pass="***"; ?>
et dans fonctions.php
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 <? function sql($request) { global $bdd; if(strrchr($request, 'SELECT')) { $req = $bdd->query($request); } else { $bdd->exec($request); } if(!empty($req)) { while ($data = $req->fetch()) { $res[] = $data; } return $res; } else { return false; } } ?>
et inclure tout sa dans mon fichier d'inscription
Suis-je vraiment à l'ouest?

21/03/2012, 11h51
stealth35

c'est <?php et pas <?

ton while sert a rien fait direct un fetchAll
21/03/2012, 14h04
grimmjow69

J'ai limite honte pour ma balise... fin bref merci pour le while c'est vrai que ça fait lourd. En tout cas vous m'avez tous bien aider merci beaucoup!!!