Closed (fixed)
Project:
Session expire
Version:
6.x-1.x-dev
Component:
Documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
5 Sep 2010 at 08:36 UTC
Updated:
20 Sep 2010 at 21:40 UTC
I was using this module to automatically log users out after a certain time of inactivity. Reading a post on the planet recently, I learned that this can be achieved simply by changing a setting in settings.php. I guess this module can be useful for other things, but for automatic logouts I guess it is unneccessary. Assuming this is actually the case, I think this should be mentioned on the project page.
Thanks.
Comments
Comment #1
kbahey commentedMany differences:
1. The PHP parameter in settings.php does not work on all distributions by default (e.g. Debian derived ones).
2. This module makes it more predictable when sessions are expired, specially when paired with Elysia cron. It can be set to happen at 3 am in the morning when traffic is at its lowest.
3. This module executes from cron, and therefore happens in the background, not at random when someone is browsing the site, subject to the unpredictable whims of garbage collection of PHP.
4. This module is also more configurable since you can expire logged in or anonymous or both.
I tried to include the module's functionality in core and got the same response: "It is a PHP configuration issue", therefore I wrote a separate module for the above reasons.
Comment #2
tstoecklerRight. I'm not saying: "This module is useless" (at least I tried to convey that in the OP). I'm saying: "There are cases, where this module is unneccessary" And since I found this module by searching for something like "automatic logout" I think a lot of other people would also install this module and not know about the settings.php way. That's why I would encourage to add only a single line to the project page such as:
Comment #3
kbahey commentedLengthy explanation and relevant PHP settings are now on the project page.
Comment #4
tstoecklerWow that's pretty awesome. Thanks a lot!!!