Closed (fixed)
Project:
Automated Logout
Version:
2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
16 Jan 2025 at 10:58 UTC
Updated:
14 May 2026 at 23:40 UTC
Jump to comment: Most recent
Comments
Comment #5
deaom commentedI can confirm that MR works as expected, as without it, the if check is never true. Rebased branch and setting status to RTBC.
Comment #6
the_g_bomb commentedWould it be worth mentioning anywhere that this needs an up-to-date version of persistent login? Perhaps we can add the note in the README that support is offered for version 2.2.2 onwards.
Willing to accept this as is, if not, as I don't use the module myself, and I note that 2.2.2 was a security update, so anyone running a version lower than that is running an insecure version.
Comment #7
the_g_bomb commentedThe other thought I have is that it would be better to consider using the validateToken() method per #3398250: Add persistent login module support D10. I can't say I am 100% confident about which is the best approach.
Comment #8
the_g_bomb commentedMerged, thank you