Scripts - Tutorials - Forum - Downloads - Showcase - Contact

Artikels

VPN vergelijken

Algemeen

Beginpagina

FAQ

Grafische worm (243)

Links

Nieuwsartikels

Nieuwsarchief

Boeken programmeren

Overzicht

Samenwerken

Webhosting

Zoek op Sitemasters

Leden

Registreren

Ledenlijst

Ons team

Links

webhostingtop10.be

Sociale media

Follow @sitemasters

Sitemasters

Adverteren op Sitemasters?

Contacteer ons

RSS

Link naar ons

Donaties

Poll

Je moet ingelogd zijn om te stemmen.

Statistieken

Linkpartners

Forum

Categorieën > PHP & SQL

veilige registratie

W0etah - 11/11/2006 12:56 (laatste wijziging 11/11/2006 13:59)
PHP beginner		ik heb nu een registratie gemaakt, maar ben niet zeker of hij wel veilig is. http://www.plaatscode.be/3086/ als je weet hoe het veiliger kan, alle hulp is welkom EDIT: heb er nog wat aan veranderd

10 antwoorden

Gesponsorde links

Stijn - 11/11/2006 13:27 (laatste wijziging 11/11/2006 13:36)

PHP expert

lijn 8: na $title nog een puntkomma!
lijn 31: vervang eens door:

php code - Bekijk de code zonder highlighting - Klap code in

                <?php
...
if( $_SERVER['REQUEST_METHOD'] == 'POST' ) { 
...
?>
            
<?php
...
if( $_SERVER['REQUEST_METHOD'] == 'POST' ) { 
...
?>
 

lijn 41: kijken of de e-mail wel correct is?

php code - Bekijk de code zonder highlighting - Klap code in

<?php
...
if( preg_match("#\w+@[a-zA-Z_]+?\.[a-zA-Z]{2,6}#si" , $_POST['email'] ) ) { 
     $email = $_POST['email'];
} else {
     die( 'e-mail adres is ongeldig!' );
}
...
?>

<?php
...
if( preg_match("#\w+@[a-zA-Z_]+?\.[a-zA-Z]{2,6}#si" , $_POST['email'] ) ) { 
     $email = $_POST['email'];
} else {
     die( 'e-mail adres is ongeldig!' );
}
...
?>

lijn 43-57: kan je op een andere en betere manier doen

php code - Bekijk de code zonder highlighting - Klap code in

<?php
...
if( empty( $gebruikersnaam ) === true || isset( $gebruikersnaam ) === false) { 
    die('U heeft geen gebruikersnaam ingevuld'); // vervang alle echo's met die die(); functie
} // dit herhaal je nu voor die andere ook!
...
?>

<?php
...
if( empty( $gebruikersnaam ) === true || isset( $gebruikersnaam ) === false) { 
    die('U heeft geen gebruikersnaam ingevuld'); // vervang alle echo's met die die(); functie
} // dit herhaal je nu voor die andere ook!
...
?>

groeten
stijn

[edit]

@jexus: ok.

Richard - 11/11/2006 13:32
Crew algemeen		Citaat: [..code..] Geen veilige regex;) Je mist de anchors voor het matchen aan begin en eind van de string php code - Bekijk de code zonder highlighting - Klap code in '#^\w+@[a-z_]+\.[a-z]{2,6}$#i' '#^\w+@[a-z_]+\.[a-z]{2,6}$#i' Aangezien je de 'i' modifier gebruikte hoef je niet [a-zA-Z] te doen, en de 's' modifier was overbodig, omdat je geen '.' hebt staan die niet ge-escaped is... Citaat: [..code..] empty() controleert ook of een variabele geset is, isset() is hier dus overbodig. Je hoeft ook niet te controleren op een true waarde, omdat de teruggave van empty() de if al data oplevert hiervoor: php code - Bekijk de code zonder highlighting - Klap code in if (empty ($gebruikersnaam)) { if (empty ($gebruikersnaam)) {

W0etah - 11/11/2006 13:37
PHP beginner		bedankt, ik hbe dit verandert in het script, maar ik zit met een probleem, hij blijft maar zeggen dat mijn wachtwoord enkel uit alfanumerieke tekens mag bestaan terwijl ik er gewoon "test" typ

Richard - 11/11/2006 13:46
Crew algemeen		Waar staat die controle dan? Ik zie het nergens staan...

W0etah - 11/11/2006 13:47
PHP beginner		regel 52

Stijn - 11/11/2006 13:55
PHP expert		regel 52 is dit: php code - Bekijk de code zonder highlighting - Klap code in echo "U heeft geen website ingevuld"; echo "U heeft geen website ingevuld"; :s

W0etah - 11/11/2006 13:59
PHP beginner		sry, was vergeten de code te update http://www.plaatscode.be/3086/

Stijn - 11/11/2006 14:12

PHP expert

php code - Bekijk de code zonder highlighting - Klap code in

} elseif(!preg_match("^[a-zA-Z0-9]$", $wachtwoord)) {

} elseif(!preg_match("^[a-zA-Z0-9]$", $wachtwoord)) {

-->

php code - Bekijk de code zonder highlighting - Klap code in

} elseif(!preg_match("#^[a-zA-Z0-9]+$#si", $wachtwoord)) {

} elseif(!preg_match("#^[a-zA-Z0-9]+$#si", $wachtwoord)) {

nu zou die moeten werken.

Richard - 11/11/2006 14:18
Crew algemeen		Je kunt in dit geval beter PHP.net: ctype_alnum gebruiken, dat is sneller;)

W0etah - 11/11/2006 14:18
PHP beginner		nu werkt hij, bedankt voor al deze informatie

Gesponsorde links

Dit onderwerp is gesloten.

Actiefste leden van de maand

Actieve forumberichten