vy commented on code in PR #2667: URL: https://github.com/apache/logging-log4j2/pull/2667#discussion_r1644292997
########## log4j-layout-template-json/src/main/java/org/apache/logging/log4j/layout/template/json/resolver/PatternResolver.java: ########## @@ -71,6 +70,16 @@ public final class PatternResolver implements EventResolver { patternLayout.serialize(logEvent, stringBuilder); } + private static String readPattern( + final EventResolverStringSubstitutor substitutor, final TemplateResolverConfig config) { + final String pattern = config.getString("pattern"); + if (Strings.isBlank(pattern)) { + throw new IllegalArgumentException("blank pattern: " + config); + } + final boolean substitutionNeeded = pattern.contains("${"); + return substitutionNeeded ? substitutor.replace(null, pattern) : pattern; Review Comment: Removed in 03993a8eebfb88617d0c602781b07d7e3e05e41b. -- 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-unsubscr...@logging.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org