tags:

views:

5814

answers:

11

What are some guidelines for maintaining responsible session security with PHP? There's information all over the web, so it's about time it all landed in one place!

+7  A: 

One guideline is to call session_regenerate_id every time a session's security level changes. This helps prevent session hijacking.

saint_groceon
+1  A: 

I think one of the major problems (which is being addressed in PHP 6) is register_globals. Right now one of the standard methods used to avoid register_globals is to use the $_REQUEST, $_GET or $_POST arrays.

The "correct" way to do it (as of 5.2, although it's a little buggy there, but stable as of 6, which is coming soon) is through filters.

So instead of:

$username = $_POST["username"];

you would do:

$username = filter_input(INPUT_POST, 'username', FILTER_SANITIZE_STRING);

or even just:

$username = filter_input(INPUT_POST, 'username');

cmcculloh
This has no relation to the question at all.
The Pixel Developer
Really? Then why in the accepted answer do they mention not to use register globals? Wouldn't, as far as most run-of-the-mill developers are concerned, register globals and form variable handling fall under the umbrella of "sessions" even if it isn't technically part of the "session" object?
cmcculloh
-1 This does not answer the question.
Tomas
I agree, this does not *fully* answer the question, but it is definitely PART of the answer to the question. Again, this fleshes out a bullet point in the accepted answer, "Don't use register globals". This tells what to do instead.
cmcculloh
+2  A: 

This is pretty trivial and obvious, but be sure to session_destroy after every use. This can be difficult to implement if the user does not log out explicitly, so a timer can be set to do this.

Here is a good tutorial on setTimer() and clearTimer().

contagious
+2  A: 

The main problem with PHP sessions and security (besides session hijacking) comes with what environment you are in. By default PHP stores the session data in a file in the OS's temp directory. Without any special thought or planning this is a world readable directory so all of your session information is public to anyone with access to the server.

As for maintaining sessions over multiple servers. At that point it would be better to switch PHP to user handled sessions where it calls your provided functions to CRUD (create, read, update, delete) the session data. At that point you could store the session information in a database or memcache like solution so that all application servers have access to the data.

Storing your own sessions may also be advantageous if you are on a shared server because it will let you store it in the database which you often times have more control over then the filesystem.

John Downey
+1  A: 

I would check both IP and User Agent to see if they change

if ($_SESSION['user_agent'] != $_SERVER['HTTP_USER_AGENT']
    || $_SESSION['user_ip'] != $_SERVER['REMOTE_ADDR'])
{
    //Something fishy is going on here?
}
Teifion
IP can legitimately change if user is behind load-balanced proxy farm.
porneL
And user_agent can change every time a user upgrades their browser.
scotts
@scotts I agree with the IP part but for the browser upgrade, you would set the session when they login so I don't see how they would upgrade there browser without creating a new session once they login again.
jasondavis
+4  A: 

Using IP address isn't really the best idea in my experience. For example; my office has two IP addresses that get used depending on load and we constantly run into issues using IP addresses.

Instead, I've opted for storing the sessions in a separate database for the domains on my servers. This way no one on the file system has access to that session info. This was really helpful with phpBB before 3.0 (they've since fixed this) but it's still a good idea I think.

Eric Lamb
+1  A: 

If you you use session_set_save_handler() you can set your own session handler. For example you could store your sessions in the database. Refer to the php.net comments for examples of a database session handler.

DB sessions are also good if you have multiple servers otherwise if you are using file based sessions you would need to make sure that each webserver had access to the same filesystem to read/write the sessions.

ejunker
+33  A: 

There is a couple of things todo in order to keep your session secure:

  1. Use SSL when authenticating users or performing sensitive operations.
  2. Regenerate the session id whenever the security level changes (such as logging in). You can even regenerate the session id every request if you wish.
  3. Have sessions timeout
  4. Don't use register globals
  5. Store authentication details on the server. That is don't send details such as username in the cookie.
  6. Check the $_SERVER['HTTP_USER_AGENT']. This adds a small barrier to session hijacking. You can also check the IP address. But this causes problems for users that have changing IP address due to load balancing on multiple internet connections etc (which is the case in our environment here).
  7. Lock down access to the sessions on the file system or use custom session handling
  8. For sensitive operations consider requiring the user to provide their authenication details
grom
Using SSL only for some operations is not enough, unless you have separate sessions for encrypted and unencrypted traffic.If you use single session over HTTPS and HTTP, attacker will steal it on first non-HTTPS request.
porneL
I agree with porneL.Also, for number 6, if an attacker has your session id, wouldn't they also have access to your user agent?
Chad
If you regenerate the session id then the session id that an attacker steals on a non-HTTPS request is useless.
grom
-1 the user agent is trivial to spoof. What you are describing wastes code and is not a security system.
Rook
Damn i wish i could give you another -1 for use of ssl. At no point can the cookie be leaked over http, thats laid out in OWASP A3.
Rook
@The Rook, yes the User Agent can be spoofed. Its just one small little barrier. And what do you mean by at no point the cookie can be leaked over http. Yes it can be stolen. http is plain text.
grom
@grom the only barrier is the one in your mind, you are stopping no attack.
Rook
@The Rook, it may be a trivial barrier (the attacker can capture a victim's user-agent using their own site) and relies on security through obscurity but it is still one extra barrier. If the User-Agent HTTP was to change during the session use, it would be extremely suspicious and most likely an attack. I never said you can use it alone. If you combine it with the other techniques you have a much more secure site.
grom
@grom I think its like putting a piece of scotch tape across your door and saying it will prevent people from breaking in.
Rook
+1  A: 
Dinoboff
+6  A: 

This session fixation paper has very good pointers where attack may come. See also session fixation page at Wikipedia.

raspi
+4  A: 

My two (or more) cents:

  • Trust no one
  • Filter input, escape output (cookie, session data are your input too)
  • Avoid XSS (keep your HTML well formed, take a look at PHPTAL or HTMLPurifier)
  • Defense in depth
  • Do not expose data

There is a tiny but good book on this topic: Essential PHP Security by Chris Shiflett.

Essential PHP Security

On the home page of the book you will find some interesting code examples and sample chapters.

You may use technique mentioned above (IP & UserAgent), described here: How to avoid identity theft

takeshin