darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-06 09:38:25 -04:00
Code Issues Packages Projects Releases Wiki Activity

Moved condition back to top of method

Browse Source
This commit is contained in:
alexjanousekGSA
2024-10-04 13:49:10 -04:00
parent c5b2535b8e
commit 73b3a61746
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
app/main/views/sign_in.py
View File

@@ -203,6 +203,11 @@ def _handle_e2e_tests(redirect_url): # pragma: no cover
@main.route("/sign-in", methods=(["GET", "POST"])) @main.route("/sign-in", methods=(["GET", "POST"]))
@hide_from_search_engines @hide_from_search_engines
def sign_in(): # pragma: no cover def sign_in(): # pragma: no cover
redirect_url = request.args.get("next")
if os.getenv("NOTIFY_E2E_TEST_EMAIL"):
return _handle_e2e_tests(redirect_url)
# If we have to revalidated the email, send the message # If we have to revalidated the email, send the message
# via email and redirect to the "verify your email page" # via email and redirect to the "verify your email page"
# and don't proceed further with login # and don't proceed further with login
@@ -214,11 +219,6 @@ def sign_in(): # pragma: no cover
): ):
return email_verify_template return email_verify_template
redirect_url = request.args.get("next")
if os.getenv("NOTIFY_E2E_TEST_EMAIL"):
return _handle_e2e_tests(redirect_url)
if current_user and current_user.is_authenticated: if current_user and current_user.is_authenticated:
if redirect_url and is_safe_redirect_url(redirect_url): if redirect_url and is_safe_redirect_url(redirect_url):
return redirect(redirect_url) return redirect(redirect_url)