Skip to content

Commit

Permalink
"Remember Me" Improvement
Browse files Browse the repository at this point in the history
Fixes an edge case (described in Ticket benedmunds#1570) where people were able to login  if the "Remember Me" feature was disabled in the app but people still had old "Remember Me" cookies.
  • Loading branch information
chland committed Sep 28, 2022
1 parent e47b492 commit 5d0f748
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion libraries/Ion_auth.php
Original file line number Diff line number Diff line change
Expand Up @@ -359,7 +359,7 @@ public function logged_in()
$recheck = $this->ion_auth_model->recheck_session();

// auto-login the user if they are remembered
if (!$recheck && get_cookie($this->config->item('remember_cookie_name', 'ion_auth')))
if (!$recheck && ($this->config->item('remember_users', 'ion_auth')) && get_cookie($this->config->item('remember_cookie_name', 'ion_auth')))
{
$recheck = $this->ion_auth_model->login_remembered_user();
}
Expand Down

0 comments on commit 5d0f748

Please sign in to comment.