Discussion:
[jira] [Created] (LOG4J2-428) Integration of GELF Appender
Mark Paluch (JIRA)
2013-10-16 05:33:41 UTC
Permalink
Mark Paluch created LOG4J2-428:
----------------------------------

Summary: Integration of GELF Appender
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Reporter: Mark Paluch


I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.

See https://github.com/mp911de/logstash-gelf for the code.



--
This message was sent by Atlassian JIRA
(v6.1#6144)
Gary Gregory (JIRA)
2013-10-16 12:45:57 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13796733#comment-13796733 ]

Gary Gregory commented on LOG4J2-428:
-------------------------------------

Hi Mark,

Any contribution should be repackaged to log4j 2 and added as a diff file to this issue.

Thank you.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Reporter: Mark Paluch
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
Mark Paluch (JIRA)
2013-10-16 19:52:42 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Paluch updated LOG4J2-428:
-------------------------------

Attachment: log4j-gelf.zip
log4j-gelf.patch

Attached log4j-gelf project as diff and as zipped project. Applies to log4j 2.x
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
Mark Paluch (JIRA)
2013-10-16 19:54:41 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Paluch updated LOG4J2-428:
-------------------------------

Affects Version/s: 2.0
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
Mark Paluch (JIRA)
2013-11-03 11:10:18 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13812330#comment-13812330 ]

Mark Paluch commented on LOG4J2-428:
------------------------------------

Any progress on this? Would like to integrate the code.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
Remko Popma (JIRA)
2013-11-03 13:38:18 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13812366#comment-13812366 ]

Remko Popma commented on LOG4J2-428:
------------------------------------

Hi Mark,
I've taken a (brief) look at the patch you submitted.

Two things:
* Can you remove the @author javadoc tags? The ASF officially discourages author tags (https://issues.apache.org/jira/browse/JSPWIKI-589). You can get credit for your contribution by being listed as a Contributor on the Project Team page (http://logging.apache.org/log4j/2.x/team-list.html).
* I saw some System.out.println output in some error handling code. Would you mind using the StatusLogger instead?
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
Mark Paluch (JIRA)
2013-11-10 09:11:19 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Paluch updated LOG4J2-428:
-------------------------------

Attachment: (was: log4j-gelf.patch)
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
Mark Paluch (JIRA)
2013-11-10 09:13:19 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Paluch updated LOG4J2-428:
-------------------------------

Attachment: (was: log4j-gelf.zip)
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
Mark Paluch (JIRA)
2013-11-10 09:13:20 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mark Paluch updated LOG4J2-428:
-------------------------------

Attachment: log4j-gelf.zip
log4j-gelf.patch
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
Mark Paluch (JIRA)
2013-11-22 18:34:35 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13830206#comment-13830206 ]

Mark Paluch commented on LOG4J2-428:
------------------------------------

Any updates on that?
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
Ralph Goers (JIRA)
2014-03-09 22:25:43 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13925345#comment-13925345 ]

Ralph Goers commented on LOG4J2-428:
------------------------------------

I've looked at the code but after looking at http://graylog2.org/gelf I am wondering why we need a whole separate subproject and appender instead of just another Layout and/or Message type?
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Ralph Goers (JIRA)
2014-03-09 22:25:43 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13925345#comment-13925345 ]

Ralph Goers edited comment on LOG4J2-428 at 3/9/14 10:25 PM:
-------------------------------------------------------------

I've looked at the code but after looking at http://graylog2.org/gelf I am wondering why we need a whole separate subproject and appender instead of just another Layout and possibly a new Message type?


was (Author: ***@dslextreme.com):
I've looked at the code but after looking at http://graylog2.org/gelf I am wondering why we need a whole separate subproject and appender instead of just another Layout and/or Message type?
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Mark Paluch (JIRA)
2014-03-10 10:04:43 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13925616#comment-13925616 ]

Mark Paluch commented on LOG4J2-428:
------------------------------------

GELF isn't just a simple JSON data structure. GELF supports compression and needs message chunking. GELF uses a set of own field names which are required by the protocol itself and prefixes custom fiedls. This would result in an customized JSONLayout. The appender it self needs to split large messages (> 8192 bytes) into multiple GELF message chunks (with message number, not only a simple partitioning of the Layout byte array). Therefore I see it necessary to have a custom appender. I guess, that there is perhaps any way to customize log4j/log4j2 with lots of settings to make GELF work (in some way) but it's way to complex.
It's up to you whether you integrate the code as component package or a sub project. It was easier to me to submit the code as subproject.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Ralph Goers (JIRA)
2014-03-10 15:29:46 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13925791#comment-13925791 ]

Ralph Goers commented on LOG4J2-428:
------------------------------------

Layouts are meant to handle the formatting of messages while Appenders are the delivery channel. The problem I have here is that you have intertwined these concepts.

The splitting of large messages could be an issue, but I think we could address that in a general way. I really don't like that the GELF appender has its own flavor of the TCP and UDP appenders. Furthermore, I see no reason why GELF shouldn't be allowed with other appenders, such as a FileAppender, FlumeAppender, etc. So making it a Layout is the preferred way to go in my opinion.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Mark Paluch (JIRA)
2014-03-10 16:08:44 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13925835#comment-13925835 ]

Mark Paluch commented on LOG4J2-428:
------------------------------------

Ok, thanks for the hint. I'll give it a try adding a GELFLayout.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Matt Sicker (JIRA)
2014-05-04 23:25:17 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matt Sicker updated LOG4J2-428:
-------------------------------

Fix Version/s: (was: 2.1)
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Matt Sicker (JIRA)
2014-05-04 23:25:18 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matt Sicker updated LOG4J2-428:
-------------------------------

Affects Version/s: (was: 2.0)
2.1
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Matt Sicker (JIRA)
2014-05-04 23:23:14 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matt Sicker updated LOG4J2-428:
-------------------------------

Fix Version/s: 2.1
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.0
Reporter: Mark Paluch
Fix For: 2.1
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Mikael Ståldal (JIRA)
2014-08-05 15:26:15 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14086372#comment-14086372 ]

Mikael Ståldal commented on LOG4J2-428:
---------------------------------------

I have made an GELFLayout, that I am willing to contribute to Log4j.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: GELFLayout.java, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Mikael Ståldal (JIRA)
2014-08-05 15:26:17 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------

Attachment: GELFLayout.java
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: GELFLayout.java, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Mikael Ståldal (JIRA)
2014-08-05 16:14:12 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------

Attachment: (was: GELFLayout.java)
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Mikael Ståldal (JIRA)
2014-08-05 16:16:12 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------

Attachment: GelfLayout.java

New version, with proper naming, and GZIP compression.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: GelfLayout.java, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Mikael Ståldal (JIRA)
2014-08-06 09:09:13 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------

Attachment: (was: GelfLayout.java)
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Mikael Ståldal (JIRA)
2014-08-06 09:11:12 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------

Attachment: GelfLayout.java

Only compress if log event data is larger than 1 KiB.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: GelfLayout.java, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
Gary Gregory (JIRA)
2014-09-11 11:17:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14129893#comment-14129893 ]

Gary Gregory commented on LOG4J2-428:
-------------------------------------

Let's see where we are here...

[~***@home.se]:
- I'm sorry to ask again, but can you provide a patch in the unified diff format. I cannot use Eclipse to apply the current patch because it tells me the format is invalid. That would make life easier.
- What is your relationship with the company that backs GELF, if any?
- Should only GELF 1.1 be supported? What about 1.0?

All:

The GELF specification is here:
http://graylog2.org/resources/gelf/specification
The page at the bottom has "© 2014 TORCH GmbH. All rights reserverd." [sic]
Is this in conflict with the Apache License? IOW, can we create code that writes out messages in a format that is not open-source?
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: GelfLayout.java, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mark Paluch (JIRA)
2014-09-11 11:24:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14129899#comment-14129899 ]

Mark Paluch commented on LOG4J2-428:
------------------------------------

[~garydgregory] Logstash uses ASLv2 (https://github.com/elasticsearch/logstash) and supports GELF. The original Graylog-Server is licensed under GPL 3 (https://github.com/Graylog2/graylog2-server)
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: GelfLayout.java, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-11 11:57:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------
Attachment: (was: GelfLayout.java)
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-11 11:58:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------
Attachment: gelflayout.patch

GelfLayout.java
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-11 12:00:35 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14129935#comment-14129935 ]

Gary Gregory commented on LOG4J2-428:
-------------------------------------

Do you have a separate patch for unit tests? You can put it all in one patch, it seems easier to me that way.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-11 12:01:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14129936#comment-14129936 ]

Mikael Ståldal commented on LOG4J2-428:
---------------------------------------

* OK, I have attached it as a patch now.

* I do not have any relationship with the company that backs GELF.

* This supports GELF 1.1. I don't know if anyone uses GELF 1.0.

BTW, it seems like I have duplicate user accounts here, and you referred to the wrong one. Is there a way to get rid of the old one?
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-11 12:18:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14129945#comment-14129945 ]

Gary Gregory commented on LOG4J2-428:
-------------------------------------

You might want to create an INFRA Jira issue for removing your old account.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-11 12:19:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14129946#comment-14129946 ]

Gary Gregory commented on LOG4J2-428:
-------------------------------------

Is the spec on the page http://graylog2.org/resources/gelf/specification also in https://github.com/elasticsearch/logstash?

What about unit tests?
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-11 13:02:35 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------
Attachment: (was: gelflayout.patch)
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-11 13:02:35 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------
Attachment: gelflayout.patch

Now with unit test.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-11 13:04:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14129987#comment-14129987 ]

Mikael Ståldal commented on LOG4J2-428:
---------------------------------------

I don't know about the specs.
Post by Mark Paluch (JIRA)
Integration of GELF Appender
----------------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-11 16:43:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gary Gregory updated LOG4J2-428:
--------------------------------
Summary: Implement a GELF layout (was: Integration of GELF Appender)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-11 16:49:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14130273#comment-14130273 ]

Gary Gregory commented on LOG4J2-428:
-------------------------------------

Can you please pull from master and add a test for compression? I have your patch in master with some extra changes on top.
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-11 17:23:35 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14130332#comment-14130332 ]

Gary Gregory commented on LOG4J2-428:
-------------------------------------

Pushed support for ZLIB too.
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-12 13:40:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------
Attachment: (was: gelflayout.patch)
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-12 13:41:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------
Attachment: gelflayout.patch

Unit test for compression and stack traces.
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-12 16:50:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14131738#comment-14131738 ]

Gary Gregory commented on LOG4J2-428:
-------------------------------------

Some notes:

- Do not use @link at the bottom of methods, it's not what this tag is for. Check out the latest from master, I'll fixed up the {{@see}} to be nicer.

The patch fails for me with:

{noformat}
java.lang.IllegalArgumentException: Can not parse expected value.
at net.javacrumbs.jsonunit.core.internal.JsonUtils.readValue(JsonUtils.java:61)
at net.javacrumbs.jsonunit.core.internal.JsonUtils.readValue(JsonUtils.java:47)
at net.javacrumbs.jsonunit.core.internal.JsonUtils.convertToJson(JsonUtils.java:78)
at net.javacrumbs.jsonunit.core.internal.Diff.create(Diff.java:73)
at net.javacrumbs.jsonunit.JsonAssert.assertJsonPartEquals(JsonAssert.java:68)
at net.javacrumbs.jsonunit.JsonAssert.assertJsonEquals(JsonAssert.java:57)
at org.apache.logging.log4j.core.layout.GelfLayoutTest.testLayout(GelfLayoutTest.java:148)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:50)
at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:459)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:675)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:382)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:192)
Caused by: com.fasterxml.jackson.core.JsonParseException: Illegal unquoted character ((CTRL-CHAR, code 13)): has to be escaped using backslash to be included in string value
at [Source: ***@7e4cb74; line: 1, column: 197]
at com.fasterxml.jackson.core.JsonParser._constructError(JsonParser.java:1419)
at com.fasterxml.jackson.core.base.ParserMinimalBase._reportError(ParserMinimalBase.java:508)
at com.fasterxml.jackson.core.base.ParserMinimalBase._throwUnquotedSpace(ParserMinimalBase.java:472)
at com.fasterxml.jackson.core.json.ReaderBasedJsonParser._finishString2(ReaderBasedJsonParser.java:1613)
at com.fasterxml.jackson.core.json.ReaderBasedJsonParser._finishString(ReaderBasedJsonParser.java:1585)
at com.fasterxml.jackson.core.json.ReaderBasedJsonParser.getText(ReaderBasedJsonParser.java:233)
at com.fasterxml.jackson.databind.deser.std.BaseNodeDeserializer.deserializeObject(JsonNodeDeserializer.java:224)
at com.fasterxml.jackson.databind.deser.std.JsonNodeDeserializer.deserialize(JsonNodeDeserializer.java:62)
at com.fasterxml.jackson.databind.deser.std.JsonNodeDeserializer.deserialize(JsonNodeDeserializer.java:14)
at com.fasterxml.jackson.databind.ObjectMapper._readMapAndClose(ObjectMapper.java:3051)
at com.fasterxml.jackson.databind.ObjectMapper.readTree(ObjectMapper.java:1804)
at net.javacrumbs.jsonunit.core.internal.JsonUtils.readValue(JsonUtils.java:59)
... 31 more
{noformat}
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-15 08:08:35 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------
Attachment: (was: gelflayout.patch)
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-15 08:10:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Ståldal updated LOG4J2-428:
----------------------------------
Attachment: gelflayout.patch

