Homec4science

Nuke sessions from the database when users logout

Authored by epriestley <git@epriestley.com> on Sep 8 2011, 23:16.

Description

Nuke sessions from the database when users logout

Summary:
@tomo ran into an issue where he had some non-SSL-only cookie or whatever, so
"Logout" had no apparent effect. Make sure "Logout" really works by destroying
the session.

I originally kept the sessions around to be able to debug session stuff, but we
have a fairly good session log now and no reprorted session bugs except for all
the cookie stuff. It's also slightly more secure to actually destroy sessions,
since it means "logout" breaks any cookies that attackers somehow stole (e.g.,
by reading your requests off a public wifi network).

Test Plan: Commented out the cookie clear and logged out. I was logged out and
given a useful error message about clearing my cookies.

Reviewers: jungejason, nh, tuomaspelkonen, aran

Reviewed By: aran

CC: tomo, aran, epriestley

Differential Revision: 911

Details

Committed
epriestley <git@epriestley.com>Sep 8 2011, 23:30
Pushed
aubortJan 31 2017, 17:16
Parents
rPH206546a6e305: Merge pull request #62 from abdul/master
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH87309734cc5c: Nuke sessions from the database when users logout (authored by epriestley <git@epriestley.com>).Sep 8 2011, 23:30