[Mail] Sécuriser formulaire mail

**LhIaScZkTer** · 12/02/2007, 13h41

Bonjour,

Puisque je vois qu'aujourd'hui la mode est à la fonction mail(). Je m'en vais aussi poser ma question.

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
<?php 
$texte="";
if(isset($_POST['message'])){
 
  $dest="destinataire@mon-fai.com";
  $objet="E-mail d'un fan";
  $texte.= $_POST['message'];
  $entete="From:de@le-fai-de.com";
 
      if(mail($dest,$objet,$texte,$entete)){
          echo "<h1>Thanks, Merci ;-)</h1>";
      }
      else{
          echo "<h1>Error !!!!! :-(</h1>";
          }
}
    ?>

Je n'ai pas mis le code du formulaire, car il ne contient qu'un textarea et un boutton submit.

Ma question est : Comment pourrais-je faire pour désactiver le boutton envoyer après l'envoie du formulaire ? J'ai essaié avec du javascript

Code :

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

onclick="this.disabled=true;"

mais ça ne marche pas, on dirait que la page se recharge à cause du if.

Je me suis dis, que je pourrais mettre dans le bouttoninput type="submit" une variable à la place du submit et que quand on rentre dans la condition if, ça change la variable et donc la valeur du type.

Je crois qu'il est temps pour un petit exemple : avant envoie $mavar="submit"; après envoie $mavar="hidden"; donc :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php 
$mavar="submit";
$texte="";
if(isset($_POST['message'])){
  
      //les instruction qui vont bien...
     
      if(mail($dest,$objet,$texte,$entete)){
           $mavar="hidden";
           echo "<h1>Thanks, Merci ;-)</h1>";
      }
      //le else
}
    ?>

Mais malheureusement ça ne marche toujours pas

Mais surtout comment désactiver le code dans mon formulaire mail ?

**RideKick** · 12/02/2007, 13h52

dans ton cas le code complet nous aiderai a t'eguiller parce la comme ça je vois pas ce qui cloche !

**illegalsene** · 12/02/2007, 13h55

Tu pourrais créer dans ton formulaire un champ de type hidden avec 1 pour valeur au départ.
Tu fais un test sur ce champ. S'il est égal à 1, tu affiches ton bouton submit avec disabled à false. S'il est égal à 0, tu mets le disabled à true Lors de la validation de ton formulaire tu mets la valeur du champ cachée à 0.

**LhIaScZkTer** · 12/02/2007, 14h10

voilà le code de tout mon formulaire, comme vous pouvez le voir il est très basique.

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
  <body>
    <form action="<?= $_SERVER['PHP_SELF'] ?>" method="post" enctype="application/x-www-form-urlencoded">
      <center>
<textarea cols="50" rows="5" name="message"></textarea>
        <br>
        <input type="<?=$submit?>"  onClick="this.form.submit();this.disabled=true;" name="envoie" value="Send/Envoyer">
      </center>
    </form>
<?php 
$texte="";
$submit="submit";
if(isset($_POST['message'])){
  $dest="mon-mail@mon-serveur.com";
  $objet="E-mail d'un fan";
  $texte.= $_POST['message'];
  $entete="From:de@mon-serveur.com";
      if(mail($dest,$objet,$texte,$entete)){
          $submit="hidden";
          echo "<h1>Thanks, Merci ;-)</h1>";
      }
      else{
          echo "<h1>Error !!!!! :-(</h1>";
          }
}
    ?>
  </body>

Voilà le problème est que quand je valide le fomulaire je reçois le mails et le echo"Thanks, Merci" s'affiche comme prévu par contre, il ne devient pas hidden ce qui est dommage, car on peut cliquer autant de foi qu'on veut sur le boutton submit ce qui est assez désagréable

Sans parler du code qui est interprété dans mon textarea :-/

**RideKick** · 12/02/2007, 14h12

Code php :

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

<input type="<?echo $submit ;?>"....

