darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-18 21:49:37 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3287 from alphagov/check-current-user-is_authenticated

Browse Source 
Reorder the methods called in sign out
This commit is contained in:
Rebecca Law
2020-02-05 16:29:31 +00:00
committed by GitHub
parent a4a6ad04f7 039628cff7
commit 60dd6bca44
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/models/user.py
View File

@@ -154,10 +154,10 @@ class User(JSONModel, UserMixin):
return True return True
def sign_out(self): def sign_out(self):
# Update the db so the server also knows the user is logged out.
session.clear() session.clear()
# Update the db so the server also knows the user is logged out.
self.update(current_session_id=None)
logout_user() logout_user()
return self.update(current_session_id=None)
@property @property
def sms_auth(self): def sms_auth(self):