minor

ITISFoundation · May 2, 2023 · c9b60c0 · c9b60c0
1 parent 965c701
commit c9b60c0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/...ervice_webserver/security/_permissions.py → ...core_service_webserver/security/_utils.py b/...ervice_webserver/security/_permissions.py → ...core_service_webserver/security/_utils.py
diff --git a/services/web/server/src/simcore_service_webserver/security/plugin.py b/services/web/server/src/simcore_service_webserver/security/plugin.py
@@ -6,7 +6,6 @@
     See login/decorators.py
     Based on https://aiohttp-security.readthedocs.io/en/latest/
 """
-# pylint: disable=assignment-from-no-return
 import logging
 
 import aiohttp_security
@@ -29,9 +28,11 @@ def setup_security(app: web.Application):
     identity_policy = SessionIdentityPolicy()
 
     # TODO: limitations is that it cannot contain checks need to be added here
-    access_model = RoleBasedAccessModel.from_rawdata(ROLES_PERMISSIONS)
+    role_based_access_model = RoleBasedAccessModel.from_rawdata(ROLES_PERMISSIONS)
 
     # TODO: create basic/bearer authentication policy based on tokens instead of cookies!!
     # when you do that, also update the openapi to reflect that
-    authorization_policy = AuthorizationPolicy(app, access_model)
+    authorization_policy = AuthorizationPolicy(
+        app, access_model=role_based_access_model
+    )
     aiohttp_security.setup(app, identity_policy, authorization_policy)