Changes to session breaks access to masquerading user.

Created on 10 August 2024, 8 months ago

Problem/Motivation

Changes made in the parent issue relating to updating the last access time resulting in internal changes to the storage of the masquerading user's uid. That issue introduced a service with a method to return if is a user is masquerading, but not a public method to return who is masquerading.

My use case is that in some instances the original user should be used, such as when they are sending email on behalf of another user.

Proposed resolution

Simply change the protected method getMasquerate() to public.

This will be backward-compatible with all other modules.

✨ Feature request

Status

Active

Version

2.0

Component

Code

Created by

🇺🇸United States danchadwick Boston

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @danchadwick
Status changed to Needs review 8 months ago8:40pm 10 August 2024
Comment 8 months ago →
🇺🇸United States danchadwick Boston
Here's a patch for anyone that might need it.
Status changed to Closed: duplicate about 1 month ago5:44pm 20 February 2025
Comment about 1 month ago →
solideogloria
Closing as duplicate of issue with a better description and a MR instead of a patch. Please review/test the change on that issue, and you should get issue credit there as well.

✨ Make Masquerade::getMasquerade public Active

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024