Skip to content

SSO/logistration: lazy‑load third_party_auth_context to break import cycle #225

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
subhashree-sahu31 wants to merge 2 commits into
base: release-ulmo
Choose a base branch
from
Open

SSO/logistration: lazy‑load third_party_auth_context to break import cycle #225

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3deed77
fix(enterprise-support): lazy-load third_party_auth_context to break …
subhashree-sahu31 Apr 9, 2026
4a693f4
fix(enterprise-support): lazy-load third_party_auth_context to break …
subhashree-sahu31 Apr 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion openedx/features/enterprise_support/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@
from openedx.core.djangoapps.site_configuration import helpers as configuration_helpers
from openedx.core.djangoapps.user_authn.cookies import standard_cookie_settings
from openedx.core.djangolib.markup import HTML, Text
from openedx.core.djangoapps.user_authn.views.utils import third_party_auth_context
from ipware import get_client_ip

ENTERPRISE_HEADER_LINKS = WaffleFlag('enterprise.enterprise_header_links', __name__) # lint-amnesty, pylint: disable=toggle-missing-annotation
Expand Down Expand Up @@ -508,6 +507,8 @@ def get_mfe_context(request, redirect_to, tpa_hint=None):
"""
# Import enterprise functions INSIDE the function to avoid circular import
from openedx.features.enterprise_support.api import enterprise_customer_for_request
# Importing third_party_auth_context here to avoid circular import since it imports get_mfe_context
from openedx.core.djangoapps.user_authn.views.utils import third_party_auth_context

ip_address = get_client_ip(request)[0]
country_code = country_code_from_ip(ip_address)
Expand Down
Loading