Merge pull request #1728 from GSA/notify-compliance-46d

another tweak for session management
GSA · Jul 16, 2024 · 2088a30 · 2088a30
2 parents b9e86f2 + 3922593
commit 2088a30
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/app/notify_client/__init__.py b/app/notify_client/__init__.py
@@ -67,6 +67,10 @@ def check_inactive_user(self, *args):
                 or "/email-code" in arg
             ):
                 still_signing_in = True
+
+            # This seems to be a weird edge case that happens intermittently with invites
+            if str(arg) == "()":
+                still_signing_in = True
         # TODO:  Update this once E2E tests are managed by a feature flag or some other main config option.
         if os.getenv("NOTIFY_E2E_TEST_EMAIL"):
             # allow end-to-end tests to skip check

diff --git a/poetry.lock b/poetry.lock