You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by "Andrei Ivanov (JIRA)" <ji...@apache.org> on 2019/03/18 09:50:00 UTC
[jira] [Commented] (LOG4J2-2551) Event starting with upper case not
found by AbstractEventLogger.logEvent
[ https://issues.apache.org/jira/browse/LOG4J2-2551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16794879#comment-16794879 ]
Andrei Ivanov commented on LOG4J2-2551:
---------------------------------------
[~ralphgoers] thoughts?
> Event starting with upper case not found by AbstractEventLogger.logEvent
> ------------------------------------------------------------------------
>
> Key: LOG4J2-2551
> URL: https://issues.apache.org/jira/browse/LOG4J2-2551
> Project: Log4j 2
> Issue Type: Bug
> Components: Log4j-Audit
> Affects Versions: Log4j-Audit 1.0.1
> Reporter: Andrei Ivanov
> Priority: Major
>
> As a consequence of my changes for LOG4J2-2442, I've introduced a regression when the name of the event starts with an upper case letter 😫
> As a fix, I'm not sure about the approach:
> 1. Move the normalization of the name when constructing the {{AuditMessage}}, from line [94|https://github.com/apache/logging-log4j-audit/blob/master/log4j-audit/log4j-audit-api/src/main/java/org/apache/logging/log4j/audit/AbstractEventLogger.java#L94] to [106|https://github.com/apache/logging-log4j-audit/blob/master/log4j-audit/log4j-audit-api/src/main/java/org/apache/logging/log4j/audit/AbstractEventLogger.java#L106]
> 2. Remove name normalization completely and use the exact name used in the event definition.
> I actually think I like option 2 better.
> What do you think?
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)