Mitt mailformulär saknar datum

PHPportalen Forum Index » PHP Mjukstart
Lägg ett bokmärke på hela tråden
Skapa nytt inlägg   Svara på inlägget
Visa föregående ämne :: Visa nästa ämne  
Startad av: Meddelande
vannar



Medlem i: 4524 dagar

Status: Offline



#741588
Inlägg Skrivet: 2017-04-09 07:47      Ämne: Mitt mailformulär saknar datum Citera

Hae en väldigt enkel mailfunktion på min sida. Den fyller dock sitt syfte men tyvärr så verkar mail från scriptet sakna datumstämpel.

Windows förstår detta men det gör tyvärr inte telefonen så dessa mail kommer längst ner i maillistan.

Jag antar att felet inte är så avancerat, troligen saknas det bara nån datumfunktion i scriptet.

<h3>E-postformul&auml;r </h3>
<?
$error_msg = '<p id="fel_meddelande"><b>Du har inte fyllt i alla obligatoriska fält</b></p>';

$error = false;

$submit = $_POST['submit'];

if(empty($submit))
$form_sent = false;
else
$form_sent = true;

if ($form_sent)
{
$to .= 'info@svahpa.se' . ', ';

$name = $_POST['name'];
$email = $_POST['email'];
$subject = $_POST['subject'];
$message = $_POST['message'];

if(!$name) { $error_msg .= '<p id="fel_meddelande_error"><i>. Du måste skriva ditt namn</i><br /></p>'; $error = true;}
if(!$email) { $error_msg .= '<p id="fel_meddelande_error"><i>. Du måste skriva din e-mail</i><br /></p>'; $error = true;}
if(!$subject) { $error_msg .= '<p id="fel_meddelande_error"><i>. Du måste skriva ett ämne</i><br /></p>'; $error = true;}
if(!$message) { $error_msg .= '<p id="fel_meddelande_error"><i>. Du måste skriva ett meddelande</i><br /></p>'; $error = true;}

if($email) { if(!eregi("^[a-z0-9]+([_\\.-][a-z0-9]+)*@([a-z0-9]+([\.-][a-z0-9]+)*)+\\.[a-z]{2,6}$", $email)){ $error_msg .= "<p id=\"pprojin\"><i>. Felaktig E-postadress</i></p>"; $error = true; }}

if(!$error)
{
$msg = "Namn: \t $name \n";
$msg .= "E-mail: \t $email \n";
$msg .= "Meddelande: \n $message \n";

$mailheaders = "MIME-Version: 1.0\r\n";
$mailheaders .= "Content-type: text/plain; charset=iso-8859-1\r\n";
$mailheaders .= "From: $name <$email>\r\n";
$mailheaders .= "Reply-To: $name <$email>\r\n";

mail($to, $subject ,stripslashes($msg), $mailheaders);
}
}

if (($form_sent) && (!$error))
{
echo '<p id="about_motion_top">Tack för ditt mail ' . $name .'.</p>';
echo '<p id="about_motion_top">Ditt mail skickades utan problem.</p>';

}
else
{
if ($error)
{
echo $error_msg;
}

?>
 
Till toppen på sidan
Visa användarprofil Skicka privat meddelande Besök användarens hemsida
harald_b
Moderator



Medlem i: 4654 dagar
Från: Tavesta
Status: Offline



#741589
Inlägget är accepterad som det rätta svaret Skrivet: 2017-04-09 12:21      Ämne: Citera

Det finns ett antal problem och säkerhetshål i ditt mailskript.
Data som stoppas in i mailheader måste kodas med t.ex mb_encode_mimeheader.
Ofta är det enklast att använda ett färdigt mailskript för att få till allt rätt, som t.ex PHPMailer.

Om datumet saknas kan du försöka få in det i din nuvarande kod med hjälp av:
PHP:
1:
 $mailheaders .= 'Date: '.date('r')."\r\n"
 

_________________
R.r - Ett fritt affärssystem
Till toppen på sidan
Visa användarprofil Skicka privat meddelande
vannar



Medlem i: 4524 dagar

Status: Offline



#741590
Inlägg Skrivet: 2017-04-09 15:21      Ämne: Citera

Tack det löste iaf problemet med datumet. Andra eventuella problem får jag ta tag i en annan gång känns det som.

harald_b skrev:
Det finns ett antal problem och säkerhetshål i ditt mailskript.
Data som stoppas in i mailheader måste kodas med t.ex mb_encode_mimeheader.
Ofta är det enklast att använda ett färdigt mailskript för att få till allt rätt, som t.ex PHPMailer.

Om datumet saknas kan du försöka få in det i din nuvarande kod med hjälp av:
PHP:
1:
 $mailheaders .= 'Date: '.date('r')."\r\n"
 
Till toppen på sidan
Visa användarprofil Skicka privat meddelande Besök användarens hemsida
info@aimdesign.se



Medlem i: 3832 dagar
Från: Märsta
Status: Offline



#741653
Inlägg Skrivet: 2017-04-26 13:33      Ämne: Citera

vannar skrev:
Tack det löste iaf problemet med datumet. Andra eventuella problem får jag ta tag i en annan gång känns det som.

harald_b skrev:
Det finns ett antal problem och säkerhetshål i ditt mailskript.
Data som stoppas in i mailheader måste kodas med t.ex mb_encode_mimeheader.
Ofta är det enklast att använda ett färdigt mailskript för att få till allt rätt, som t.ex PHPMailer.

Om datumet saknas kan du försöka få in det i din nuvarande kod med hjälp av:
PHP:
1:
 $mailheaders .= 'Date: '.date('r')."\r\n"


Varför? Att implementera PHPMailer är mycket enkelt och tar inte många minuter. Ditt mejlscript öppnar upp för spamare att skicak allt möjligt skräp från ditt script. Är det inte bättre att fixa det direkt istället för att skjuta det framför sig? Bara ett förslag.
 

_________________
På svengelska -> It's not the fart that kills you, it's the smäll
Till toppen på sidan
Visa användarprofil Skicka privat meddelande Besök användarens hemsida
Visa tidigare inlägg:   
Skapa nytt inlägg   Svara på inlägget
PHPportalen Forum Index » PHP Mjukstart
Hoppa till:  
Du kan inte skapa nya inlägg i det här forumet
Du kan inte svara på inlägg i det här forumet
Du kan inte ändra dina inlägg i det här forumet
Du kan inte ta bort dina inlägg i det här forumet
Du kan inte rösta i det här forumet
Du kan inte bifoga filer i detta forum
Du kan inte ladda ner filer från detta forum
Kontakta oss på adressen: info@phpportalen.net
Webbplatsen bygger i grunden på phpBB © 2001, 2002 phpBB Group

Modifieringar har senare gjorts i systemet av PHPportalen
Sid och logotypdesign skapad av Daren Jularic