]> git.ipfire.org Git - ipfire.org.git/commitdiff
Revert "auth: Wrap login and logout in a single transaction"
authorMichael Tremer <michael.tremer@ipfire.org>
Tue, 7 May 2019 15:54:17 +0000 (16:54 +0100)
committerMichael Tremer <michael.tremer@ipfire.org>
Tue, 7 May 2019 15:54:17 +0000 (16:54 +0100)
This reverts commit 32d0ee1cba55825d13b192c6d53d8bd507d76a6b.

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
src/web/auth.py

index 2fc70eec09a56e0e239e2878ba3dfd1ee53e61c1..e080ff70aab054392a2a3f51d2c853d29f96a5a7 100644 (file)
@@ -20,9 +20,8 @@ class AuthenticationMixin(object):
 
        def login(self, account):
                # User has logged in, create a session
-               with self.db.transaction():
-                       session_id, session_expires = self.backend.accounts.create_session(
-                               account, self.request.host)
+               session_id, session_expires = self.backend.accounts.create_session(
+                       account, self.request.host)
 
                # Check if a new session was created
                if not session_id:
@@ -37,10 +36,7 @@ class AuthenticationMixin(object):
                if not session_id:
                        return
 
-               # Destroy session
-               with self.db.transaction():
-                       success = self.backend.accounts.destroy_session(session_id, self.request.host)
-
+               success = self.backend.accounts.destroy_session(session_id, self.request.host)
                if success:
                        self.clear_cookie("session_id")