The modules "Remember me" or "Persistant login" ables the user to stay logged in. The Automated Logout module forces that user to be logged off anyway. Is there a way to check if the user want to be "remembered" and keep the session?

CommentFileSizeAuthor
#6 autologout-no_logout_on_remember_me-1604994-7461474.patch1.58 KBjohnennew
#4 autologout-no_logout_on_remember_me-1604994-7461224.patch1.58 KBjohnennew
#2 autologout-no_logout_on_remember_me-1604994.patch622 bytesrdatar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dandrews’s picture

dandrews CreditAttribution: dandrews commented

What I am getting is that you want it so if a user checks the box to "Keep me logged in"/"Remember me" what ever its titled, they will not be autologedout. And if the user does not click that box to be autologedout. It seems to be almost a niche market however it shouldn't be hard to add. Ill add it to my list and will post back within a few days with progress or a patch.

rdatar’s picture

rdatar CreditAttribution: rdatar commented
Status: Active » Needs review
FileSize
autologout-no_logout_on_remember_me-1604994.patch622 bytes

You can try this patch for autologout & remember_me to sync up.
With this patch, if the user has set remember_me as TRUE using remember_me module, the on-idle timeout generated by autologout is ignored.
Ideal when you want autologout for non-remember_me users while wanting remember_me to set custom timeout for users clicking remember_me.

johnennew’s picture

johnennew CreditAttribution: johnennew commented
Version: 7.x-2.x-dev » 7.x-4.x-dev
Status: Needs review » Needs work

Moving to head to check if this can be added to the supported 4.x branch.

johnennew’s picture

johnennew CreditAttribution: johnennew commented
Status: Needs work » Needs review
FileSize
autologout-no_logout_on_remember_me-1604994-7461224.patch1.58 KB

Patch for 7.x-4.x branch attached with tests.

johnennew’s picture

johnennew CreditAttribution: johnennew commented
Version: 7.x-4.x-dev » 6.x-4.x-dev
Status: Needs review » Needs work

Tests pass! Committed to 7.x-4.x, setting to 6.x-4.x for the back port

johnennew’s picture

johnennew CreditAttribution: johnennew commented
Status: Needs work » Needs review
FileSize
autologout-no_logout_on_remember_me-1604994-7461474.patch1.58 KB

backport for 6.x-4.x attache for review

johnennew’s picture

johnennew CreditAttribution: johnennew commented
Status: Needs review » Fixed

Tests pass! Committed to 6.x-4.x branch. Closing issue.

Status: Fixed » Closed (fixed)

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

  • Commit 025c8d7 on 7.x-4.x, 8.x-1.x authored by rdatar, committed by ceng: 
    Issue #1604994 by rdatar and ceng: Added support for the remember_me...