Mercurial > prosody-hg
comparison plugins/mod_authz_internal.lua @ 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 | eb676b6f05e3 |
| children | f5c7fe7bbe3b |
comparison
equal
deleted
inserted
replaced
| 13676:40c309f8b676 | 13678:acb87cc2d48b |
|---|---|
| 178 end | 178 end |
| 179 return role; | 179 return role; |
| 180 end | 180 end |
| 181 | 181 |
| 182 function add_user_secondary_role(user, role_name) | 182 function add_user_secondary_role(user, role_name) |
| 183 if not role_registry[role_name] then | 183 local role = role_registry[role_name]; |
| 184 if not role then | |
| 184 return error("Cannot assign default user an unknown role: "..tostring(role_name)); | 185 return error("Cannot assign default user an unknown role: "..tostring(role_name)); |
| 185 end | 186 end |
| 186 role_map_store:set(user, role_name, true); | 187 local ok, err = role_map_store:set(user, role_name, true); |
| 188 if not ok then | |
| 189 return nil, err; | |
| 190 end | |
| 191 return role; | |
| 187 end | 192 end |
| 188 | 193 |
| 189 function remove_user_secondary_role(user, role_name) | 194 function remove_user_secondary_role(user, role_name) |
| 190 role_map_store:set(user, role_name, nil); | 195 return role_map_store:set(user, role_name, nil); |
| 191 end | 196 end |
| 192 | 197 |
| 193 function get_user_secondary_roles(user) | 198 function get_user_secondary_roles(user) |
| 194 local stored_roles, err = role_store:get(user); | 199 local stored_roles, err = role_store:get(user); |
| 195 if not stored_roles then | 200 if not stored_roles then |