From 7ad22561c74eedf78e1c80e075966d105c7fbc26 Mon Sep 17 00:00:00 2001 From: Alexander VT Date: Tue, 25 Jan 2022 15:08:41 -0600 Subject: [PATCH] fix(passport): don't close session as it needs to be used downstream --- fence/blueprints/login/ras.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/fence/blueprints/login/ras.py b/fence/blueprints/login/ras.py index fd2046af7..88f52cc4c 100644 --- a/fence/blueprints/login/ras.py +++ b/fence/blueprints/login/ras.py @@ -57,11 +57,6 @@ def post_login(self, user=None, token_result=None, id_from_idp=None): # do an on-the-fly usersync for this user to give them instant access after logging in through RAS # if GLOBAL_PARSE_VISAS_ON_LOGIN is true then we want to run it regardless of whether or not the client sent parse_visas on request if parse_visas: - # Close previous db sessions. Leaving it open causes a race condition where we're - # viewing user.project_access while trying to update it in usersync - # not closing leads to partially updated records - current_session.close() - # get passport then call sync on it try: passport = (