rgoers edited a comment on pull request #363: URL: https://github.com/apache/logging-log4j2/pull/363#issuecomment-636038992
The test is in the API module and is only testing the property name resolution. The CompositeConfiguration is in Core so these properties can't be used for anything in this test. Matt shows up on this because he went through all of Log4j scrounging up the properties and "normalized" and documented them. He probably made a simple error with this. The fact that the documentation is wrong is worse then the test being wrong. But this fix looks good to me. Note that this patch is for master. It will have to also be cherry-picked to release-2.x. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
