Introduce JSON logging

[dmlloyd]

Fixes #1847

[dmlloyd]

So here's an interesting speed bump. If I want to enable JSON logging, I need the JSONP libraries. However we definitely don't want them if it's not going to be enabled.

Should we always fold in the JSON libraries? Or, should we allow an incomplete class path in this case?

[dmlloyd]

Alternatively I could introduce an entire extension mechanism for formatter type, and then we'd have a quarkus-logging-json extension. It'd be a bit funny though given we don't have a quarkus-logging extension.

[dmlloyd]

For now I've just added the JSON library as a core dependency.

[gsmet]

I added an important question inline. Could you take a look?

[gsmet]

I can't say I'm excited to add a dependency to javax JSON in the core.I have another question: does it work for a native executable? I would have expected the quarkus-jsonp extension to be necessary.

If so, it makes the case of a specific extension IMHO.

[dmlloyd]

It seemed to work. But yeah maybe a specific extension is warranted.

[mkouba]

For the record - quarkus-jsonp only registers org.glassfish.json.JsonProviderImpl for reflection so that javax.json.spi.JsonProvider could instantiate the class.

[mkouba]

org.glassfish:javax.json is quite small so I wouldn't probably mind adding it to the core as it could be useful. On the other hand, if we decide to include this in the core we should probably remove the quarkus-jsonp extension and move the JsonpProcessor logic to the core.

[aloubyansky]

@dmlloyd do you want me to merge this and create an issue for the dedicated extension? Or other preference?

[dmlloyd]

I think it should be reworked to be a separate extension. I just haven't gotten around to it yet. If you want, we can close the PR for now to reduce clutter, and I'll reopen it if/when it's ready.

[stuartwdouglas]

I think this should definitely be a seperate extension. We don't want to introduce any additional core dependencies unless we absolutely have to.

[cescoffier]

@dmlloyd Should we close this PR and get some fresh start around a separate extension?

[dmlloyd]

Sure.

[cescoffier]

Thanks!