You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by GitBox <gi...@apache.org> on 2022/02/28 08:13:03 UTC

[GitHub] [logging-log4j2] ppkarwasz opened a new pull request #777: [LOG4J2-3418] Fix Log4j2CloudConfigLoggingSystem registration

ppkarwasz opened a new pull request #777:
URL: https://github.com/apache/logging-log4j2/pull/777


   This PR adds support for the `spring.factories` registration method of the `Log4j2CloudConfigLoggingSystem` to replace the old Java system properties injection method.
   
   A Log4j 2.x `log4j2.disableCloudConfigLoggingSystem` property is added to enable a choice between the `Log4j2CloudConfigLoggingSystem` and `Log4j2LoggingSystem`.


-- 
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: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [logging-log4j2] ppkarwasz commented on pull request #777: [LOG4J2-3418] Fix Log4j2CloudConfigLoggingSystem registration

Posted by GitBox <gi...@apache.org>.
ppkarwasz commented on pull request #777:
URL: https://github.com/apache/logging-log4j2/pull/777#issuecomment-1063246235


   > Could you add documentation how to activate this feature in future?
   
   The behavior of the artifact does not change: you add it to the classpath and Spring Boot detects it. What changes is the way this is done: before Java system properties were used, so **all** Spring Boot applications in the JVM tried to use this plugin. After this PR only the one with `log4j-spring-boot` will use it.


-- 
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: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [logging-log4j2] membersound commented on pull request #777: [LOG4J2-3418] Fix Log4j2CloudConfigLoggingSystem registration

Posted by GitBox <gi...@apache.org>.
membersound commented on pull request #777:
URL: https://github.com/apache/logging-log4j2/pull/777#issuecomment-1062647691


   Could you add documentation how to activate this feature in future?


-- 
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: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [logging-log4j2] ppkarwasz merged pull request #777: [LOG4J2-3418] Fix Log4j2CloudConfigLoggingSystem registration

Posted by GitBox <gi...@apache.org>.
ppkarwasz merged pull request #777:
URL: https://github.com/apache/logging-log4j2/pull/777


   


-- 
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: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org