Welcome Guest, Not a member yet? Register   Sign In
The Auth Library 1.0.3 - The Easiest Auth Library for CodeIgniter
#23

[eluser]gyo[/eluser]
Hi Adam,
I really appreciate your effort in making this library, but I think it still needs to grow.
Auth (authentication and authorization) is a very critical step, let's say it has to be bomb proof... Wink

Here is some feedback:
You should remove all the checking like min and max password chars from the lib; I would do that kind of checks in a model, without throwing all the error messages with show_error() because that limits a lot the flexibility.
I see you're loading all the libraries and config twice (both in __constructor and Auth methods); you're mixing php4 and php5, and the only element of php5 is the __constructor. Just remove it, since all the library is php4 and it will work in php5, or optimize it for php5 without backward compatibility.
Since I'm paranoid about security I would filter all the input->post for XSS, apart from checking the data types:

$this->input->post('some_data', TRUE);

Take a look at Redux Auth which has been developed with security as the main idea:

http://codeigniter.com/wiki/Redux_Auth/


Anyway... I'm trying to be constructive with my comments, and I'm always open to different points of view! Wink


Byee


Messages In This Thread
The Auth Library 1.0.3 - The Easiest Auth Library for CodeIgniter - by El Forum - 03-10-2009, 03:26 AM



Theme © iAndrew 2016 - Forum software by © MyBB