[Minor] Log the different actions in DefaultPrivilegeHandler
This commit is contained in:
parent
fa7309cb19
commit
37562fdc27
|
@ -456,6 +456,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate to persistence handler
|
// delegate to persistence handler
|
||||||
this.persistenceHandler.addUser(newUser);
|
this.persistenceHandler.addUser(newUser);
|
||||||
|
|
||||||
|
logger.info("Created new user " + newUser.getUsername());
|
||||||
|
|
||||||
return newUser.asUserRep();
|
return newUser.asUserRep();
|
||||||
|
|
||||||
} finally {
|
} finally {
|
||||||
|
@ -511,6 +513,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate to persistence handler
|
// delegate to persistence handler
|
||||||
this.persistenceHandler.replaceUser(newUser);
|
this.persistenceHandler.replaceUser(newUser);
|
||||||
|
|
||||||
|
logger.info("Replaced user " + newUser.getUsername());
|
||||||
|
|
||||||
return newUser.asUserRep();
|
return newUser.asUserRep();
|
||||||
|
|
||||||
} finally {
|
} finally {
|
||||||
|
@ -590,6 +594,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate to persistence handler
|
// delegate to persistence handler
|
||||||
this.persistenceHandler.replaceUser(newUser);
|
this.persistenceHandler.replaceUser(newUser);
|
||||||
|
|
||||||
|
logger.info("Updated user " + newUser.getUsername());
|
||||||
|
|
||||||
return newUser.asUserRep();
|
return newUser.asUserRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -613,6 +619,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate user removal to persistence handler
|
// delegate user removal to persistence handler
|
||||||
this.persistenceHandler.removeUser(username);
|
this.persistenceHandler.removeUser(username);
|
||||||
|
|
||||||
|
logger.info("Removed user " + username);
|
||||||
|
|
||||||
return existingUser.asUserRep();
|
return existingUser.asUserRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -659,6 +667,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate user replacement to persistence handler
|
// delegate user replacement to persistence handler
|
||||||
this.persistenceHandler.replaceUser(newUser);
|
this.persistenceHandler.replaceUser(newUser);
|
||||||
|
|
||||||
|
logger.info("Added role " + roleName + " to " + newUser.getUsername());
|
||||||
|
|
||||||
return newUser.asUserRep();
|
return newUser.asUserRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -696,6 +706,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate user replacement to persistence handler
|
// delegate user replacement to persistence handler
|
||||||
this.persistenceHandler.replaceUser(newUser);
|
this.persistenceHandler.replaceUser(newUser);
|
||||||
|
|
||||||
|
logger.info("Removed role " + roleName + " from " + newUser.getUsername());
|
||||||
|
|
||||||
return newUser.asUserRep();
|
return newUser.asUserRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -730,6 +742,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
this.persistenceHandler.persist();
|
this.persistenceHandler.persist();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
logger.info("Set locale to " + locale + " for " + newUser.getUsername());
|
||||||
|
|
||||||
return newUser.asUserRep();
|
return newUser.asUserRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -780,6 +794,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
invalidateSession(certificate);
|
invalidateSession(certificate);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
logger.info("Updated password for " + newUser.getUsername());
|
||||||
|
|
||||||
} finally {
|
} finally {
|
||||||
clearPassword(password);
|
clearPassword(password);
|
||||||
}
|
}
|
||||||
|
@ -809,6 +825,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate user replacement to persistence handler
|
// delegate user replacement to persistence handler
|
||||||
this.persistenceHandler.replaceUser(newUser);
|
this.persistenceHandler.replaceUser(newUser);
|
||||||
|
|
||||||
|
logger.info("Set state of user " + newUser.getUsername() + " to " + state);
|
||||||
|
|
||||||
return newUser.asUserRep();
|
return newUser.asUserRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -840,6 +858,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate to persistence handler
|
// delegate to persistence handler
|
||||||
this.persistenceHandler.addRole(newRole);
|
this.persistenceHandler.addRole(newRole);
|
||||||
|
|
||||||
|
logger.info("Added new role " + newRole.getName());
|
||||||
|
|
||||||
return newRole.asRoleRep();
|
return newRole.asRoleRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -875,6 +895,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate to persistence handler
|
// delegate to persistence handler
|
||||||
this.persistenceHandler.replaceRole(newRole);
|
this.persistenceHandler.replaceRole(newRole);
|
||||||
|
|
||||||
|
logger.info("Replaced role " + newRole.getName());
|
||||||
|
|
||||||
return newRole.asRoleRep();
|
return newRole.asRoleRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -909,6 +931,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate role removal to persistence handler
|
// delegate role removal to persistence handler
|
||||||
this.persistenceHandler.removeRole(roleName);
|
this.persistenceHandler.removeRole(roleName);
|
||||||
|
|
||||||
|
logger.info("Removed role " + roleName);
|
||||||
|
|
||||||
return existingRole.asRoleRep();
|
return existingRole.asRoleRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -963,6 +987,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate role replacement to persistence handler
|
// delegate role replacement to persistence handler
|
||||||
this.persistenceHandler.replaceRole(newRole);
|
this.persistenceHandler.replaceRole(newRole);
|
||||||
|
|
||||||
|
logger.info("Added/replaced privilege " + privilegeRep.getName() + " to " + roleName);
|
||||||
|
|
||||||
return newRole.asRoleRep();
|
return newRole.asRoleRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1003,6 +1029,8 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// delegate user replacement to persistence handler
|
// delegate user replacement to persistence handler
|
||||||
this.persistenceHandler.replaceRole(newRole);
|
this.persistenceHandler.replaceRole(newRole);
|
||||||
|
|
||||||
|
logger.info("Removed privilege " + privilegeName + " from " + roleName);
|
||||||
|
|
||||||
return newRole.asRoleRep();
|
return newRole.asRoleRep();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1085,8 +1113,7 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
persistSessions();
|
persistSessions();
|
||||||
|
|
||||||
// log
|
// log
|
||||||
DefaultPrivilegeHandler.logger
|
logger.info(MessageFormat.format("User {0} authenticated: {1}", username, certificate)); //$NON-NLS-1$
|
||||||
.info(MessageFormat.format("User {0} authenticated: {1}", username, certificate)); //$NON-NLS-1$
|
|
||||||
|
|
||||||
// return the certificate
|
// return the certificate
|
||||||
return certificate;
|
return certificate;
|
||||||
|
@ -1094,7 +1121,7 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
} catch (RuntimeException e) {
|
} catch (RuntimeException e) {
|
||||||
String msg = "User {0} Failed to authenticate: {1}"; //$NON-NLS-1$
|
String msg = "User {0} Failed to authenticate: {1}"; //$NON-NLS-1$
|
||||||
msg = MessageFormat.format(msg, username, e.getMessage());
|
msg = MessageFormat.format(msg, username, e.getMessage());
|
||||||
DefaultPrivilegeHandler.logger.error(msg);
|
logger.error(msg);
|
||||||
throw e;
|
throw e;
|
||||||
} finally {
|
} finally {
|
||||||
clearPassword(password);
|
clearPassword(password);
|
||||||
|
@ -1807,7 +1834,7 @@ public class DefaultPrivilegeHandler implements PrivilegeHandler {
|
||||||
// log
|
// log
|
||||||
String msg = "The system user ''{0}'' is logged in with session {1}"; //$NON-NLS-1$
|
String msg = "The system user ''{0}'' is logged in with session {1}"; //$NON-NLS-1$
|
||||||
msg = MessageFormat.format(msg, systemUsername, systemUserCertificate.getSessionId());
|
msg = MessageFormat.format(msg, systemUsername, systemUserCertificate.getSessionId());
|
||||||
DefaultPrivilegeHandler.logger.info(msg);
|
logger.info(msg);
|
||||||
|
|
||||||
return privilegeContext;
|
return privilegeContext;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue