carterkozak edited a comment on pull request #655: URL: https://github.com/apache/logging-log4j2/pull/655#issuecomment-1000906261
> Woa, aren't we missing unit tests that exercise those lookups? I think the issue is abundance of branches. The non-plugin constructor is used initially when the configuration is created to interpolate the Property values themselves, then the interpolator is replaced with one which does use plugins. Existing uses of `config.getStrSubstitutor()` would work fine, while `new StrSubstitutor()` and some overloads would not. -- 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]
