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
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.
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
+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
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
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
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