adamgriffiths / the-codeigniter-authentication-library
- Source
- Commits
- Network (5)
- Issues (5)
- Downloads (2)
- Graphs
-
Branch:
master
click here to add a description
click here to add a homepage
-
on line 96 it uses a meta refresh for the redirect. unless this is very clever for some reason I suggest it changed into a normal header location redirect.
Comments
-
I frequently leave my computer on and happened to leave Firefox open with myself at the login screen for a very long time, so when I came back to working on things I put in the wrong information to begin with but when I did I got an error:
A PHP Error was encountered Severity: Notice Message: Undefined index: login_attempts Filename: libraries/Auth.php Line Number: 325
As best I could tell, this was due to not checking if the session existed or something. Anyways I thought I would share it.
Comments
adamgriffiths
Sun Aug 02 03:59:09 -0700 2009
| link
What version of The Authentication Library are you using?
This issue was fixed a few revisions ago.
Thanks.
xdeskproductions
Sun Aug 02 06:39:01 -0700 2009
| link
First time downloading it a few days ago.
Please log in to comment.
adamgriffiths
Sat Aug 08 04:44:59 -0700 2009
| link
I found the problem. I'll work out the best way to fix it and commit soon.
Thanks.
-
Other than the obvious not being able to get the random url, in the offline processing your line 361 has the following code which in itself produces an error.
[code] $token = ”; [/code]
I fixed the issue using '' instead of the special character quotation.
Only reason I noticed the error is I am driving to Anaheim, CA for Blizzcon and working in the backseat off an AT&T Wireless card. Should produce similar results by gasp unplugging.
Comments
Please log in to comment. -
I realize it is only the header text, but it is misleading that a regular user and admin see the same page title 'admin panel'.
Comments
Please log in to comment. -
Logged out user should be redirected to login screen
0 comments Created about 1 month ago by konModification below:
<h2>Logout</h2> <div class="box"> You have been successfully logged out! Redirecting... </div>
<!script type="text/JavaScript">
<!--<br/> function timedRefresh(timeoutPeriod) {<br/> </p> <pre><code>setTimeout("window.location = '/login';",timeoutPeriod); </code></pre> <p>} window.onload=timedRefresh(3000);<br/> // --> </!script>
Comments
Please log in to comment.




Is there a specific problem with using the helper redirect() function?
I have no qualms with changing it I would just like to know the benefits.
Thanks.
Hi Adam
I think it could be that this is not a big deal, that's why this is more of a question.
The only issue I've had with it is that it didn't redirect me at all (becase I had disabled meta redirects in my browser). But I'm pretty sure almost no one has that.
With a auth solution I feel that I would take that option away from the user, "if you're not authenticated, get out", and be sure that they are infact redirected, which they would be in all cases as far as I know with a location redirect (default).
But maybe you had some good reasons for going for the refresh redirect, if so please share :)