Discussion :

[grimmjow69]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[stealth35]

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

[redoran]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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é.

[stealth35]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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é.

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

[redoran]

Re;

pourquoi SET NAMES utf8 ?

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[stealth35]

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

[grimmjow69]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
<?
$BDD_hote="localhost";
$BDD_nmDB="monsite";
$BDD_user="***";
$BDD_pass="***";
?>

et dans fonctions.php

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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?

[stealth35]

c'est <?php et pas <?

ton while sert a rien fait direct un fetchAll

[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!!!