Merge pull request #375 from magni2000/correct_config_url_validation

This corrects an issue with the validation of the org url and app url…
Nike-Inc · Dec 21, 2022 · 5034e62 · 5034e62
2 parents b6a37cc + 555786d
commit 5034e62
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/gimme_aws_creds/config.py b/gimme_aws_creds/config.py
@@ -320,7 +320,7 @@ def _get_org_url_entry(self, default_entry):
                 "okta-emea.com",
             ]
 
-            if url_parse_results.scheme == "https" and okta_org_url in allowlist:
+            if url_parse_results.scheme == "https" and any(urlelement in url_parse_results.hostname for urlelement in allowlist):
                 okta_org_url_valid = True
             else:
                 ui.default.error(
@@ -366,7 +366,7 @@ def _get_appurl_entry(self, default_entry):
                 "okta-emea.com",
             ]
 
-            if url_parse_results.scheme == "https" and app_url in allowlist:
+            if url_parse_results.scheme == "https" and any(urlelement in url_parse_results.hostname for urlelement in allowlist):
                 okta_org_url_valid = True
             else:
                 ui.default.warning(