Skip to content

Commit

Permalink
fix: Redirect non-enterprise SAML to authn MFE
Browse files Browse the repository at this point in the history
The original request was that enterprise users with tpa hint and SAML should not be redirected to MFE. The current condition also excludes regular non-enterprise users with SAML authentication from the MFE.
  • Loading branch information
angonz committed Nov 27, 2024
1 parent 1ea8996 commit 0cde3ca
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion openedx/core/djangoapps/user_authn/views/login_form.py
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ def login_and_registration_form(request, initial_mode="login"):
enterprise_customer = enterprise_customer_for_request(request)

if should_redirect_to_authn_microfrontend() and not \
(enterprise_customer and tpa_hint_provider and saml_provider):
(enterprise_customer and tpa_hint_provider and saml_provider):

# This is to handle a case where a logged-in cookie is not present but the user is authenticated.
# Note: If we don't handle this learner is redirected to authn MFE and then back to dashboard
Expand Down

0 comments on commit 0cde3ca

Please sign in to comment.