PatternLayoutBase custom converter classloading trouble in OSGi about logback

[enapps-enorman]

Hi @ceki

Checking back in again since we haven't gotten any feedback on the PR since it was proposed 4 months ago.

We would appreciate a review of this proposal if you (or someone else) has the time. Knowing if this proposal will be accepted or not will help us decide how to move forward.

Thanks

[ceki]

Hi @enapps-enorman,

Thank you for this PR. I imagine that the suppliers would need to be created by the OSGi platform or the application itself.
How/when will these suppliers be created?

How do you intend to inject the suppliers into logback? You would need hooks in logback itself but these seem to be missing in the PR.

[enapps-enorman]

Hi @ceki

Thanks for your response. This proposal was to resolve a problem discovered during work to migrate existing code from older slf4j(1.x)+logback(1.2.x) versions to the latest releases and the refactoring needed to deal with everything that had changed. For your reference, that pending PR remains at: apache/sling-org-apache-sling-commons-log#18

Thank you for this PR. I imagine that the suppliers would need to be created by the OSGi platform or the application itself. How/when will these suppliers be created?

Yes, our use case does involve some significant customization of the logback init logic for modularization. This was to support supplying the log config more dynamically based on which OSGi configuration+bundles are active in the application runtime.

For your reference, those logback integration customization details are described briefly in the documentation at: https://sling.apache.org/documentation/development/logging.html#logback-integration

How do you intend to inject the suppliers into logback? You would need hooks in logback itself but these seem to be missing in the PR.

The custom logging init code basically has a custom factory for various PatternLayout objects that programmatically populates the instance converter supplier map with the supplier lambda expressions. This output "masking" code was written by someone else and I believe the purpose was to mask certain characters in the output for https://issues.apache.org/jira/browse/SLING-11393

For your reference, the following is the method (from a pending PR) that customizes the PatternLayout with the suppliers:
https://github.com/enapps-enorman/sling-org-apache-sling-commons-log/blob/issue/SLING-11906/src/main/java/org/apache/sling/commons/log/logback/internal/MaskingMessageUtil.java#L53

Does that make sense?