rgoers opened a new pull request #677: URL: https://github.com/apache/logging-log4j2/pull/677
Reverts to using prior logic but prevents endless conversion. Note that the StrSubstitutor implementation had some issues - it still could get a StackOverflowError, just in a different scenario. StrSubtitutor.replace returns the same string when there is no match whereas strSubst will return an empty string when keys don't have a matching value. Given that we don't know how users are using this class (they are treating it like a utility method to use in their own code) making these changes is the safest approach. -- 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]
