fix LOGBACK-1698, [Nested appenders are not allowed] warning using Si…

…ftingAppender Signed-off-by: Ceki Gulcu <ceki@qos.ch>
qos-ch · Nov 18, 2022 · 0d3ac63 · 0d3ac63
1 parent a64b8d4
commit 0d3ac63
Show file tree

Hide file tree

Showing 16 changed files with 32 additions and 17 deletions.
diff --git a/logback-classic/src/test/input/joran/sift/completeCycle.xml b/logback-classic/src/test/input/joran/sift/completeCycle.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
   <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
 

diff --git a/logback-classic/src/test/input/joran/sift/compositeProperty.xml b/logback-classic/src/test/input/joran/sift/compositeProperty.xml
@@ -1,5 +1,5 @@
 
-<configuration debug="true">
+<configuration debug="false">
 
   <property name="X" value="composite"/>  
 

diff --git a/logback-classic/src/test/input/joran/sift/defaultLayoutRule.xml b/logback-classic/src/test/input/joran/sift/defaultLayoutRule.xml
@@ -1,4 +1,4 @@
-<configuration debug="true">
+<configuration debug="false">
 
   <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
     <discriminator>

diff --git a/logback-classic/src/test/input/joran/sift/fileAppender.xml b/logback-classic/src/test/input/joran/sift/fileAppender.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
 	<appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
 

diff --git a/logback-classic/src/test/input/joran/sift/hoard0.xml b/logback-classic/src/test/input/joran/sift/hoard0.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
   <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
 

diff --git a/logback-classic/src/test/input/joran/sift/lingering.xml b/logback-classic/src/test/input/joran/sift/lingering.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
   <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
 

diff --git a/logback-classic/src/test/input/joran/sift/maxAppenderCount.xml b/logback-classic/src/test/input/joran/sift/maxAppenderCount.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
     <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
         <discriminator>

diff --git a/logback-classic/src/test/input/joran/sift/multipleNesting.xml b/logback-classic/src/test/input/joran/sift/multipleNesting.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
   <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
 

diff --git a/logback-classic/src/test/input/joran/sift/propertyDefinedInSiftElement.xml b/logback-classic/src/test/input/joran/sift/propertyDefinedInSiftElement.xml
@@ -1,4 +1,4 @@
-<configuration debug="true">
+<configuration debug="false">
 
   <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
     <discriminator>

diff --git a/logback-classic/src/test/input/joran/sift/propertyPropagation.xml b/logback-classic/src/test/input/joran/sift/propertyPropagation.xml
@@ -1,4 +1,4 @@
-<configuration debug="true">
+<configuration debug="false">
 
 
   <property name="X" value="Y"/>  

diff --git a/logback-classic/src/test/input/joran/sift/smoke.xml b/logback-classic/src/test/input/joran/sift/smoke.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
   <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
 

diff --git a/logback-classic/src/test/input/joran/sift/timeout.xml b/logback-classic/src/test/input/joran/sift/timeout.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
     <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
 

diff --git a/logback-classic/src/test/input/joran/sift/unsetDefaultValueProperty.xml b/logback-classic/src/test/input/joran/sift/unsetDefaultValueProperty.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
   <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
 

diff --git a/logback-classic/src/test/input/joran/sift/zeroNesting.xml b/logback-classic/src/test/input/joran/sift/zeroNesting.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!DOCTYPE configuration>
 
-<configuration debug="true">
+<configuration debug="false">
 
   <appender name="SIFT" class="ch.qos.logback.classic.sift.SiftingAppender">
 

diff --git a/logback-classic/src/test/java/ch/qos/logback/classic/sift/SiftingAppenderTest.java b/logback-classic/src/test/java/ch/qos/logback/classic/sift/SiftingAppenderTest.java
@@ -103,6 +103,7 @@ public void smoke() throws JoranException {
         List<ILoggingEvent> eventList = listAppender.list;
         assertEquals(1, listAppender.list.size());
         assertEquals("smoke", eventList.get(0).getMessage());
+        statusChecker.assertIsWarningOrErrorFree();
     }
 
     private AppenderTracker<ILoggingEvent> getAppenderTracker() {

diff --git a/...e/src/main/java/ch/qos/logback/core/joran/sanity/AppenderWithinAppenderSanityChecker.java b/...e/src/main/java/ch/qos/logback/core/joran/sanity/AppenderWithinAppenderSanityChecker.java
@@ -19,6 +19,7 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.stream.Collectors;
 
 public class AppenderWithinAppenderSanityChecker extends ContextAwareBase implements SanityChecker  {
 
@@ -34,13 +35,26 @@ public void check(Model model) {
 
         List<Pair<Model, Model>> nestedPairs = deepFindNestedSubModelsOfType(AppenderModel.class, appenderModels);
 
-        if(nestedPairs.isEmpty())
-            return;
+        List<Pair<Model, Model>> filteredNestedPairs = nestedPairs.stream().filter(pair -> !isSiftingAppender(pair.first)).collect(Collectors.toList());
 
+        if(filteredNestedPairs.isEmpty()) {
+            return;
+        }
         addWarn(NESTED_APPENDERS_WARNING);
-        for(Pair<Model, Model> pair: nestedPairs) {
+        for(Pair<Model, Model> pair: filteredNestedPairs) {
             addWarn("Appender at line "+pair.first.getLineNumber() + " contains a nested appender at line "+pair.second.getLineNumber());
         }
     }
 
+    private boolean isSiftingAppender(Model first) {
+        if(first instanceof  AppenderModel) {
+            AppenderModel appenderModel = (AppenderModel) first;
+            String classname = appenderModel.getClassName();
+            if(classname == null)
+                return false;
+            return appenderModel.getClassName().contains("SiftingAppender");
+        }
+        return false;
+    }
+
 }