PHP

Création de forum : problème avec l'inscription

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Création de forum : problème avec l'inscription

Tristou

Keep calm and hack cookies

Bonsoir,

je suis en train de réaliser un forum, j'ai actuellement un problème avec la page d'inscription. En effet, après des vérifications du pseudo et de l'e-mail, après avoir protégé les données, après avoir généré le mot de passe aléatoirement, j'insère dans la base de donnée, je créé les variable de session et j'ai un message selon si cela a marché ou pas. Seul problème, au moment de valider, la seule chose que j'ai, c'est une page "blanche"

Voici mon code (du moins une partie) :

Code :

// Pour gérer le cas ?valid=inscription
else if ($confirmation == 'inscription')
{
$pseudo = $_POST['pseudo'];
$email = $_POST['email'];
$i = 0;
// Vérification de la conformité du pseudo
$nombre_pseudo = mysql_result(mysql_query("SELECT COUNT(membre_pseudo) FROM forum_membres WHERE membre_pseudo = '".mysql_real_escape_string($_POST['pseudo'])."'" )
, 0) or die(mysql_error());
if ($nombre_pseudo != 0)
{
$erreur_pseudo1 = 'Votre pseudo est déjà utilisé par un membre';
$i++;
}
if (strlen($_POST['pseudo']) < 3 || strlen($_POST['pseudo']) > 15)
{
$erreur_pseudo2 = 'Votre pseudo fais moins de 3 ou plus de 15 caractères';
$i++;
}
if (!preg_match("/[^a-z0-9_ ]+/i",$pseudo))
{
$erreur_pseudo3 = 'Il y a des caractères interdits dans ce pseudo. Seuls abcdefghijklmnopqrstuvwxyz0123456789 et _ sont aurorisés';
$i++;
}
// Vérification de la conformité de l'e-mail
$nombre_email = mysql_result(mysql_query("SELECT COUNT(membre_email) FROM forum_membres WHERE membre_email = '".mysql_real_escape_string($_POST['email'])."'" )
, 0) or die(mysql_error());
if ($nombre_email != 0)
{
$erreur_mail1 ='Cette adresse e-mail est déjà utilisée';
$i++;
}
if(!preg_match("#^[a-z0-9._-]+@[a-z0-9._-]{2,}\.[a-z]{2,4}$#", $_POST['email']) || !empty($_POST['email']))
{
$erreur_mail2 = 'Le format d\'e-mail n\'est pas correct';
$i++;
}
if($i == 0)
{
//On traite les données à insérer dans la BDD
$temps = time();
$pseudo = htmlspecialchars($_POST['pseudo'], ENT_QUOTES);
$email = htmlspecialchars($_POST['email'], ENT_QUOTES);
// On s'occupe de la génération du mot de passe
$caracteres = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789';
$caracteres_melange = str_shuffle($caracteres); // mélange des caractères de la variable $caracteres
$motdepasse = stubstr($caracteres_melange, 0, 8); //selection de 8caractères
$mdp_crypte = md5($motdepasse); //hashage du mot de passe
mysql_query ('INSERT INTO forum_membres(membre_pseudo, membre_mdp, membre_email, membre_inscrit, membre_derniere_visite)
VALUES ('.$pseudo.', '.$mdp_crypte.', '.$email.', '.$temps.', '.$temps.')');
$_SESSION['pseudo'] = $pseudo;
$_SESSION['id'] = mysql_insert_id();
$SESSION['level'] = 1;
echo'<div class="acces_ok"><img src="./template/images/accept.png"/>Félicitations' ,$_POST['pseudo'], ',
vous êtes inscrit sur le forum.<br/> Notez ce mot de passe temporaire :' ,$motdepasse, '
<br/><a href="index.php" title="Aller à l\'index du forum">Retour à l\'index du forum</a>.</div>';
}
else
{
echo '<div class="erreur_forum"><img src="./template/images/cancel.png"/> Erreur : les probl&grave;mes suivants sont survenus :<br/>' ,
$erreur_pseudo1, '<br/>' ,
$erreur_pseudo1, '<br/>' ,
$erreur_email1, '<br/>' ,
$erreur_email2, '<br/>
<a href="index.php" title="Aller à l\'index du forum">Retour à l\'index du forum</a>.</div>';
}
}
else
{
echo '<div class="erreur_forum"><img src="./template/images/cancel.png"/> Erreur : vous ne devez pas arriver sur cette page
<br><a href="index.php" title="Aller à l\'index du forum">Retour à l\'index du forum</a>.</div>';
}

Comment puis-je régler ce problème ? Merci d'avance

Message édité par Tristou le 03-05-2007 à 14:40:21

Publicité

Shinuza

This is unexecpected

Code :

if (strlen($_POST['pseudo']) < 3 || strlen($_POST['pseudo']) > 15)
{
$erreur_pseudo2 = 'Votre pseudo est trop grand /trop petit';
$i++;
}
if (!preg_match("/[^a-z0-9_ ]+/i",$pseudo))
{
$erreur_pseudo3 = 'Il y a des caractères interdits dans ce pseudo';
$i++;
}

Juste un problème d'ergonomie, ces messages d'erreurs n'aident pas l'utilisateur, tu dois rappeler les conditions qui font foirer ton test.

Pour le reste il manque un bout.

Message édité par Shinuza le 03-05-2007 à 04:42:41

---------------
Mains power can kill, and it will hurt the entire time you’re dying from it.

Tristou

Keep calm and hack cookies

Je vais modifier cela

Autrement, je n'ai mis la que la partie du script qui concerne le cas valid=inscription (et il y a tout )

NewsletTux

<Insérez ici votre vie />

Encore des SELECT * ... Pourquoi faire un SELECT * (ou SELECT COUNT(*)) qui rapatrie TOUT l'enregistrement alors que seul le pseudo est vérifié ?

Après on dit que php n'est pas sécurisé, mais ce sont les gens qui ne savent pas l'utiliser, en fait.

2èmement, il y a une magnifique injection SQL dans ta recherche des pseudos, encore une faille de sécurité.

Idem pour la vérif du mail.

Tristou

Keep calm and hack cookies

Je ne me suis pas prétendu expert en PHP, c'est pour cela que je viens demander de l'aide

Pour les deux requêtes de recherche, ceci serait-il plus optimisé ?

Code :

mysql_query("SELECT COUNT(membre_pseudo) FROM forum_membres

Code :

mysql_query("SELECT COUNT(membre_email) FROM forum_membres

Ensuite, à propos de l'injection SL, où est le problème et comment le corriger ?

Cela peut-il aider pour le problème de départ qui est le problème d'inscription [:toph] ?

dwogsi

Défaillance cérébrale...

Si tu n'as pas les magic_quote activés c'est dangereux. Et même avec c'est peu recommandable. Imagine que comme pseudo je saisisse ' OR 1=2 -- ta requête ne renverra jamais de résultat et je pourrais donc créer des doublons à la pelle. pour éviter cela : http://fr.php.net/mysql_real_escape_string

Message édité par dwogsi le 03-05-2007 à 14:38:58

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

soulmanto

Chat Noir replica

Citation :

$motdepasse = stubstr($caracteres_melange, 0, 8); //selection de 8caractères

je sais pas si c'est une faute de frappe dans le code que tu as mis ici, mais ça m'a tout l'air d'une erreur!

Message édité par soulmanto le 03-05-2007 à 14:22:52

Tristou

Keep calm and hack cookies

J'ai corrigé les requêtes, le substr et ajouté un mysql_real_escape_string() pour les deux requêtes de recherche mais j'ai toujours la page blanche

Message édité par Tristou le 03-05-2007 à 14:37:57

dwogsi

Défaillance cérébrale...

C'est quoi ce stockage des erreurs?? C'est un peu moyen limite bof...
Plutôt que d'utiliser un $i et des tonnes de variables pour tes erreurs utilise plutôt quelque chose comme ça :
Ligne 6, remplacer par :
$lesErreurs = array();

A chaque fois que tu veux ajouter une erreur tu fais simplement :
$lesErreurs[] = "Le texte blablabla erreur dégage tu pue";

Et pour l'affichage, plutôt que de faire plusieurs echo tu fais un simple :
foreach($lesErreurs as $erreur) echo $erreur,'<br />';

A la place du test sur le $i tu fais :
if(count($lesErreurs)==0)

Enfin je dis ça... C'est comme tu veux mais ça me parait bien plus logique et plus clair.

Pour ton problème de page blanche, tu es sûr que le problème ne vient pas du code que tu ne nous montre pas?

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

Shinuza

This is unexecpected

Google-> Prepared Statements

---------------
Mains power can kill, and it will hurt the entire time you’re dying from it.

Publicité

dwogsi

Défaillance cérébrale...

+1
mais long a expliquer/comprendre...

Message cité 1 fois

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

Shinuza

This is unexecpected

dwogsi a écrit :

+1
mais long a expliquer/comprendre...

Gné? :??:

---------------
Mains power can kill, and it will hurt the entire time you’re dying from it.

dwogsi

Défaillance cérébrale...

Ba j'le vois bien revennir avec pas mal de questions s'il part la dessus.
Donc je proposais de simples mysql_real_escape_string(). Mais je suis tout à fait d'accord, les prepared statements c'est bien mieux.

Message édité par dwogsi le 03-05-2007 à 14:56:07

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

Tristou

Keep calm and hack cookies

Je prends en compte toutes les suggestions, avec un array, ça sera peut-être plus "simple", je vais voir ça

Voilà le début de mon code( ça gère la validation de la connexion) :

Code :

<?php
session_start();
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" >
<head>
<title>Confirmation</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<link rel="stylesheet" media="screen" type="text/css" title="Design" href="./template/style.css" />
</head>
<body>
<?php
$confirmation = $_GET['valid'];
include ('./fonctions/fonctions.php');
mysql_connect($hote, $login, $motdepasse);
mysql_select_db($base);
// Pour gérer le cas ?valid=connexion
if($confirmation == 'connexion')
{
$pseudo = $_POST['pseudo'];
$password = $_POST['password'];
if (empty($_POST['pseudo']) || empty($_POST['password']))
{
echo '<div class="erreur_forum"><img src="./template/images/cancel.png"/> Erreur : un des deux champs n\'est pas rempli<br>
<a href="connexion.php" title="Recommencer l\'identification">Retour au formulaire de connexion</a>.</div>';
}
else
{
if(isset($_SESSION['pseudo']))
{
echo '<div class="erreur_forum"><img src="./template/images/cancel.png"/> Erreur : vous ne pouvez accéder cette page
<br><a href="index.php" title="Aller à l\'index du forum">Retour à l\'index du forum</a>.</div>';
}
else
{
$pseudo = htmlspecialchars($_POST['pseudo'], ENT_QUOTES);
$motdepasse = htmlspecialchars($_POST['password'], ENT_QUOTES);
$requete_connexion = mysql_query(' SELECT membre_mdp, membre_id, membre_rang FROM forum_membres WHERE membre_pseudo = "'.mysql_real_escape_string($pseudo).'" ') or die (mysql_error());
$affichage = mysql_fetch_assoc($requete_connexion);
if($affichage['membre_mdp'] == $password)
{
$_SESSION['pseudo'] = $pseudo;
$_SESSION['level'] = $affichage['membre_rang'];
$_SESSION['id'] = $affichage['membre_id'];
echo'<div class="acces_ok"><img src="./template/images/accept.png"/> Bienvenue ' ,$_SESSION['pseudo'], ',
vous êtes maintenant connecté.<br/><a href="index.php" title="Aller à l\'index du forum">Retour à l\'index du forum</a>.</div>';
}
else {
echo '<div class="erreur_forum"><img src="./template/images/cancel.png"/> Erreur : Le pseudo et/ou le mot de passe est/sont incorrect(s).<br>
<a href="connexion.php" title="Recommencer l\'identification">Retour au formulaire de connexion</a>.</div>';
}
}
}
}

Message édité par Tristou le 03-05-2007 à 14:58:16

dwogsi

Défaillance cérébrale...

Franchement, si $_GET['valid'] a bien pour valeur inscription je vois pas trop comment tu pourrais avoir une page blanche. Tu es sûr de la valeur?

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

Tristou

Keep calm and hack cookies

C'est à dire :??:

Pour le foreach, j'ai procédé comme tu m'as indiqué, j'ai une parse error

Code :

echo '<div class="erreur_forum"><img src="./template/images/cancel.png"/> Erreur : les probl&grave;mes suivants sont survenus :<br/>' ,
foreach ($erreurs as $erreur) echo $erreur, '<br/>
<a href="index.php" title="Aller à l\'index du forum">Retour à l\'index du forum</a>.</div>';

dwogsi

Défaillance cérébrale...

Code :

echo '<div class="erreur_forum"><img src="./template/images/cancel.png"/> Erreur : les probl&grave;mes suivants sont survenus :<br/>';
foreach ($erreurs as $erreur) echo $erreur, '<br/>';
echo '<a href="index.php" title="Aller à l\'index du forum">Retour à l\'index du forum</a>.</div>';

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

Tristou

Keep calm and hack cookies

Ah ok, d'accord [:joce]

Sinon j'ai vérifié, dans la page inscription.php, j'ai bin mis confirmation.php?valid=inscription, soit la valeur du $_GET['valid']. S'il en avait été autrement, il y a le else qui affiche un message d'erreur disant que l'on ne devrait pas se trouver là.

dwogsi

Défaillance cérébrale...

Oui c'est aussi ce que je me disais en fait.
Bon, par page blanche tu entends quoi? Visuellement? Si oui, y-a-t-il quelque chose dans la source de la page?
Si c'est en ligne, envois un lien.

---------------
-- Debian -- Le système d'exploitation universel | Le gras c'est la vie! | /(bb|[^b]{2})/

Tristou

Keep calm and hack cookies

Bah y'a vraiment rien, juste le fond de page (qui est coloré mais OSEF )

J'ai ça dans la source

Code :

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" >
<head>
<title>Confirmation</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<link rel="stylesheet" media="screen" type="text/css" title="Design" href="./template/style.css" />
</head>
<body>
<!-- Fonctions.php
Regroupe les différentes fonctions qui permetront de faire fonctionner le forum -->

Mes balises body et html sont bien fermées dans mon code, j'ai vérifié.

Par contre j'ai rien mis en ligne .

EDIT : et le commentaire vient du fichier où j'ai mis mes fonctions (identifiants de connexion, calcul du temps de génération de la page) qui est inclus dans ce fichier-ci

Message édité par Tristou le 03-05-2007 à 15:30:41

Tristou

Keep calm and hack cookies

Bon, on a trouvé la solution. Ca pas été simple mais on a trouvé comment corriger le problème. Si ça intéresse quelqu'un de savoir, j epeux mettre le code final :jap:

Publicité

FORUM HardWare.fr

Programmation

PHP

Création de forum : problème avec l'inscription

Sujets relatifs
Problème de bordures en CSS?	[JAR] Probleme avec la creation d'un .jar
Problème avec l'insertion d'une date	Probleme affichage image dans une applet
problème de requete en fonction de l'url de la page	[C++] Problème de redirections des I/O standards
Probleme Flux RSS	Probleme avec une page PHP
Problème avec la fonction avg en double jointure
Plus de sujets relatifs à : Création de forum : problème avec l'inscription

Page générée en 0.134 secondes