Skip to content

Commit

Permalink
Merge pull request #14 from peazz/patch-1
Browse files Browse the repository at this point in the history
 !ctype_alnum($pass) is removed so now special chars (@ ! ?) can be used for password
  • Loading branch information
Emre Akay committed Jul 22, 2014
2 parents cf7b065 + 05c660c commit 77a532f
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions application/libraries/Aauth.php
Original file line number Diff line number Diff line change
Expand Up @@ -100,8 +100,16 @@ public function login($email, $pass, $remember = FALSE) {

$this->CI->input->set_cookie($cookie);

// verification
if( !valid_email($email) or !ctype_alnum($pass) or strlen($pass) < 5 or

/*
*
* User Verification
*
* Removed or !ctype_alnum($pass) from the IF statement
* It was causing issues with special characters in passwords
* and returning false even if the password matches.
*/
if( !valid_email($email) or strlen($pass) < 5 or
strlen($pass) > $this->config_vars['max'] )
{
$this->error($this->config_vars['wrong']);
Expand Down

0 comments on commit 77a532f

Please sign in to comment.