Merge pull request Ericsson#1402 from csordasmarton/change-user-permi…

…ssion-error Change existing/removed permission log message
yuleyl · Feb 27, 2018 · cb3e52a · cb3e52a
2 parents b91a1ab + f232fc1
commit cb3e52a
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/libcodechecker/server/permissions.py b/libcodechecker/server/permissions.py
@@ -166,10 +166,10 @@ def add_permission(self, auth_name, is_group=False):
                         (is_group and len(users) == 1 and len(groups) == 1):
                     self._rem_perm_impl("*", False)
         else:
-            LOG.error("Permission '{0}' adding for {1} '{2}' failed!"
-                      .format(self._perm_name,
-                              'group' if is_group else 'user',
-                              auth_name))
+            LOG.info("Permission '{0}' already added for {1} '{2}'!".format(
+                self._perm_name,
+                'group' if is_group else 'user',
+                auth_name))
 
     def remove_permission(self, auth_name, is_group=False):
         """
@@ -193,10 +193,10 @@ def remove_permission(self, auth_name, is_group=False):
                 if len(users) == 0 and len(groups) == 0:
                     self._add_perm_impl("*", False)
         else:
-            LOG.error("Permission '{0}' removal from {1} '{2}' failed!"
-                      .format(self._perm_name,
-                              'group' if is_group else 'user',
-                              auth_name))
+            LOG.info("Permission '{0}' already removed from {1} '{2}'!".format(
+                self._perm_name,
+                'group' if is_group else 'user',
+                auth_name))
 
     def has_permission(self, auth_session):
         """