local-it/dashboard
0
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

#29-add-admin-role #2

Merged
philipp merged 10 commits from #29-add-admin-role into fork 2022-11-08 16:36:18 +01:00
Conversation 0 Commits 10 Files changed 38 +5 -5
2 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 755a443c96 - Show all commits

2
backend/areas/users/users.py
View file

@ -14,7 +14,7 @@ from .lit_user_service import UserService
@api_v1.route("/users", methods=["GET"]) @api_v1.route("/users", methods=["GET"])
@jwt_required() @jwt_required()
@cross_origin() @cross_origin()
@admin_required() # @admin_required() TODO: not needed as authentik checks permissions?
def get_users(): def get_users():
return jsonify(UserService.get_users()) return jsonify(UserService.get_users())

8
backend/helpers/auth_guard.py
View file

@ -11,10 +11,10 @@ def admin_required():
@wraps(fn) @wraps(fn)
def decorator(*args, **kwargs): def decorator(*args, **kwargs):
verify_jwt_in_request() verify_jwt_in_request()
# claims = get_jwt() claims = get_jwt()
# user_id = claims["user_id"]
is_admin = True # RoleService.is_user_admin(user_id) user_id = claims["user_id"]
# TODO: actually check if admin is_admin = RoleService.is_user_admin(user_id)
if is_admin: if is_admin:
return fn(*args, **kwargs) return fn(*args, **kwargs)
else: else: