Session management from a code review perspective should focus on the creation, renewal and destruction of a users session throughout the application. The code review process should ensure the following:
- authenticated users have a robust and cryptographically secure association with their session. The session identifier (Session ID) shoud not be predictable and generation of such should be left to the underlaying framework. The development effort to produce a session with sufficient entrophy is subject to errors and best left to tried and trusted methods.
- applications enforce authorization checks
Applications should check if the session is valid prior to servicing any user requests. The users session object may also hold authorisation data.
- applications avoid or prevent common web attacks, such as replay, request forging and man-in-the-middle.
Session identifiers should be passed to the user in a secure manner such as not using HTTP GET with the session ID being placed in the query string. - Such data (query string) is logged in web server logs.