-
Notifications
You must be signed in to change notification settings - Fork 41.2k
Add support for log4j2 composite configuration #27110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Thanks for the suggestion. Supporting a comma-separated list and turning that into a |
I am not keen to see such loose use of a property that's logging system agnostic. It's tricky because creating a specific property will create another set of issues. |
We could support a |
I will be working on this issue |
I just saw this issue as I noticed the change in the latest source code. I should point out that Log4j already provided support for this in its log4j-spring-boot module. In addition, the support that was implemented here doesn't support http(s) connections that require authentication which log4j-spring-boot does. I'd be more than happy to be able to remove log4j-spring-boot once Spring Boot provides the equivalent functionality. |
https://logging.apache.org/log4j/2.x/manual/configuration.html#CompositeConfiguration
Here is my use case, I want output logs to ELK in production, so I add extra
log4j2-kafka-appender.xml
besidelog4j2.xml
.I need merge two files without composite configuration supports, which will make configuration redundant.
The text was updated successfully, but these errors were encountered: