PHP

Bien sécuriser un formulaire de contact html/php

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Bien sécuriser un formulaire de contact html/php

dexenium

Bonjour,

J'utilise ce code php pour envoyer un message via un formulaire sur la page html d'un site, mais je ne m'y connais pas assez pour savoir si ce code est bien sécurisé, qu'en pensez vous ?

On me dit que ce code serait sensible au cross scripting

Code :

<?
$adresse="contact@pouet.com";
$from="From: $prenom $nom<$email>\r\n";
$from.="Reply-To: $prenom $nom <$email>\n";
$from.="X-priority:3\n";
$texte="<html>";
$texte.="$texte";
$texte.="</html>";
$from.="MIME-Version: 1.0\r\n";
$from.="Content-Type: text/html";
if (mail($adresse,$sujet,$texte,$from))
echo("<center>Votre message a bien été envoyé ! Merci! <a href=index.html>Cliquez ici pour revenir à l'accueil</a></center>" );
else
echo("<center> Impossible d'envoyer votre message Nous ne pouvons actuellement traiter votre demande Merci de nous contacter par email sur<a href=\"mailto:$adresse\">contact@pouet.com</a> " );
?>

Est-ce qu'il est également judicieux de mettre un captcha ?
Merci pour vos conseils

Message édité par dexenium le 29-01-2007 à 13:33:23

Publicité

sielfried

$texte="<html>";
$texte.="$texte";
$texte.="</html>";

Hmm. [:pingouino]

---------------
StarCraft Professional Gaming Database | [Ze Topic] Starcraft/BroodWar

dexenium

Citation :

$texte="<html>";
$texte.="$texte";
$texte.="</html>";

Hmm. [:pingouino]

Que veux tu dire par là ?

Message cité 1 fois

chani_t

From Dune

Il veux dire que c'est affreux comme code, qu'il ne faut pas entourer de " une variable si tu veux l'afficher, que tu prends trois ligne alors qu'une aurait été plus claire.

$texte='<html>'.$texte.'</html>';

Et puis tu ne semble pas faire de vérification sur les information que tu vas balancer dans ton mail...

skeye

dexenium a écrit :

Citation :

$texte="<html>";
$texte.="$texte";
$texte.="</html>";

Hmm. [:pingouino]

Que veux tu dire par là ?

Que tu fais $texte='<html><html></html>';, ce qui est déjà louche, mais en 3 lignes, ce qui l'est encore plus?

Message édité par skeye le 29-01-2007 à 14:31:22

---------------
Can't buy what I want because it's free -

dexenium

Citation :

Et puis tu ne semble pas faire de vérification sur les information que tu vas balancer dans ton mail...

est-ce qu'on le fait avec addslashes()

skeye

La question est : d'où viennent toutes tes variables, là? :??:

---------------
Can't buy what I want because it's free -

dexenium

Ben c'est le texte saisi par le client non ?

chani_t

From Dune

si c'est le cas, un addslashes ne suffit pas... du moins ça dépend des champs et de ce que tu vas en faire aprés... en bref, donne plus d'info sur le contexte... tu récupére les info pour les mettre dans une bdd ?..

skeye

...je devine ça comment, moi? [:moule_bite]
Le seul moyen pour comprendre ce qu'il pourrait y avoir comme menaces sur ton script, c'est de savoir quelles sont les données qui viennent de "l'extérieur", et ce que tu en fais exactement.

---------------
Can't buy what I want because it's free -

Publicité

dexenium

Alors c'est un formulaire qui permet tout simplement d'envoyer un mail vers une messagerie. Je ne stocke pas le message, à la rigueur je pourrais demander à récupérer l'adresse de l'expéditeur pour la stocker dans une table de ma bdd.
Les champs sont ; nom, prénom, adresse mail, message.

chani_t

From Dune

Alors un spamming de ton adresse tu ne risque pas grand chose, si tu ne stocke rien.

Maintenant si tu stocke des données en base, il faut impérativement les passiver (éviter les injections SQL) donc Addslashes est bien util.

omega2

Chani_t > Addslash est trés insufisant pour protéger des requettes sql. le mieux, c'est les "prepared statements" et les instructions dédiés tels que mysql_escape_string.

chani_t

From Dune

je m'en doutais... mais je n'avais plus le nom de la fonction pour mysql en tête .. merci de la précision

dexenium

Merci !

FORUM HardWare.fr

Programmation

PHP

Bien sécuriser un formulaire de contact html/php

Sujets relatifs
Retour arrière sur un formulaire sans effacer le contenu.	[HTML/CSS/JavaScript]Projet interface site web 3d inspiré de XGl
envoyer formulaire POST d'une popup a la page opener	Modification d'un fichier html
ouvrir nouvelle page apres validation formulaire connex	Aide sur ce formulaire
Forcer l'ouverture d'un fichier d'une page html	[Javascript/HTML] balise <object> et javascipt
[Html,css,javascript] Gallerie Lightbox	[HTML/CSS] Problème affichage d'images
Plus de sujets relatifs à : Bien sécuriser un formulaire de contact html/php

Page générée en 0.060 secondes