When I switched on the module on a year-old website, I was forced to manually one-by-one delete my previous 50+ sessions.

This patch wipes all previous sessions up to the amount except session_limit_max , when session_limit_auto_drop is enabled.

CommentFileSizeAuthor
#2 session_limit_disconnect_all_v2.patch1.32 KBbatje
session_limit_disconnect_all_v1.patch1.32 KBbatje

Comments

deekayen’s picture

deekayen commented
Status: Needs review » Needs work

Maybe I'm just reading the patch wrong, but it looks like it finds only 1 row that is the oldest session and kills that one. Shouldn't it disconnect *all* the oldest sessions leaving only the sessions left in the permissible limit range?

batje’s picture

batje commented
StatusFileSize
new session_limit_disconnect_all_v2.patch1.32 KB

Indeed, i did swap the two filenames while creating the patch.

batje’s picture

batje commented
Status: Needs work » Needs review

changed status

deekayen’s picture

deekayen commented

I'm rolling this in right now, but you might also want to combine it with http://drupal.org/project/session_expire

deekayen’s picture

deekayen commented
Status: Needs review » Fixed

Committed to CVS

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.