From 9cb3b9ea786a3e27426558f5ab8009bfa8008c6a Mon Sep 17 00:00:00 2001
From: Rebecca Callis <rldavid@clemson.edu>
Date: Mon, 6 Jan 2025 09:43:04 -0500
Subject: [PATCH] feat(config): Rework how we grab the `mfe_config` based on
 domain change.

With a S3/Cloudfront distribution fro the MFE frontends, we went with a https://{mfe}.{lms-domain} change making the MFE app a subdomain of the lms domain.

Example of additional `BASE_URL` needed within each MFE app configuraiton.
```
{
    "MFE_CONFIG": {
        "BASE_URL": "https://apps.{lms-domain}"
    },
    "MFE_CONFIG_ACCOUNT": {
        "BASE_URL": "https://account.{lms-domain}"
    },
    "MFE_CONFIG_COURSE-AUTHORING": {
        "BASE_URL": "https://course-authoring.{lms-domain}"
    },
    "MFE_CONFIG_GRADEBOOK": {
        "BASE_URL": "https://gradebook.{lms-domain}"
    },
    "MFE_CONFIG_LEARNING": {
        "BASE_URL": "https://learning.{lms-domain}"
    },
    "MFE_CONFIG_PROFILE": {
        "BASE_URL": "https://profile.{lms-domain}"
    }
}
```
---
 lms/djangoapps/mfe_config_api/views.py | 30 +++++++++++++++++++-------
 1 file changed, 22 insertions(+), 8 deletions(-)

diff --git a/lms/djangoapps/mfe_config_api/views.py b/lms/djangoapps/mfe_config_api/views.py
index 14773d22c670..25b9e16f6456 100644
--- a/lms/djangoapps/mfe_config_api/views.py
+++ b/lms/djangoapps/mfe_config_api/views.py
@@ -65,26 +65,40 @@ def get(self, request):
             referer = re.sub(r'^https?:\/\/', '', request.META.get('HTTP_REFERER')).split('/')[0]  # pylint: disable=anomalous-backslash-in-string
 
             for site_config in SiteConfiguration.objects.all():
+                configuration = getattr(site_config.site, "configuration", None)
                 mfe_config = site_config.site_values.get("MFE_CONFIG", {})
+
+                if request.query_params.get('mfe'):
+                    mfe = str(request.query_params.get('mfe')).upper()
+
+                    mfe_config.update(configuration.get_value(
+                        f'MFE_CONFIG_{mfe}', getattr(settings, f'MFE_CONFIG_{mfe}', {}))
+                    )
+
                 if mfe_config.get("BASE_URL"):
                     mfe_config_base_url = re.sub(r'^https?:\/\/', '', mfe_config.get("BASE_URL")).split('/')[0]  # pylint: disable=anomalous-backslash-in-string
 
+                    log.info(
+                        "MFEConfigView.get - mfe_config_base_url (%s), referrer (%s)",
+                        mfe_config_base_url, referer
+                    )
                     if mfe_config_base_url == referer:
                         log.info(
                             "Found the site configuration that matches the MFE base domain."
                         )
 
-                        configuration = getattr(site_config.site, "configuration", None)
+                        # configuration = getattr(site_config.site, "configuration", None)
 
-                        mfe_config = configuration.get_value(
-                            'MFE_CONFIG', getattr(settings, 'MFE_CONFIG', {}))
+                        # mfe_config = configuration.get_value(
+                        #     'MFE_CONFIG', getattr(settings, 'MFE_CONFIG', {})
+                        # )
 
-                        if request.query_params.get('mfe'):
-                            mfe = str(request.query_params.get('mfe')).upper()
+                        # if request.query_params.get('mfe'):
+                        #     mfe = str(request.query_params.get('mfe')).upper()
 
-                            mfe_config.update(configuration.get_value(
-                                f'MFE_CONFIG_{mfe}', getattr(settings, f'MFE_CONFIG_{mfe}', {}))
-                            )
+                        #     mfe_config.update(configuration.get_value(
+                        #         f'MFE_CONFIG_{mfe}', getattr(settings, f'MFE_CONFIG_{mfe}', {}))
+                        #     )
 
                         # Exit out of the loop once you find first correct
                         # MFE_CONFIG in Site Configuration.