Apache
/
activemq-artemis
mirror of https://github.com/apache/activemq-artemis.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ARTEMIS-3902 Adding reason to Audit Security Message

This commit is contained in:
Clebert Suconic 2022-07-22 11:22:30 -04:00
parent 3d5f588bbe
commit db8f530256
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
artemis-commons/src/main/java/org/apache/activemq/artemis/logs/AuditLogger.java
View File

@ -2267,13 +2267,13 @@ public interface AuditLogger extends BasicLogger {
void handleManagementMessage2(String user, Object source, Object... args);
static void securityFailure(Exception cause) {
BASE_LOGGER.securityFailure(getCaller(), cause);
static void securityFailure(Exception cause, String reason) {
BASE_LOGGER.securityFailure(getCaller(), reason, cause);
}
@LogMessage(level = Logger.Level.INFO)
@Message(id = 601264, value = "User {0} gets security check failure", format = Message.Format.MESSAGE_FORMAT)
void securityFailure(String user, @Cause Throwable cause);
@Message(id = 601264, value = "User {0} gets security check failure, reason={1}", format = Message.Format.MESSAGE_FORMAT)
void securityFailure(String user, String reason, @Cause Throwable cause);
static void createCoreConsumer(Object source, Subject user, String remoteAddress, Object... args) {

2
artemis-server/src/main/java/org/apache/activemq/artemis/core/security/impl/SecurityStoreImpl.java
View File

@ -290,7 +290,7 @@ public class SecurityStoreImpl implements SecurityStore, HierarchicalRepositoryC
} else {
ex = ActiveMQMessageBundle.BUNDLE.userNoPermissionsQueue(session.getUsername(), checkType, bareQueue, bareAddress);
}
AuditLogger.securityFailure(ex);
AuditLogger.securityFailure(ex, ex.getMessage());
throw ex;
}