Download & Extend

Instead of only deleting 1 old session, delete all old sessions

Project:Session Limit
Version:6.x-1.0
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

AttachmentSize
session_limit_disconnect_all_v1.patch1.32 KB

Comments

#1

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?

#2

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

AttachmentSize
session_limit_disconnect_all_v2.patch 1.32 KB

#3

Status:needs work» needs review

changed status

#4

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

#5

Status:needs review» fixed

Committed to CVS

#6

Status:fixed» closed (fixed)

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

nobody click here