[PHP] Sterk wachtwoord eisen aan gebruiker

[DeepFreeze.NL]

Met mijn basiskennis van PHP heb ik na een tijdje rommelen het volgende script in elkaar kunnen zetten:

PHP:

// Wachtwoord moet minimaal bestaan uit 8 tekens waarvan 1 kleine letter, 1 hoofdletter en 1 cijfer
$alpha_list = '/[a-z]/';
$caps_list = '/[A-Z]/';
$numeric_list = '/[0-9]/';
$symbol_list = '/^[~!@#$%^&*()-_=+[]{}\/;:<>?.]$/';

if (
strlen($_POST['wachtwoord']) > 7 // at least 7 chars
&& preg_match($alpha_list, $_POST['wachtwoord'])
&& preg_match($caps_list, $_POST['wachtwoord'])
&& preg_match($numeric_list, $_POST['wachtwoord'])
&& preg_match($symbol_list, $_POST['wachtwoord'])
) {
echo "goed";
} else {
die('Wachtwoord moet minimaal bestaan uit 8 tekens waarvan 1 kleine letter, 1 hoofdletter en 1 cijfer.');
}

Dit script is een onderdeel van een registratieformulier zodat nieuwe gebruikers alleen maar een sterk wachtwoord kunnen gebruiken. Nu krijg ik alleen dat er altijd een melding wordt gegeven dat het wachtwoord onjuist is, ook als ik aan die condities voldoe. Wat doe ik nou fout?

[Spider.007]

'$alpha_list'

Je moet variabelen natuurlijk niet binnen quotes zetten; als je dat dan toch wil moet je dubbele quotes gebruiken

[Verwijderd]

Doe eens

PHP:

echo '$alpha_list'

en je wist precies wat er fout gaat.
Zet nooit variabelen tussen quotes als dat niet hoeft. Met dubbele quotes is het niet nodig, en met enkele quotes werkt het in dit geval niet zoals jij verwacht.

Daarnaast is die $symbol_list niet eens een regular expression, en je zou over elk van de 4 preg_match calls een foutmelding horen krijgen omdat je geen geldige delimiters zou hebben.
Kortom, zorg er ook even voor dat je alle waarschuwingen/errors/notificaties te zien krijgt. Configureer je server zo dat je er fatsoenlijk op kunt programmeren.

Tot slot is het wellicht eenvoudiger om hiervoor de cracklib functies te gebruiken, maar die is niet altijd voor handen.

[ Voor 9% gewijzigd door Verwijderd op 01-04-2007 15:17 ]

[DeepFreeze.NL]

Ik heb die haakjes weggehaald, en de server warnings aanstaan.
Ik krijg nog steeds dezelfde melding (Wachtwoord moet minimaal bestaan ... ...) en geen andere warnings ofzo.

En echo $alpha_list geeft /[a-z]/ als uitkomst.

[ Voor 12% gewijzigd door DeepFreeze.NL op 01-04-2007 15:27 ]

[Verwijderd]

Het wachtwoord waarmee je test is langer dan 7 tekens, dus minstens 8?
En heb je die 4e regular expression al gerepareerd met vierkante haakjes, en het escapen van de tekens die geëscapet moeten worden?

[Spider.007]

Warning: preg_match(): No ending delimiter '~' found in test.php on line 12

Maar als ik jou was zou ik nog eens een regexp handleiding doorlezen want de missende delimiter is niet het enige wat er fout is. Tip; kijk eens naar je werkende character-blocks; en vergelijk die met de laatste

[WormLord]

Wat wil je eigenlijk met die match op $symbol_list controleren?
Want als ik de correcte escaping er even bij denk, dan zie ik daar een controle of het wachtwoord bestaat uit precies 1 teken uit de opgegeven lijst. En dat zal dus altijd conflicteren met de eis van minimaal 8 teken .

[Verwijderd]

Die ^ aan het begin en $ aan het eind horen daar inderdaad niet. Die stonden er eerst ook niet. Ik vind het behoorlijk irritant dat de topicstart geëdit wordt, zo snapt dadelijk niemand er meer wat van.