User tests: Successful: Unsuccessful:
Pull Request for Issue #45637 , #45032.
delete metadata from db too
When executed via the backend, the task successfully removes expired sessions.
When executed via CLI, the task is marked as "Completed" in the execution log, but expired session records remain in the database the session cleanup does not occur.
Expired session records (older than the configured lifetime) are removed from the #__session table both when:
The task is executed via the backend
The task is executed via the CLI (cron)
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
Status | New | ⇒ | Pending |
Category | ⇒ | Front End Plugins |
Labels |
Added:
PR-5.3-dev
|
I think it will be best to restore the code to your original commit. Could you please do that? Sorry for the inconvenience.
@joomdonation yes done and don't worry it is always better to have another couple of eyes
Sorry @alikon. Thinking more about it, this still does not work properly with the combination of following conditions:
To make it works for every possible cases, the if condition will be more complicated, difficult to understand for a very little benefit, so after thinking for a while, I think it will be best to restore the code to your original commit. Could you please do that? Sorry for the inconvenience.