Re: [Log4j] On the use of JSON for a default format in 3.x

2022-02-27 Thread Matt Sicker
Started a PR for this: https://github.com/apache/logging-log4j2/pull/775 On Thu, Feb 17, 2022 at 6:39 AM Volkan Yazıcı wrote: > > There is already one: JsonReader – a single-file JSON parser provided by > JsonTemplateLayout. > > On Thu, Feb 17, 2022 at 10:41 AM Jochen Wiedmann > wrote: > > > How

Re: [Log4j] On the use of JSON for a default format in 3.x

2022-02-17 Thread Volkan Yazıcı
There is already one: JsonReader – a single-file JSON parser provided by JsonTemplateLayout. On Thu, Feb 17, 2022 at 10:41 AM Jochen Wiedmann wrote: > How about adopting a minimal Json parser into log4j-core? Examples: > > 1: https://github.com/KasparNagu/plain-java-json (ASL) > 2: https://code.

Re: [Log4j] On the use of JSON for a default format in 3.x

2022-02-17 Thread Jochen Wiedmann
How about adopting a minimal Json parser into log4j-core? Examples: 1: https://github.com/KasparNagu/plain-java-json (ASL) 2: https://code.google.com/archive/p/json-simple/ (ASL) 3: https://github.com/ralfstx/minimal-json (MIT) Seems doable, and would work. Jochen On Thu, Feb 17, 2022 at 9:56 A

Re: [Log4j] On the use of JSON for a default format in 3.x

2022-02-17 Thread Volkan Yazıcı
+1 for supporting JSON as a built-in configuration format If we happen to move JsonReader to somewhere else, we shouldn't forget moving JsonReaderTest too. On Wed, Feb 16, 2022 at 7:32 PM Matt Sicker wrote: > We've had some conversations about this off and on, though I don't > think we've had a

Re: [Log4j] On the use of JSON for a default format in 3.x

2022-02-16 Thread Gary Gregory
This would be a good one to discuss and flush out in a conference call. Gary On Wed, Feb 16, 2022, 13:32 Matt Sicker wrote: > We've had some conversations about this off and on, though I don't > think we've had any formal decision or discussion on it. For Log4j > 3.x, we'd like to make it so th

Re: [Log4j] On the use of JSON for a default format in 3.x

2022-02-16 Thread Ralph Goers
Actually, I was planning on working on converting the JSON configuration to use JsonReader as soon as we get 2.17.2 released. There are just so many things worth fixing for this one that it is taking longer than I had hoped. I will also be fixing the other things that require modules other than

[Log4j] On the use of JSON for a default format in 3.x

2022-02-16 Thread Matt Sicker
We've had some conversations about this off and on, though I don't think we've had any formal decision or discussion on it. For Log4j 3.x, we'd like to make it so that log4j-api, log4j-plugins, and log4j-core only require the java.base module with optional dependencies allowed for certain plumbing