Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Password strength validator, how would you improve this?

First of all, I am validating password length with StringLength validator so I want to keep that out of the PasswordStrength validator. Any ideas how to improve this?

I think my approach with arrays and array_diff is not very elegant but the only other way I can think of are regular expressions which is even more ugly.

<?php
class My_Validate_PasswordStrength extends Zend_Validate_Abstract
{
    const MSG_NO_NUMBER = 'msgNoNumber';
    const MSG_NO_LOWER_CASE_LETTER = 'msgNoLowerCaseLetter';
    const MSG_NO_UPPER_CASE_LETTER = 'msgNoUpperCaseLetter';

    protected $_messageTemplates = array(
        self::MSG_NO_NUMBER => "'%value%' must contain at least one number",
        self::MSG_NO_LOWER_CASE_LETTER => "'%value%' must contain at least one lower case letter",
        self::MSG_NO_UPPER_CASE_LETTER => "'%value%' must contain at least one upper case letter"
    );

    public function isValid($value)
    {
        $this->_setValue($value);

        $arr = str_split($value);
        $numbers = array('0', '1', '2', '3', '4', '5', '6', '7', '8', '9');
        $lowerCaseLetters = array('a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 
        'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z');
        $upperCaseLetters = array('A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 
        'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z');

        if (count(array_diff($numbers, $arr)) === 10) {
            $this->_error(self::MSG_NO_NUMBER);
            return FALSE;
        }

        if (count(array_diff($lowerCaseLetters, $arr)) === 26) {
            $this->_error(self::MSG_NO_LOWER_CASE_LETTER);
            return FALSE;
        }

        if (count(array_diff($upperCaseLetters, $arr)) === 26) {
            $this->_error(self::MSG_NO_UPPER_CASE_LETTER);
            return FALSE;
        }

        return TRUE;
    }
}
like image 696
Richard Knop Avatar asked Sep 25 '11 13:09

Richard Knop


1 Answers

I don't think regular expressions have to be ugly.

public function isValid($value)
{
    $this->_setValue($value);

    if (preg_match('/[0-9]/', $value) !== 1) {
        $this->_error(self::MSG_NO_NUMBER);
        return FALSE;
    }

    if (preg_match('/[a-z]/', $value) !== 1) {
        $this->_error(self::MSG_NO_LOWER_CASE_LETTER);
        return FALSE;
    }

    if (preg_match('/[A-Z]/', $value) !== 1) {
        $this->_error(self::MSG_NO_UPPER_CASE_LETTER);
        return FALSE;
    }

    return TRUE;
}
like image 143
nachito Avatar answered Oct 07 '22 07:10

nachito