[zookeeper] branch branch-3.8 updated: ZOOKEEPER-4465: zooinspector logback pattern config add escape for '(' and ')'

[symat]

This is an automated email from the ASF dual-hosted git repository.

symat pushed a commit to branch branch-3.8
in repository https://gitbox.apache.org/repos/asf/zookeeper.git


The following commit(s) were added to refs/heads/branch-3.8 by this push:
     new d4fe0c6  ZOOKEEPER-4465: zooinspector logback pattern config add 
escape for '(' and ')'
d4fe0c6 is described below

commit d4fe0c64721a1bc0fa28ded40b07f5da79cae77e
Author: 67 <6...@gd67.com>
AuthorDate: Tue Feb 15 11:29:09 2022 +0000

    ZOOKEEPER-4465: zooinspector logback pattern config add escape for '(' and 
')'
    
    zooinspect logback config file `logback.xml` currently use a pattern of this
    `<pattern>%5p [%t] (%F:%L) - %m%n</pattern> `
    which not escape the '(' and ')', cause logback to ignore parts after ')'.
    
    according to logback documents, '(' and ')' is used for grouping, need 
escape by `\` if used as normal char
    https://logback.qos.ch/manual/layouts.html#grouping
    
    this pr update it to (add '\' to escape)
    `<pattern>%5p [%t] \(%F:%L\) - %m%n</pattern> `
    
    Author: 67 <6...@gd67.com>
    
    Reviewers: Enrico Olivelli <eolive...@apache.org>, Andor Molnar 
<an...@apache.org>, Mate Szalay-Beko <sy...@apache.org>
    
    Closes #1814 from iamgd67/ZOOKEEPER-4465
    
    (cherry picked from commit 48191b63dfaa4d6d71572608a31338e15f02f0fa)
    Signed-off-by: Mate Szalay-Beko <sy...@apache.org>
---
 .../src/main/resources/logback.xml                 |  2 +-
 .../org/apache/zookeeper/inspector/LoggerTest.java | 48 ++++++++++++++++++++++
 2 files changed, 49 insertions(+), 1 deletion(-)

diff --git 
a/zookeeper-contrib/zookeeper-contrib-zooinspector/src/main/resources/logback.xml
 
b/zookeeper-contrib/zookeeper-contrib-zooinspector/src/main/resources/logback.xml
index 983c188..6f81303 100644
--- 
a/zookeeper-contrib/zookeeper-contrib-zooinspector/src/main/resources/logback.xml
+++ 
b/zookeeper-contrib/zookeeper-contrib-zooinspector/src/main/resources/logback.xml
@@ -22,7 +22,7 @@
 <configuration>
   <appender name="stdout" class="ch.qos.logback.core.ConsoleAppender">
     <encoder>
-      <pattern>%5p [%t] (%F:%L) - %m%n</pattern>
+      <pattern>%5p [%t] \(%F:%L\) - %m%n</pattern>
     </encoder>
   </appender>
   <root level="INFO">
diff --git 
a/zookeeper-contrib/zookeeper-contrib-zooinspector/src/test/java/org/apache/zookeeper/inspector/LoggerTest.java
 
b/zookeeper-contrib/zookeeper-contrib-zooinspector/src/test/java/org/apache/zookeeper/inspector/LoggerTest.java
new file mode 100644
index 0000000..310e332
--- /dev/null
+++ 
b/zookeeper-contrib/zookeeper-contrib-zooinspector/src/test/java/org/apache/zookeeper/inspector/LoggerTest.java
@@ -0,0 +1,48 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.zookeeper.inspector;
+
+import org.junit.Assert;
+import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import java.io.ByteArrayOutputStream;
+import java.io.PrintStream;
+import java.nio.charset.StandardCharsets;
+
+public class LoggerTest {
+    Logger LOG = LoggerFactory.getLogger(LoggerTest.class);
+    String testMessage = "a test message";
+
+    @Test
+    public void testLogStdOutConfig() {
+        ByteArrayOutputStream byteArrayOutputStream = new 
ByteArrayOutputStream();
+        PrintStream realStdOut = System.out;
+        System.setOut(new PrintStream(byteArrayOutputStream));
+        LOG.info(testMessage);
+        System.setOut(realStdOut);
+
+        //log to stdout for debug
+        LOG.info(testMessage);
+        String bufferMessage = new String(byteArrayOutputStream.toByteArray(), 
StandardCharsets.UTF_8);
+        LOG.info(bufferMessage);
+
+        Assert.assertTrue(bufferMessage.contains(testMessage));
+    }
+}