🐛[#114] make endpoints required

maykinmedia · Aug 2, 2024 · aca9492 · aca9492
1 parent aca7802
commit aca9492
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/docs/setup_configuration.rst b/docs/setup_configuration.rst
@@ -42,9 +42,11 @@ Required:
 
 ::
 
-    ADMIN_OIDC_DEFAULT_GROUPS
     ADMIN_OIDC_OIDC_RP_CLIENT_ID
     ADMIN_OIDC_OIDC_RP_CLIENT_SECRET
+    ADMIN_OIDC_OIDC_OP_AUTHORIZATION_ENDPOINT
+    ADMIN_OIDC_OIDC_OP_TOKEN_ENDPOINT
+    ADMIN_OIDC_OIDC_OP_USER_ENDPOINT
 
 
 All settings:

diff --git a/mozilla_django_oidc_db/setupconfig/boostrap.py b/mozilla_django_oidc_db/setupconfig/boostrap.py
@@ -17,16 +17,16 @@ class AdminOIDCConfigurationStep(BaseConfigurationStep):
     required_settings = [
         "ADMIN_OIDC_OIDC_RP_CLIENT_ID",
         "ADMIN_OIDC_OIDC_RP_CLIENT_SECRET",
+        "ADMIN_OIDC_OIDC_OP_AUTHORIZATION_ENDPOINT",
+        "ADMIN_OIDC_OIDC_OP_TOKEN_ENDPOINT",
+        "ADMIN_OIDC_OIDC_OP_USER_ENDPOINT",
     ]
     all_settings = required_settings + [
         "ADMIN_OIDC_OIDC_RP_SCOPES_LIST",
         "ADMIN_OIDC_OIDC_RP_SIGN_ALGO",
         "ADMIN_OIDC_OIDC_RP_IDP_SIGN_KEY",
         "ADMIN_OIDC_OIDC_OP_DISCOVERY_ENDPOINT",
         "ADMIN_OIDC_OIDC_OP_JWKS_ENDPOINT",
-        "ADMIN_OIDC_OIDC_OP_AUTHORIZATION_ENDPOINT",
-        "ADMIN_OIDC_OIDC_OP_TOKEN_ENDPOINT",
-        "ADMIN_OIDC_OIDC_OP_USER_ENDPOINT",
         "ADMIN_OIDC_USERNAME_CLAIM",
         "ADMIN_OIDC_GROUPS_CLAIM",
         "ADMIN_OIDC_CLAIM_MAPPING",