Re: [PR] StrSubstitutor cannot parse nested braces (logging-log4j2)

2024-07-09 Thread via GitHub
ngocnhan-tran1996 closed pull request #2712: StrSubstitutor cannot parse nested braces URL: https://github.com/apache/logging-log4j2/pull/2712 -- 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 spe

Re: [PR] StrSubstitutor cannot parse nested braces (logging-log4j2)

2024-07-09 Thread via GitHub
ngocnhan-tran1996 commented on PR #2712: URL: https://github.com/apache/logging-log4j2/pull/2712#issuecomment-2217232851 @ppkarwasz I see, thanks for your help. I will close this PR -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] StrSubstitutor cannot parse nested braces (logging-log4j2)

2024-07-09 Thread via GitHub
ppkarwasz commented on PR #2712: URL: https://github.com/apache/logging-log4j2/pull/2712#issuecomment-2216804216 > IMHO `StrLookup` should be aware of the existence and syntax of pattern specifiers `%{...}`, it should just count the number of `{` and `}`. We also need to have a mechanism to

Re: [PR] StrSubstitutor cannot parse nested braces (logging-log4j2)

2024-07-06 Thread via GitHub
ngocnhan-tran1996 commented on PR #2712: URL: https://github.com/apache/logging-log4j2/pull/2712#issuecomment-2211724941 @ppkarwasz I try to create class `ConverterKeysSubstitutor` for pattern specifiers ` %{...}`. Please take a look and give me more test or some advice. -- This i

Re: [PR] StrSubstitutor cannot parse nested braces (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on PR #2712: URL: https://github.com/apache/logging-log4j2/pull/2712#issuecomment-2210973106 @ngocnhan-tran1996, Thank you for the PR. Unfortunately this does not solve #2679, but just your particular problem with the `%d{...}` pattern converter as default value

[PR] StrSubstitutor cannot parse nested braces (logging-log4j2)

2024-07-04 Thread via GitHub
ngocnhan-tran1996 opened a new pull request, #2712: URL: https://github.com/apache/logging-log4j2/pull/2712 As reported in: [2679](https://github.com/apache/logging-log4j2/issues/2679) -- This is an automated message from the Apache Git Service. To respond to the message, please log