I’m using CodeIgniter 2 along with the Ion Auth authorization system by Ben Edmunds.
After creating my project, I would sometimes get a CodeIgniter error upon certain login attempts but this error was intermittent.
The action you have requested is not allowed.
After some troubleshooting, it became apparent this error was caused by an invalid CSRF token. Why is the token invalid? Well, in CodeIgniter’s configuration file, it’s set to expire in 4 hours. So if you load your login page and allow it to sit there for 4 hours before attempting a login, the CSRF tokens will expire and this will generate the error message as above. Simply reloading the login page avoids any issues.
You can verify this error message for yourself by deleting the CSRF cookie after you load the login page.
A cleaner solution would be to redirect to a custom error page or to display a flash message. However, this solution is not as simple as it sounds because when you extend the CodeIgniter Security class, certain hook-points are not available and you cannot yet access CodeIgniter’s Super Object using `get_instance()`.
So when you extend the Security class, you’re limited to standard PHP. In this case, I’m using PHP `header()` to redirect the offending login page (or any form page) back to itself.
<?php class MY_Security extends CI_Security { public function __construct() { parent::__construct(); } public function csrf_show_error() { // show_error('The action you have requested is not allowed.'); // default code // force page "refresh" - redirect back to itself with sanitized URI for security // a page refresh restores the CSRF cookie to allow a subsequent login header('Location: ' . htmlspecialchars($_SERVER['REQUEST_URI']), TRUE, 200); } }
This works fine except that the user gets a screen refresh without any indication why they have to enter their login credentials a seconds time.
I decided to make this a bit more user-friendly by adding another function into a Controller, in my case, the Ion Auth controller…
function csrf_redirect() { $flash = 'Session cookie automatically reset due to expired browser session. Please try again.'; $this->session->set_flashdata('message', $flash); redirect('/login', 'location'); }
As you can see, this function sets a flash message telling the user what happened and then redirects them to a fresh instance of the login page.
Session cookie automatically reset due to expired browser session. Please try again.
Instead of using PHP `header()` to redirect a page refresh, redirect to this new Ion Auth controller function at `/auth/csrf_redirect`.
<?php class MY_Security extends CI_Security { public function __construct() { parent::__construct(); } public function csrf_show_error() { // show_error('The action you have requested is not allowed.'); // default code // force redirect to the csrf_redirect function // this gives the user a useful message instructing them to login again // while the CSRF cookie is also refreshed to allow a new login header('Location: /auth/csrf_redirect', TRUE, 302); } }
The minor downside to this method is that you are always redirected back to the login page rather than a refresh of whatever page/form you’re trying to submit. However, that should be a moot point, since the session cookie expires at nearly the same time as the CSRF cookie, you’d be redirected back to the same login page regardless. You may also not be requiring the user be logged in for your particular form, so please be aware and re-direct accordingly.