Exception thrown when session doesn't exist

Created on 19 August 2024, about 1 year ago

After upgrading to autologout 1.5, drush commands that execute as a particular user throw an exception. This problem is also present on version 2.0.

Install CiviCRM:

composer config extra.enable-patching true
composer require civicrm/civicrm-{core,packages,drupal-8}
composer require civicrm/cli-tools

Use the `drush cvapi` command, i.e. `drush cvapi System.getactions`.

This occurs due to the change to `autologout_check_session_variable` in commit ae12b9b1d40343b7f85fbefc71b2f32e43cbce8e

Wrapping the call to `getSession()` in an `if` that checks if the session exists resolves this.

None (after patch review).

None.

None.

None.

🐛 Bug report

Status

Active

Version

2.0

Component

Code

Created by

🇺🇸United States MegaphoneJon

Live updates comments and jobs are added and updated live.

Merge Requests

!64Exception thrown when session doesn't exist
Open
🇺🇸United States MegaphoneJon
updated about 1 year ago

Issue created by @MegaphoneJon
Merge request !64Check for session existence before getting session variable → (Open) created by MegaphoneJon
Pipeline finished with Success
about 1 year ago
Total: 468s
#258623
Pipeline finished with Success
about 1 year ago
Total: 414s
#258624
Status changed to Needs review 8 months ago8:46pm 18 December 2024
Comment 8 months ago →
🇺🇦Ukraine UsingSession
Issue was unassigned.
Status changed to RTBC 19 days ago10:22am 6 August 2025
Comment 19 days ago →
🇸🇮Slovenia deaom
I could not reproduce this issue with the provided steps, but since it's just an additional check, I see no harm in this being merged, so will RTBC it.

Production build 0.71.5 2024