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

Marsh Posté le 03-05-2007 à 02:36:00

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

Reply

Marsh Posté le 03-05-2007 à 02:36:00

Reply

Marsh Posté le 03-05-2007 à 04:42:30

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.

Reply

Marsh Posté le 03-05-2007 à 10:30:12

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 )

Reply

Marsh Posté le 03-05-2007 à 11:23:08

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.

Reply

Marsh Posté le 03-05-2007 à 13:33:40

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] ?

Reply

Marsh Posté le 03-05-2007 à 13:51:03

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})/

Reply

Marsh Posté le 03-05-2007 à 14:22:11

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

Reply

Marsh Posté le 03-05-2007 à 14:37:29

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

Reply

Marsh Posté le 03-05-2007 à 14:44:19

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})/

Reply

Marsh Posté le 03-05-2007 à 14:45:39

Google-> Prepared Statements

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

Reply

Marsh Posté le 03-05-2007 à 14:45:39

Reply

Marsh Posté le 03-05-2007 à 14:48:17

+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})/

Reply

Marsh Posté le 03-05-2007 à 14:50:09

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.

Reply

Marsh Posté le 03-05-2007 à 14:51:29

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})/

Reply

Marsh Posté le 03-05-2007 à 14:54:07

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

Reply

Marsh Posté le 03-05-2007 à 15:00:25

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})/

Reply

Marsh Posté le 03-05-2007 à 15:11:41

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>';

Reply

Marsh Posté le 03-05-2007 à 15:15:02

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})/

Reply

Marsh Posté le 03-05-2007 à 15:20:47

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

Reply

Marsh Posté le 03-05-2007 à 15:23:15

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})/

Reply

Marsh Posté le 03-05-2007 à 15:27:54

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

Reply

Marsh Posté le 04-05-2007 à 00:03:28

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:

Reply

Marsh Posté le

Reply

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

Sujets relatifs:

Leave a Replay