The test worked for me on Linux, but I guess you run it in Windows. Try this on Windows. (It's a line ending issues, \n vs \r\n).
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-15 13:44:33 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gary Gregory resolved LOG4J2-428.
---------------------------------
Resolution: Fixed
Fix Version/s: 2.1

Setting to resolve as the new tests are done: ZLIB, GZIP, and no compression.

Please verify and close.
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Fix For: 2.1
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-15 13:47:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14133894#comment-14133894 ]

Gary Gregory edited comment on LOG4J2-428 at 9/15/14 1:47 PM:
--------------------------------------------------------------

Setting to resolve as the new tests are done: ZLIB, GZIP, and no compression.

Please verify and close.

Feel free to submit more tests and/or code!

Thank you!



was (Author: garydgregory):
Setting to resolve as the new tests are done: ZLIB, GZIP, and no compression.

Please verify and close.
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Fix For: 2.1
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Mikael Ståldal (JIRA)
2014-09-15 16:00:36 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14134046#comment-14134046 ]

Mikael Ståldal commented on LOG4J2-428:
---------------------------------------

Looks fine to me.

However, I don't like the fact that it now depends on Jackson.
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Fix For: 2.1
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Gary Gregory (JIRA)
2014-09-15 16:36:34 UTC
Permalink
[ https://issues.apache.org/jira/browse/LOG4J2-428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14134094#comment-14134094 ]

Gary Gregory commented on LOG4J2-428:
-------------------------------------

Core already depends on Jackson. Also the previous implementation seemed
inefficient and was undocumented as to which cases might or might not
handled. The Jackson implementation is optimized, or at least is documented
as optimized.

Gary

On Mon, Sep 15, 2014 at 12:00 PM, Mikael Ståldal (JIRA) <***@apache.org>
--
E-Mail: ***@gmail.com | ***@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
Post by Gary Gregory (JIRA)
Implement a GELF layout
-----------------------
Key: LOG4J2-428
URL: https://issues.apache.org/jira/browse/LOG4J2-428
Project: Log4j 2
Issue Type: Wish
Components: Appenders
Affects Versions: 2.1
Reporter: Mark Paluch
Fix For: 2.1
Attachments: gelflayout.patch, log4j-gelf.patch, log4j-gelf.zip
I'd like to propose the integration of a GELF (Graylog Extended Log Format) Appender into log4j2 on a basis like the flume-ng Appender.
See https://github.com/mp911de/logstash-gelf for the code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
Loading...