Software / code / prosody
Changeset
13678:acb87cc2d48b 13.0
mod_authz_internal: Fix return values of secondary role management methods
usermanager expects (role, err) and (ok, err)
| author | Matthew Wild <mwild1@gmail.com> |
|---|---|
| date | Thu, 13 Feb 2025 15:30:23 +0000 |
| parents | 13676:40c309f8b676 |
| children | 13679:f5c7fe7bbe3b |
| files | plugins/mod_authz_internal.lua |
| diffstat | 1 files changed, 8 insertions(+), 3 deletions(-) [+] |
line wrap: on
line diff
--- a/plugins/mod_authz_internal.lua Thu Feb 13 14:00:08 2025 +0000 +++ b/plugins/mod_authz_internal.lua Thu Feb 13 15:30:23 2025 +0000 @@ -180,14 +180,19 @@ 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)