ça ira bcp mieu

**LhIaScZkTer** · 13/02/2007, 09h26

Bonjour à tous et merci pour vos réponse,

Pour une fois que je suis matinal

j'ai pas le choix j'ai un exa sur java dans une semaine et j'ai une semaine pour me mettre tout le semestre dans la tête

illegalsene merci pour ta réponse j'ai donc fait les testes que tu m'as dit et ça ne marchait toujours pas, jusqu'à ce que je me dise, mais c'est normal :-)
Et c'est là que l'argument de RideKick prend toute ça splandeur, car il n'est pas vraiment juste, mais il m'a permis de comprendre.

Si je mets un echo $submit dans mon champ input il ne me renverra jamais le résultat demandé, puisque la variable est vide à ce moment là et n'as jamais été crée(logique) Par contre on pourrait se dire ouai mais quand j'envoie le formulaire, elle est crée initialisé et son état change. Oui, mais non, car elle est crée seulement dans la partie qui traite le formulaire. Donc la réponse devient évidente :-)

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
  <body>
<?php 
$texte="";
$submit="submit";
if(isset($_POST['message'])){
  $dest="mon-mail@mon-serveur.com";
  $objet="E-mail d'un fan";
  $texte.= $_POST['message'];
  $texte = htmlentities($texte);
  $entete="From:de@mon-serveur.com";
      if(mail($dest,$objet,$texte,$entete)){
          $submit="hidden";
          echo "<h1>Thanks, Merci ;-)</h1>";
      }
      else{
          echo "<h1>Error !!!!! :-(</h1>";
          }
} 
?>
    <form action="<?= $_SERVER['PHP_SELF'] ?>" method="post" enctype="application/x-www-form-urlencoded">
      <center>
<textarea cols="50" rows="5" name="message"></textarea>
        <br>
        <input type="<? echo $submit; ?>"  onClick="this.form.submit();this.disabled=true;" name="envoie" value="Send/Envoyer">
      </center>
    </form>
 
  </body>

Puis pour rajouter un peu plus de sécurité j'ai mis $texte=htmlentities($texte); croyez-vous que c'est suffisant pour éviter l'insertion de code dans mon mail ?

Il faut dire aussi que de passé le input en HIDDEN n'est pas vraiment top car avec Firefox si on a la webdevelopper toolbar on peut éditer tout le champ caché et donc continuer à m'envoyer un million de spam

Avez-vous d'autre suggestion ?

En tout cas merci pour votre aide

**RideKick** · 13/02/2007, 09h44

Code php :

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
31
32
33
34
  <body>
<?php 
$texte="";
$envoye="non";
if(isset($_POST['message']) && $_POST['envoi'] == 'non'){
  $dest="mon-mail@mon-serveur.com";
  $objet="E-mail d'un fan";
  $texte.= $_POST['message'];
  $texte = htmlentities($texte);
  $entete="From:de@mon-serveur.com";
      if(mail($dest,$objet,$texte,$entete)){
          $envoye="oui";
          echo "<h1>Thanks, Merci ;-)</h1>";
      }
      else{
          echo "<h1>Error !!!!! :-(</h1>";
          }
} 
else
{
	echo "<h1>Vous avez déjà utilisé le formulaire pour envoyer un mail</h1>";
 
}
?>
    <form action="<?= $_SERVER['PHP_SELF'] ?>" method="post" enctype="application/x-www-form-urlencoded">
      <center>
<textarea cols="50" rows="5" name="message"></textarea>
        <br>
        <input type="submit"  onClick="this.form.submit();this.disabled=true;" name="envoie" value="Send/Envoyer">
        <input type="hidden" name="envoi" value="<? echo $envoye ;?>">
      </center>
    </form>
 
  </body>

ça peut etre une idee...

[Mail] Sécuriser formulaire mail

Langage PHP

Vue hybride

Discussions similaires

Partager

Partager