[GitHub] [logging-log4j2] rgoers opened a new pull request #677: Log4 j2 3301

[GitBox <gi...@apache.org>]

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

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

[GitHub] [logging-log4j2] rgoers merged pull request #677: Log4 j2 3306

[GitBox <gi...@apache.org>]

rgoers merged pull request #677:
URL: https://github.com/apache/logging-log4j2/pull/677


   


-- 
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