feat: Add environment variable to enable or disable authentication on…

… role listing
flavien-hugs · Sep 18, 2024 · 927d98e · 927d98e
1 parent 924482f
commit 927d98e
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/src/routers/roles.py b/src/routers/roles.py
@@ -22,8 +22,6 @@
             Depends(AuthorizedHTTPBearer),
             Depends(CheckPermissionsHandler(required_permissions={"auth:can-create-role"})),
         ]
-        if settings.LIST_ROLES_ENDPOINT_SECURITY_ENABLED
-        else []
     ),
     response_model=Role,
     summary="Create role",
@@ -36,10 +34,14 @@ async def create_role(payload: RoleModel = Body(...)):
 @role_router.get(
     "",
     response_model=customize_page(Role),
-    dependencies=[
-        Depends(AuthorizedHTTPBearer),
-        Depends(CheckPermissionsHandler(required_permissions={"auth:can-display-role"})),
-    ],
+    dependencies=(
+        [
+            Depends(AuthorizedHTTPBearer),
+            Depends(CheckPermissionsHandler(required_permissions={"auth:can-display-role"})),
+        ]
+        if settings.LIST_ROLES_ENDPOINT_SECURITY_ENABLED
+        else []
+    ),
     summary="Get all roles",
     status_code=status.HTTP_200_OK,
 )