From 622e730017bbda6a498dda18798603807a68d4ff Mon Sep 17 00:00:00 2001 From: kakwa Date: Sun, 21 Jun 2015 23:41:24 +0200 Subject: [PATCH] fix adding multiple time to same group + logs --- ldapcherry/__init__.py | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/ldapcherry/__init__.py b/ldapcherry/__init__.py index 67ba017..7653ae6 100644 --- a/ldapcherry/__init__.py +++ b/ldapcherry/__init__.py @@ -27,6 +27,7 @@ from cherrypy.lib.httputil import parse_query_string #mako template engines imports from mako.template import Template from mako import lookup +from sets import Set SESSION_KEY = '_cp_username' @@ -428,7 +429,17 @@ class LdapCherry(object): roles.append(r) groups = self.roles.get_groups(roles) for b in groups: - self.backends[b].add_to_groups(username, groups[b]) + self.backends[b].add_to_groups(username, Set(groups[b])) + + cherrypy.log.error( + msg = "User '" + username + "' made member of " + str(roles)+ " by '" + admin + "'", + severity = logging.INFO + ) + + cherrypy.log.error( + msg = "User '" + username + "' groups: " + str(groups), + severity = logging.DEBUG + ) def _modify(self, params): pass