[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742017#comment-13742017
]
Keir commented on LOG4J2-344:
-
Sorry, been away, will post up later today
Maybe we just need another plugin for the 2nd name then. Subclass or
delegate?
On Friday, August 16, 2013, Ralph Goers wrote:
I had the same thought. People switching from log4j 1 or logback will
probably make that mistake a lot. Plus this breaks virtually everyone
currently using Log4j 2.
On Fri, Aug 16, 2013 at 9:38 AM, Remko Popma remko.po...@gmail.com wrote:
Maybe we just need another plugin for the 2nd name then. Subclass or
delegate?
This is the kind of growing pains / evolution alpha or beta software goes
through IMO. We are not jumping through hoops for the change from
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742319#comment-13742319
]
Keir commented on LOG4J2-344:
-
16-Aug-2013 16:42:48 o'clock BST Error HTTP BEA-101020
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742320#comment-13742320
]
Keir commented on LOG4J2-344:
-
dependency
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742323#comment-13742323
]
Keir commented on LOG4J2-344:
-
?xml version=1.0 encoding=UTF-8?
configuration
Tibor Benke created LOG4J2-355:
--
Summary: Add support for multiple SD-ELEMENTs in a RFC 5424 syslog
message
Key: LOG4J2-355
URL: https://issues.apache.org/jira/browse/LOG4J2-355
Project: Log4j 2
[
https://issues.apache.org/jira/browse/LOG4J2-355?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Tibor Benke updated LOG4J2-355:
---
Attachment: add_multime_SD_ELEMENT_support.diff
Add support for multiple SD-ELEMENTs in a RFC
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742334#comment-13742334
]
Gary Gregory commented on LOG4J2-344:
-
Hm... it looks like the {{initializer}} is
[
https://issues.apache.org/jira/browse/LOG4J2-168?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742328#comment-13742328
]
Tibor Benke commented on LOG4J2-168:
I created a new issue:
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742370#comment-13742370
]
Ralph Goers commented on LOG4J2-344:
The javadoc for init says - The init method must
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742370#comment-13742370
]
Ralph Goers edited comment on LOG4J2-344 at 8/16/13 4:38 PM:
-
Hi I'm trying to update some documentation and figure out if an Log4j wiki link
is dead. The URL is:
http://wiki.apache.org/logging-log4j/UsefulCode
Is that going to be re-loaded again soon? Has it been migrated somewhere else?
Sorry if this is the wrong place to post this. Tried IRC and
I'm adding an aliases attribute to the Plugin annotation.
On Aug 16, 2013, at 6:38 AM, Remko Popma wrote:
Maybe we just need another plugin for the 2nd name then. Subclass or delegate?
On Friday, August 16, 2013, Ralph Goers wrote:
I had the same thought. People switching from log4j 1 or
In Layout I see (my bold):
/**
* Returns the format for the layout format.
* @return The footer.
* * @doubt the concept of header and footer is not universal, should
not be on the base interface.
* (RG) I agree with this.*
*/
byte[] getFooter();
/**
*
On Fri, Aug 16, 2013 at 1:13 PM, Ralph Goers ralph.go...@dslextreme.comwrote:
I'm adding an aliases attribute to the Plugin annotation.
Hold on to your horses ;)
Another way to look at this is that our config parsing that is already
case-insensitive could be augmented to strip out -s, no
I'm not sure if this ship has fully sailed, but I'd prefer to see us
stick with he dash format due to folks being familiar with it from
log4j 1.
Scott
On 8/16/13, Gary Gregory garydgreg...@gmail.com wrote:
On Fri, Aug 16, 2013 at 1:13 PM, Ralph Goers
ralph.go...@dslextreme.comwrote:
I'm
On Fri, Aug 16, 2013 at 2:28 PM, Scott Deboy scott.de...@gmail.com wrote:
I'm not sure if this ship has fully sailed, but I'd prefer to see us
stick with he dash format due to folks being familiar with it from
log4j 1.
That's a thin argument IMO considering you'll have to read the version 2
On Fri, Aug 16, 2013 at 2:38 PM, Gary Gregory garydgreg...@gmail.comwrote:
On Fri, Aug 16, 2013 at 2:28 PM, Scott Deboy scott.de...@gmail.comwrote:
I'm not sure if this ship has fully sailed, but I'd prefer to see us
stick with he dash format due to folks being familiar with it from
log4j 1.
I'm not in favor of just allowing arbitrary '-' characters wherever users want.
But allowing aliases makes it possible to allow for variations. I already have
this working.
Ralph
On Aug 16, 2013, at 11:39 AM, Gary Gregory wrote:
On Fri, Aug 16, 2013 at 2:38 PM, Gary Gregory
On Fri, Aug 16, 2013 at 3:32 PM, Ralph Goers ralph.go...@dslextreme.comwrote:
I'm not in favor of just allowing arbitrary '-' characters wherever users
want. But allowing aliases makes it possible to allow for variations. I
already have this working.
Well, once we opened the door with mixed
Gary Gregory created LOG4J2-356:
---
Summary: Create a JSON Layout
Key: LOG4J2-356
URL: https://issues.apache.org/jira/browse/LOG4J2-356
Project: Log4j 2
Issue Type: New Feature
[
https://issues.apache.org/jira/browse/LOG4J2-356?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Gary Gregory resolved LOG4J2-356.
-
Resolution: Fixed
Assignee: Gary Gregory
{noformat}
commit -m [LOG4J2-356] Create a JSON
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742334#comment-13742334
]
Gary Gregory edited comment on LOG4J2-344 at 8/16/13 8:56 PM:
--
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742601#comment-13742601
]
Gary Gregory commented on LOG4J2-344:
-
And yet there is the NPE :(
[
https://issues.apache.org/jira/browse/LOG4J2-353?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Gary Gregory resolved LOG4J2-353.
-
Resolution: Fixed
I think this is done now.
Use consistent conventions in
Nick?
Gary
On Wed, Aug 14, 2013 at 3:10 AM, YuCheng Ting yucht...@gmail.com wrote:
Hi all,
I use *beta8 *log4j2 and wrote log4j2.xml like example in document (
http://logging.apache.org/log4j/2.x/manual/appenders.html#NoSQLAppender ):
*appenders
NoSql name=databaseAppender
Ralph,
Don't forget the error-ref attribute for AsyncAppender.
Remko
On Saturday, August 17, 2013, Ralph Goers wrote:
I'm not in favor of just allowing arbitrary '-' characters wherever users
want. But allowing aliases makes it possible to allow for variations. I
already have this working.
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742618#comment-13742618
]
Nick Williams commented on LOG4J2-344:
--
The stack trace in question can _*ONLY*_
Seems reasonable.
Nick
On Aug 16, 2013, at 1:07 PM, Gary Gregory wrote:
In Layout I see (my bold):
/**
* Returns the format for the layout format.
* @return The footer.
* @doubt the concept of header and footer is not universal, should not
be on the base interface.
Answers inline.
On Aug 14, 2013, at 2:10 AM, YuCheng Ting wrote:
Hi all,
I use beta8 log4j2 and wrote log4j2.xml like example in document
(http://logging.apache.org/log4j/2.x/manual/appenders.html#NoSQLAppender ):
appenders
NoSql name=databaseAppender
MongoDb
Abhinav Shah created LOG4J2-357:
---
Summary: log4jConfiguration via log4j.xml on classpath
Key: LOG4J2-357
URL: https://issues.apache.org/jira/browse/LOG4J2-357
Project: Log4j 2
Issue Type: Bug
Nick Williams created LOG4J2-358:
Summary: NoSQLAppender using MongoDB provider ignores username and
password attributes
Key: LOG4J2-358
URL: https://issues.apache.org/jira/browse/LOG4J2-358
Project:
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742647#comment-13742647
]
Gary Gregory commented on LOG4J2-344:
-
Should we bullet-proof our filter such that it
Thank you for the update Nick!
:)
Gary
On Fri, Aug 16, 2013 at 5:39 PM, Nick Williams
nicho...@nicholaswilliams.net wrote:
Answers inline.
On Aug 14, 2013, at 2:10 AM, YuCheng Ting wrote:
Hi all,
I use *beta8 *log4j2 and wrote log4j2.xml like example in document (
I'm not sure. What would the code look like if we introduce this
subinterface? Do we pass Layout to Appenders or PagedLayouts? Would we need
to check with instanceof? This does introduce complexity.
Paying the cost of complexity would be fine if we get some benefit in
return, but what would that
Abhinav Shah created LOG4J2-359:
---
Summary: Log4jServletContextListener does not work on Weblogic
12.1.1 (12c) with web-app version 2.5
Key: LOG4J2-359
URL: https://issues.apache.org/jira/browse/LOG4J2-359
Now I'm re-thinking my seems reasonable. Lol.
Nick
On Aug 16, 2013, at 5:13 PM, Remko Popma wrote:
I'm not sure. What would the code look like if we introduce this
subinterface? Do we pass Layout to Appenders or PagedLayouts? Would we need
to check with instanceof? This does introduce
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742657#comment-13742657
]
Nick Williams edited comment on LOG4J2-344 at 8/16/13 10:20 PM:
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742657#comment-13742657
]
Nick Williams commented on LOG4J2-344:
--
I thought about it, but doing so makes me
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742664#comment-13742664
]
Remko Popma commented on LOG4J2-344:
Keir,
Your stacktrace says Truncated. see log
[
https://issues.apache.org/jira/browse/LOG4J2-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742668#comment-13742668
]
Nick Williams commented on LOG4J2-359:
--
Thanks for the thorough bug report. This may
[
https://issues.apache.org/jira/browse/LOG4J2-359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Nick Williams updated LOG4J2-359:
-
Assignee: Nick Williams
Log4jServletContextListener does not work on Weblogic 12.1.1 (12c)
[
https://issues.apache.org/jira/browse/LOG4J2-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742668#comment-13742668
]
Nick Williams edited comment on LOG4J2-359 at 8/16/13 10:27 PM:
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742672#comment-13742672
]
Nick Williams commented on LOG4J2-344:
--
Keir, just for kicks, can you also post your
[
https://issues.apache.org/jira/browse/LOG4J2-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742677#comment-13742677
]
Abhinav Shah commented on LOG4J2-359:
-
Thanks Nick. I already mentioned that if I
[
https://issues.apache.org/jira/browse/LOG4J2-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742680#comment-13742680
]
Nick Williams commented on LOG4J2-359:
--
Oh. Haha. I missed that part, somehow.
Guys, I'm having a hard time with this simple fix that should have taken five
minutes. I'm getting test failures due to NoClassDefFoundErrors that shouldn't
happen.
Here are the tests in error:
CategoryTest.setupClass:52 ? NoClassDefFound com/mongodb/MongoException
[
https://issues.apache.org/jira/browse/LOG4J2-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742690#comment-13742690
]
Remko Popma commented on LOG4J2-344:
Keir, it would also be good to know if your log
It's likely that MongoDB needs another jar on the CP.
G
On Fri, Aug 16, 2013 at 6:48 PM, Nick Williams
nicho...@nicholaswilliams.net wrote:
Guys, I'm having a hard time with this simple fix that should have taken
five minutes. I'm getting test failures due to NoClassDefFoundErrors that
What is this LoggingTest.cleanupClass:49 NullPointer at the bottom of
your list of errors? What does that do?
On Sat, Aug 17, 2013 at 8:27 AM, Gary Gregory garydgreg...@gmail.comwrote:
It's likely that MongoDB needs another jar on the CP.
G
On Fri, Aug 16, 2013 at 6:48 PM, Nick Williams
Make sure you can see the messages from the status logger, maybe there is
an error processing the log4j config file early on.
G
On Fri, Aug 16, 2013 at 7:38 PM, Remko Popma remko.po...@gmail.com wrote:
What is this LoggingTest.cleanupClass:49 NullPointer at the bottom of
your list of
Several inline comments.
On Aug 16, 2013, at 6:43 PM, Gary Gregory wrote:
Make sure you can see the messages from the status logger, maybe there is an
error processing the log4j config file early on.
There are no messages from the status logger.
G
On Fri, Aug 16, 2013 at 7:38 PM,
Just remove the @doubt and comments
Sent from my iPhone
On Aug 16, 2013, at 3:21 PM, Nick Williams nicho...@nicholaswilliams.net
wrote:
Now I'm re-thinking my seems reasonable. Lol.
Nick
On Aug 16, 2013, at 5:13 PM, Remko Popma wrote:
I'm not sure. What would the code look like if we
On Fri, Aug 16, 2013 at 8:31 PM, Nick Williams
nicho...@nicholaswilliams.net wrote:
Several inline comments.
On Aug 16, 2013, at 6:43 PM, Gary Gregory wrote:
Make sure you can see the messages from the status logger, maybe there is
an error processing the log4j config file early on.
On Fri, Aug 16, 2013 at 9:29 PM, Ralph Goers ralph.go...@dslextreme.comwrote:
Just remove the @doubt and comments
Done.
Gary
Sent from my iPhone
On Aug 16, 2013, at 3:21 PM, Nick Williams nicho...@nicholaswilliams.net
wrote:
Now I'm re-thinking my seems reasonable. Lol.
Nick
On
If you look at org.apache.logging.log4j.core.lookup.StrLookup, you can see
the interface and its impls in the same package.
For org.apache.logging.log4j.core.LayoutT extends Serializable though,
the impls are in org.apache.logging.log4j.core.layout.
Shouldn't we move Layout to
On Fri, Aug 16, 2013 at 9:42 PM, Gary Gregory garydgreg...@gmail.comwrote:
If you look at org.apache.logging.log4j.core.lookup.StrLookup, you can see
the interface and its impls in the same package.
For org.apache.logging.log4j.core.LayoutT extends Serializable though,
the impls are in
On Aug 16, 2013, at 8:38 PM, Gary Gregory wrote:
On Fri, Aug 16, 2013 at 8:31 PM, Nick Williams
nicho...@nicholaswilliams.net wrote:
Several inline comments.
On Aug 16, 2013, at 6:43 PM, Gary Gregory wrote:
Make sure you can see the messages from the status logger, maybe there is an
Easily done, assuming we have consensus. I am hearing two options:
1) strip '-' characters from element names.
2) allow aliases for element names.
These are not mutually exclusive. I see no reason not to go ahead with number
2 and we can continue to discuss where else number 1 might be used.
Yes. The interfaces are at core and the implementations are in their respective
sub-packages. It seemed reasonable when I did it to group them together like
that, and it still does, at least to me.
Ralph
On Aug 16, 2013, at 6:44 PM, Gary Gregory wrote:
On Fri, Aug 16, 2013 at 9:42 PM, Gary
I'm fine with number 2 (for the elements and attributes that used to have a
hyphen in their name).
Not a big fan of 1, not sure what the benefit would be.
On Sat, Aug 17, 2013 at 10:58 AM, Ralph Goers ralph.go...@dslextreme.comwrote:
Easily done, assuming we have consensus. I am hearing two
[
https://issues.apache.org/jira/browse/LOG4J2-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742828#comment-13742828
]
Ralph Goers commented on LOG4J2-359:
Nick, Log4jServletContainerInitializer is adding
[
https://issues.apache.org/jira/browse/LOG4J2-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13742828#comment-13742828
]
Ralph Goers edited comment on LOG4J2-359 at 8/17/13 3:44 AM:
-
https://github.com/mongodb/mongo-java-driver/blob/master/src/main/com/mongodb/MongoException.java
shows an import for org.bson.BSONObject. The pom.xml for mongo-java-driver
doesn't contain a transitive dependency for that and mvn dependency:tree on
core doesn't show it.
Ralph
On Aug 16,
FWIW, rather than drag in another dependency you might just want to catch
RuntimeException instead of MongoException. Of course it will have to come
after the catch block for the IllegalStateException. And if you want you can
just do
if
https://github.com/mongodb/mongo-java-driver/blob/master/src/main/com/mongodb/DB.java
That also shows an import for org.bson.BSONObject, but the tests still run with
just DB and no MongoException. org.bson is in the org.mongodb:mongo-java-driver
JAR file. So, no, that's not the problem. There's
Ralph Goers created LOG4J2-360:
--
Summary: Allow Plugins to have aliases
Key: LOG4J2-360
URL: https://issues.apache.org/jira/browse/LOG4J2-360
Project: Log4j 2
Issue Type: Improvement
After following the chain of stuff that gets brought in via BSONObject I still
recommend the approach in my other email of just catching RuntimeException. A
bunch of other classes are being referenced, one of which is creating a static
Logger from java.util.logging. I have no idea why that
That approach concerns me. Catching RuntimeException essentially opens it up to
thousands of possible exceptions that could be the cause, as opposed to looking
for that exact cause. I suppose I don't have a choice, though. This apparently
just isn't going to work.
Definitely agreed that there
It seems inconsistent to me, but then StrLookup should be moved up to
match the style of the others?
Gary
On Aug 16, 2013, at 22:05, Ralph Goers ralph.go...@dslextreme.com wrote:
StrLookup
-
To unsubscribe, e-mail:
I think we should do both.
Gary
On Aug 16, 2013, at 21:59, Ralph Goers ralph.go...@dslextreme.com wrote:
Easily done, assuming we have consensus. I am hearing two options:
1) strip '-' characters from element names.
2) allow aliases for element names.
These are not mutually exclusive. I see
Why not try to create the proper dependencies and see if we can get it
working? How else can we unit test otherwise?
Gary
On Sat, Aug 17, 2013 at 12:49 AM, Nick Williams
nicho...@nicholaswilliams.net wrote:
That approach concerns me. Catching RuntimeException essentially opens it
up to
The problem is we don't know what's missing and we can't tell what's missing.
I'm not even convinced that the problem IS a missing dependency. Whenever I've
seen a missing dependency, the NoClassDefFoundError was for the actual missing
dependency, not for the class that depended on it. This
Is there any way it could be getting loaded in another class loader?
Verbose classes flag help to determine that?
On Aug 16, 2013 10:37 PM, Nick Williams nicho...@nicholaswilliams.net
wrote:
The problem is we don't know what's missing and we can't tell what's
missing. I'm not even convinced
Yeah. StrLookup should be moved up to core.
Ralph
On Aug 16, 2013, at 10:17 PM, Gary Gregory wrote:
It seems inconsistent to me, but then StrLookup should be moved up to
match the style of the others?
Gary
On Aug 16, 2013, at 22:05, Ralph Goers ralph.go...@dslextreme.com wrote:
I'll reiterate what I wrote. Catch the RuntimeException and then do
if (e.class.getName().equals(com.mongodb.MongoException)) {
LOGGER.error(...);
} else {
throw e;
}
This should give you the same behavior.
Ralph
On Aug 16, 2013, at 9:49 PM, Nick Williams wrote:
That approach concerns
So far yours is the only vote for that. Anyone else?
Ralph
On Aug 16, 2013, at 10:19 PM, Gary Gregory wrote:
I think we should do both.
Gary
On Aug 16, 2013, at 21:59, Ralph Goers ralph.go...@dslextreme.com wrote:
Easily done, assuming we have consensus. I am hearing two options:
On Sat, Aug 17, 2013 at 1:44 AM, Ralph Goers ralph.go...@dslextreme.comwrote:
So far yours is the only vote for that. Anyone else?
Whomever else mentioned it in the first place! ;) I can't recall who... but
it's 2am here...
G
Ralph
On Aug 16, 2013, at 10:19 PM, Gary Gregory wrote:
I
79 matches
Mail list logo