You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by "Ralph Goers (JIRA)" <ji...@apache.org> on 2016/07/02 20:39:11 UTC
[jira] [Resolved] (LOG4J2-1332) Null/empty messages are not handled
properly in RFC5424Layout
[ https://issues.apache.org/jira/browse/LOG4J2-1332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ralph Goers resolved LOG4J2-1332.
---------------------------------
Resolution: Not A Problem
Actually, I misread the document the overall grammar of a syslog message is
SYSLOG-MSG = HEADER SP STRUCTURED-DATA [SP MSG]
so the message is optional and the current code is correct. Also, example 4 in section 6 shows a syslog message with only structured data and no message text.
The only thing we are doing incorrectly is not checking for a null Message in the LogEvent, but that really should have been checked or caused other problems long before here.
If you agree with this please close. Otherwise, please provide a reference to where this disagrees with the spec.
> Null/empty messages are not handled properly in RFC5424Layout
> -------------------------------------------------------------
>
> Key: LOG4J2-1332
> URL: https://issues.apache.org/jira/browse/LOG4J2-1332
> Project: Log4j 2
> Issue Type: Bug
> Components: Layouts
> Reporter: Michael Day
> Assignee: Ralph Goers
>
> In the following code snippet, you can see that null/empty messages are not appended to the buffer. This results in a violation of RFC5424.
> {code}
> private void appendMessage(StringBuilder buffer, LogEvent event) {
> Message message = event.getMessage();
> String text = message instanceof StructuredDataMessage?message.getFormat():message.getFormattedMessage();
> if(text != null && text.length() > 0) {
> buffer.append(' ').append(this.escapeNewlines(text, this.escapeNewLine));
> }
> ...
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org