Mailing-List: contact log4j-user-help@logging.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Log4J Users List" <log4j-user@logging.apache.org>
MIME-Version: 1.0
In-Reply-To: 
 <CACZkXPxOggp0R8Oty8fBSZ8NDaHsPkx0o+_G1g9X0fcCMOTxNw@mail.gmail.com>
References: <62FDFA39-3596-45C0-B1CC-D7677BCE5349@chewy.com>
	<CACZkXPxf0WkyTHfXGrOxS4htD43tj7KJ78Kr1+K5pTLDXHY=OQ@mail.gmail.com>
	<CACZkXPxOggp0R8Oty8fBSZ8NDaHsPkx0o+_G1g9X0fcCMOTxNw@mail.gmail.com>
Date: Wed, 20 Jan 2016 18:21:34 -0800
Message-ID: 
 <CACZkXPzcCFuLxNv+ng+ehfSfN=qVJN40ovJf2uQkoxkgkCDBig@mail.gmail.com>
Subject: Re: TlsSyslogFrame calculates message length incorrectly
From: Gary Gregory <garydgregory@gmail.com>
To: Log4J Users List <log4j-user@logging.apache.org>
Content-Type: multipart/alternative; boundary=001a11c32c62f386ee0529cec2c8

--001a11c32c62f386ee0529cec2c8
Content-Type: text/plain; charset=UTF-8

On Wed, Jan 20, 2016 at 6:19 PM, Gary Gregory <garydgregory@gmail.com>
wrote:

> I'm also wondering if the Layout should really be a String layout.
>
> This is smelly in TlsSyslogFrame:
>
>     @Override
>     public String toString() {
>         final String length = Integer.toString(messageLengthInBytes);
>         return length + Chars.SPACE + message;
>     }
>
> combined with this in the Rfc5424 layout toSerializable(logEvent):
>
>         if (useTlsMessageFormat) {
>             return new TlsSyslogFrame(buf.toString()).toString();
>         }
>
> Which means that the size header is of variable length. Can that be right?
>

I guess that is OK since I now see the following in the spec:

MSG-LEN = NONZERO-DIGIT *DIGIT

G


>
> Gary
>
> On Wed, Jan 20, 2016 at 6:04 PM, Gary Gregory <garydgregory@gmail.com>
> wrote:
>
>> Hm, shouldn't we be using UTF-8 instead of Charset.defaultCharset() in:
>>
>>     public byte[] getBytes() {
>>         final String frame = toString();
>>         return frame.getBytes(Charset.defaultCharset());
>>     }
>>
>> ?
>>
>> Gary
>>
>> On Wed, Jan 20, 2016 at 4:58 PM, Blake Day <blake@chewy.com> wrote:
>>
>>> TlsSyslogFrame appears to be miscalculating the message length.
>>> According to RFC5425, the message length must be the octet count of the
>>> SYSLOG-MSG in the frame.  Though the variable below is aptly named
>>> messageLengthInBytes, it is assigned the value from message.length() (where
>>> message is a String) rather than the bytes.
>>>
>>> See below for relevant portions of code:
>>>
>>> private String message;
>>> private int messageLengthInBytes;
>>>
>>> private void setLengthInBytes() {
>>>     messageLengthInBytes = message.length();
>>> }
>>>
>>> @Override
>>> public String toString() {
>>>     final String length = Integer.toString(messageLengthInBytes);
>>>     return length + Chars.SPACE + message;
>>> }
>>>
>>>
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: log4j-user-unsubscribe@logging.apache.org
>>> For additional commands, e-mail: log4j-user-help@logging.apache.org
>>>
>>>
>>
>>
>> --
>> E-Mail: garydgregory@gmail.com | ggregory@apache.org
>> Java Persistence with Hibernate, Second Edition
>> <http://www.manning.com/bauer3/>
>> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
>> Spring Batch in Action <http://www.manning.com/templier/>
>> Blog: http://garygregory.wordpress.com
>> Home: http://garygregory.com/
>> Tweet! http://twitter.com/GaryGregory
>>
>
>
>
> --
> E-Mail: garydgregory@gmail.com | ggregory@apache.org
> Java Persistence with Hibernate, Second Edition
> <http://www.manning.com/bauer3/>
> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
> Spring Batch in Action <http://www.manning.com/templier/>
> Blog: http://garygregory.wordpress.com
> Home: http://garygregory.com/
> Tweet! http://twitter.com/GaryGregory
>


-- 
E-Mail: garydgregory@gmail.com | ggregory@apache.org
Java Persistence with Hibernate, Second Edition
<http://www.manning.com/bauer3/>
JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
Spring Batch in Action <http://www.manning.com/templier/>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

--001a11c32c62f386ee0529cec2c8--