paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-10-23 04:52:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

sessions: Use the narrow user fetch when expiring sessions.

Browse Source
This commit is contained in:
Alex Vandiver
2025-02-11 19:42:19 +00:00
committed by Tim Abbott
parent 58bf2a7935
commit 44f0e936c2
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
zerver/lib/sessions.py
View File

@@ -16,7 +16,7 @@ from django.utils.timezone import now as timezone_now
from zerver.lib.request import RequestNotes
from zerver.lib.timestamp import datetime_to_timestamp, timestamp_to_datetime
from zerver.models import Realm, UserProfile
from zerver.models.users import get_user_profile_by_id, get_user_profile_narrow_by_id
from zerver.models.users import get_user_profile_narrow_by_id
class SessionEngine(Protocol):
@@ -71,7 +71,7 @@ def delete_all_deactivated_user_sessions() -> None:
user_profile_id = get_session_user_id(session)
if user_profile_id is None: # nocoverage # TODO: Investigate why we lost coverage on this
continue
user_profile = get_user_profile_by_id(user_profile_id)
user_profile = get_user_profile_narrow_by_id(user_profile_id)
if not user_profile.is_active or user_profile.realm.deactivated:
logging.info("Deactivating session for deactivated user %s", user_profile.id)
delete_session(session)