[PHP]Beveiligde contactscript

Hallo,

Ik heb dus een contact script op mijn website:
http://www.hambouwvof.nl/contact.php

En die is niet beveiligd.
Zouden jullie mij willen helpen om deze beveiligd te maken, of hebben jullie gratis 1 voor me?

Hier is mijn contact script:

Codeblok (voorbeeld.php):

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51

$action=isset($_REQUEST[action]);
if ($action=="")
{
?>


}
else
{
$name=$_REQUEST[name];
$email=$_REQUEST[email];
$message=$_REQUEST[message];
if (($name=="")||($email=="")||($message==""))
{
echo "U heeft niet alles ingevult, vul het formulier alstublieft opnieuw in.";
}else{
$from="From: $name<$email>
Return-path: $email";
$subject="Hambouw contact form";
mail("MAILADRESl", $subject, $message, $from);
echo "Email verzonden!";
}
}
?>

Groetjes,
Jordymen

het staat bol van de fouten.

ja dat zei ik toch ook?

Jordymen schreef:

ja dat zei ik toch ook?

Dus je wilt dat wij die fouten gaan maken? Doe zelf eens wat als je een website maakt.

Ik maak wel een beginnetje:

Codeblok (voorbeeld.php):

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
35
36
37
38
39
40
41
42
43


if(!isset($_POST[action])) {
?>


} else {
$check = array(name, email, message);
foreach($check as $data) {
if(trim($_POST[$data) == ) {
echo "U heeft niet alles ingevult, vul het formulier alstublieft opnieuw in.";
} else {
$from="From: $name<$email>rnReturn-path: $email";
$subject="Hambouw contact form";
$message = htmlentities($_POST[message], ENT_QUOTES);
if(mail("MAILADRESl", $subject, $message, $from)) {
echo "Email verzonden!";
} else {
echo Oeps, er ging iets mis!;
}
}
}
}
?>

niet getest.
Maak jij nu maar verder.

// edit: er zit ook nog een leuk foutje in! maar geen zin om te fixen.

Dylanh schreef:

Ik maak wel een beginnetje:

Codeblok (voorbeeld.php):

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
35
36
37
38
39
40
41
42
43


if(!isset($_POST[action])) {
?>


} else {
$check = array(name, email, message);
foreach($check as $data) {
if(trim($_POST[$data) == ) {
echo "U heeft niet alles ingevult, vul het formulier alstublieft opnieuw in.";
} else {
$from="From: $name<$email>rnReturn-path: $email";
$subject="Hambouw contact form";
$message = htmlentities($_POST[message], ENT_QUOTES);
if(mail("MAILADRESl", $subject, $message, $from)) {
echo "Email verzonden!";
} else {
echo Oeps, er ging iets mis!;
}
}
}
}
?>

niet getest.
Maak jij nu maar verder.

// edit: er zit ook nog een leuk foutje in! maar geen zin om te fixen.

Deze code is misschien iets beter dan bovenstaand, neemt niet weg dat het uiterst slecht is...

Rogier schreef:

Eerdere citaten tonen

Dylanh schreef:

Ik maak wel een beginnetje:

Codeblok (voorbeeld.php):

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
35
36
37
38
39
40
41
42
43


if(!isset($_POST[action])) {
?>


} else {
$check = array(name, email, message);
foreach($check as $data) {
if(trim($_POST[$data) == ) {
echo "U heeft niet alles ingevult, vul het formulier alstublieft opnieuw in.";
} else {
$from="From: $name<$email>rnReturn-path: $email";
$subject="Hambouw contact form";
$message = htmlentities($_POST[message], ENT_QUOTES);
if(mail("MAILADRESl", $subject, $message, $from)) {
echo "Email verzonden!";
} else {
echo Oeps, er ging iets mis!;
}
}
}
}
?>

niet getest.
Maak jij nu maar verder.

// edit: er zit ook nog een leuk foutje in! maar geen zin om te fixen.

Deze code is misschien iets beter dan bovenstaand, neemt niet weg dat het uiterst slecht is...

Ik zei ook niet dat dit de beste code is. Ik zei ik maak wel eenbeginnetje

Nielsba schreef:

Eerdere citaten tonen

Jordymen schreef:

ja dat zei ik toch ook?

Dus je wilt dat wij die fouten gaan maken? Doe zelf eens wat als je een website maakt.

nee, hij wilt dat wij hem helpen. Waarom zou hij willen dat wij ook fouten gaan maken.
Beetje onzinnige reactie vind je niet?

Heb je al op internet gekeken? Daar staan genoeg kant en klare contact formulieren.

[vergeet je headers niet mee te sturen ]

Rudie schreef:

Eerdere citaten tonen

Nielsba schreef:

Eerdere citaten tonen

Jordymen schreef:

ja dat zei ik toch ook?

Dus je wilt dat wij die fouten gaan maken? Doe zelf eens wat als je een website maakt.

nee, hij wilt dat wij hem helpen. Waarom zou hij willen dat wij ook fouten gaan maken.
Beetje onzinnige reactie vind je niet?

Heb je al op internet gekeken? Daar staan genoeg kant en klare contact formulieren.

[vergeet je headers niet mee te sturen ]

Sorry je hebt mij verkeerd begrepen. Ik bedoelde sorry was een verkeerde zins opbouw Dat je ook eens zelf eerst iets kan doen dan meteen te vragen hoe moet ik dit doen enz? Dus ik bedoelde je wilt dat wij die fout gaan oplossen doe zelf eens wat

Ik maak wel een voor jullie, want dit is best wel om te huilen hier

Shahiem schreef:

Ik maak wel een voor jullie, want dit is best wel om te huilen hier

ik heb het al? ik ben er mee bezig

- Gebruik isset()
- Gebruik $_POST
- Variabelen buiten je quotes
- Die isset voor $_POST[action] weghalen want je doet er niks mee
- ipv " gebruiken als je HTML in je echos gaat zetten