mod_authz_internal: Fix return values of secondary role management methods

usermanager expects (role, err) and (ok, err)

1 files changed, 8 insertions, 3 deletions

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)