From dc810fea2d37597aeddbe546886fb0f60d8c6c07 Mon Sep 17 00:00:00 2001 From: Matthew Wild Date: Thu, 13 Feb 2025 15:30:23 +0000 Subject: mod_authz_internal: Fix return values of secondary role management methods usermanager expects (role, err) and (ok, err) --- plugins/mod_authz_internal.lua | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'plugins/mod_authz_internal.lua') diff --git a/plugins/mod_authz_internal.lua b/plugins/mod_authz_internal.lua index 7a06c904..b1df5579 100644 --- a/plugins/mod_authz_internal.lua +++ b/plugins/mod_authz_internal.lua @@ -180,14 +180,19 @@ function set_user_role(user, role_name) end function add_user_secondary_role(user, role_name) - if not role_registry[role_name] then + local role = role_registry[role_name]; + if not role then return error("Cannot assign default user an unknown role: "..tostring(role_name)); end - role_map_store:set(user, role_name, true); + local ok, err = role_map_store:set(user, role_name, true); + if not ok then + return nil, err; + end + return role; end function remove_user_secondary_role(user, role_name) - role_map_store:set(user, role_name, nil); + return role_map_store:set(user, role_name, nil); end function get_user_secondary_roles(user) -- cgit v1.2.3