adamgriffiths / the-codeigniter-authentication-library

The Easiest Authentication Library for the CodeIgniter PHP Framework. — Read more

Sort by: Priority | Votes | Last Updated

Loading…

4.

0 votes

Users Table CI DB Prefix & Auth Library Config
0 comments Created 5 months ago by xdeskproductions

Title

Body
File: /libraries/My_Controller.php Lines: 80, 100, 112 All three lines bypass the CI dbprefix as well as the Auth Library $config['auth_user_table'] values in use of the table 'users'.
or cancel

File: /libraries/My_Controller.php
Lines: 80, 100, 112

All three lines bypass the CI dbprefix as well as the Auth Library $config['auth_user_table'] values in use of the table 'users'.

Comments

Parsed with GitHub Flavored Markdown
2.

0 votes

infinite redirecting loop
2 comments Created 5 months ago by bluevibe

Title

Body
The value of config['auth_login'] is used for both failed and successful logins. So, if you try to access some controller without being authenticated for it , you're redirected to config['auth_login'], but naturally, you're not authenticated for that either, so it redirects you .. to config['auth_login']. and so forth.
or cancel

The value of config['auth_login'] is used for both failed and successful logins.

So, if you try to access some controller without being authenticated for it , you're redirected to config['auth_login'], but naturally, you're not authenticated for that either, so it redirects you .. to config['auth_login']. and so forth.

Comments

adamgriffiths Thu Jul 30 11:06:11 -0700 2009 | link

Thanks for letting me know. I'm fixing this right now.

Comment
Thanks for letting me know. I'm fixing this right now.
or cancel

adamgriffiths Thu Jul 30 11:17:38 -0700 2009 | link

Fixed redirect error on unsuccessful login closed by 5d6b8e8

Comment
Fixed redirect error on unsuccessful login closed by 5d6b8e817981961743d1da1358f024b698ab2acd
or cancel

Parsed with GitHub Flavored Markdown
1.

0 votes

insufficient group privileges / group_id issue
2 comments Created 6 months ago by JadedEvan

Title

Body
Noticed a bug in the libraries/Auth.php file, near line 82: Reads: $user_level = $this->CI->session->userdata('group'); When it should be: $user_level = $this->CI->session->userdata('group_id'); As in the login() function in the same file you set the session variable as 'group_id', not 'group'
or cancel
Noticed a bug in the libraries/Auth.php file, near line 82:

Reads:
$user_level = $this->CI->session->userdata('group');

When it should be:
$user_level = $this->CI->session->userdata('group_id');

As in the login() function in the same file you set the session variable as 'group_id', not 'group'

Comments
JadedEvan Thu Jun 25 01:28:49 -0700 2009 | link
Similarly, I added a redirect based on the user's group settings at the end of the login() function. Again there is trouble because of the group/group_id issue. Around line 143:

$auth_type => $username, 'username' => $row['username'], 'user_id' => $row['id'], 'group_id' => $row['group_id'], 'logged_in' => TRUE );

Nowhere in that block is the 'group' being set. It'd be nice to access that instead of passing around the group_id.
Comment
Similarly, I added a redirect based on the user's group settings at the end of the login() function. Again there is trouble because of the group/group_id issue. Around line 143: $auth_type => $username, 'username' => $row['username'], 'user_id' => $row['id'], 'group_id' => $row['group_id'], 'logged_in' => TRUE ); Nowhere in that block is the 'group' being set. It'd be nice to access that instead of passing around the group_id.
or cancel
adamgriffiths Sat Jul 11 09:06:11 -0700 2009 | link

I have fixed this issue in the latest commit to the repo. The correct reference for the group ID is 'group_id' not 'group'.

Hopefully this clears things up.

If you'd like to double check this in the repo I can close this issue then.

Thanks.

Comment
I have fixed this issue in the latest commit to the repo. The correct reference for the group ID is 'group_id' not 'group'. Hopefully this clears things up. If you'd like to double check this in the repo I can close this issue then. Thanks.
or cancel

Parsed with GitHub Flavored Markdown

Labels

adamgriffiths / the-codeigniter-authentication-library

Pledgie Donations

Users Table CI DB Prefix & Auth Library Config

Comments

infinite redirecting loop

Comments

insufficient group privileges / group_id issue

Comments