jvz opened a new pull request #744: URL: https://github.com/apache/logging-log4j2/pull/744
This replaces the previous attempt at unifying dependency injection and configuration from a CDI-like API to an API more like javax.inject used by libraries like Feather and Guice. https://cwiki.apache.org/confluence/display/LOGGING/Dependency+Injection+and+Configuration ### Draft Status There is still some work to be done in swapping out the old plugin system in `AbstractConfiguration` along with some updates to `LoggerContext` and potentially `Log4jContextFactory` to have `Injector` instances. This will result in removal of at least 14 more classes in log4j-plugins in `bind` and `inject` along with everything in `org.apache.logging.log4j.core.config.plugins.visitors`, `PluginConfigurationInjector`, and at least 100+ lines in `AbstractConfiguration`. Further code simplifications follow from there. I'm opening this draft PR before everything is ready for merging, so here's the remaining tasks: - [ ] Hooked in to `AbstractConfiguration` - [ ] Removal of old classes - [ ] API docs - [ ] Developer docs - [ ] Port over debug logging from previous classes (injection strategies) - [ ] Default `Module` with low priority to bind default implementations of customizable classes -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
