Good point. Let me see what I can do. On 2 March 2016 at 21:10, Gary Gregory <[email protected]> wrote:
> No tests? ;-) > > Gary > > ---------- Forwarded message ---------- > From: <[email protected]> > Date: Wed, Mar 2, 2016 at 6:41 PM > Subject: logging-log4j2 git commit: LOG4J2-1252 - JeroMqAppender should > support layouts > To: [email protected] > > > Repository: logging-log4j2 > Updated Branches: > refs/heads/master ebc53bbdc -> 60d1ccd93 > > > LOG4J2-1252 - JeroMqAppender should support layouts > > > Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo > Commit: > http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/60d1ccd9 > Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/60d1ccd9 > Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/60d1ccd9 > > Branch: refs/heads/master > Commit: 60d1ccd9349e4601464d53926595146a59ac4beb > Parents: ebc53bb > Author: Matt Sicker <[email protected]> > Authored: Wed Mar 2 20:42:04 2016 -0600 > Committer: Matt Sicker <[email protected]> > Committed: Wed Mar 2 20:42:04 2016 -0600 > > ---------------------------------------------------------------------- > .../logging/log4j/core/appender/mom/jeromq/JeroMqAppender.java | 5 +++-- > src/changes/changes.xml | 3 +++ > 2 files changed, 6 insertions(+), 2 deletions(-) > ---------------------------------------------------------------------- > > > > http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/60d1ccd9/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/jeromq/JeroMqAppender.java > ---------------------------------------------------------------------- > diff --git > a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/jeromq/JeroMqAppender.java > b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/jeromq/JeroMqAppender.java > index 974477b..99968e4 100644 > --- > a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/jeromq/JeroMqAppender.java > +++ > b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/jeromq/JeroMqAppender.java > @@ -244,8 +244,9 @@ public final class JeroMqAppender extends > AbstractAppender { > > @Override > public synchronized void append(final LogEvent event) { > - final String formattedMessage = > event.getMessage().getFormattedMessage(); > - if (getPublisher().send(formattedMessage, 0)) { > + final Layout<? extends Serializable> layout = getLayout(); > + final byte[] formattedMessage = layout.toByteArray(event); > + if (getPublisher().send(getLayout().toByteArray(event))) { > sendRcTrue++; > } else { > sendRcFalse++; > > > http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/60d1ccd9/src/changes/changes.xml > ---------------------------------------------------------------------- > diff --git a/src/changes/changes.xml b/src/changes/changes.xml > index 30f7a67..646a41f 100644 > --- a/src/changes/changes.xml > +++ b/src/changes/changes.xml > @@ -181,6 +181,9 @@ > <action issue="LOG4J2-1262" dev="mattsicker" type="fix"> > Stop throwing unnecessary exception in > Log4jServletContextListener.contextDestroyed(). > </action> > + <action issue="LOG4J2-1252" dev="mattsicker" type="add"> > + JeroMqAppender should support layouts. > + </action> > </release> > <release version="2.5" date="2015-12-06" description="GA Release 2.5"> > <action issue="LOG4J2-324" dev="rpopma" type="fix"> > > > > > -- > E-Mail: [email protected] | [email protected] > 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 > -- Matt Sicker <[email protected]>
