Merge pull request #578 from java-sec/refactor-remove-misleading-logs

pom.xml中junit和jmh提升到全局依赖，不再每个模块单独引入 & api收集时的日志打印忽略已知异常

Author: CC11001100

dongtai-agent/pom.xml

@@ -25,13 +25,6 @@
             <version>${commons-lang3.version}</version>
         </dependency>
 
-        <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <version>${junit.version}</version>
-            <scope>test</scope>
-        </dependency>
-
         <dependency>
             <groupId>org.json</groupId>
             <artifactId>json</artifactId>

dongtai-api-gather/dongtai-api-gather-openapi/pom.xml

@@ -24,12 +24,6 @@
             <version>${fastjson2.version}</version>
             <scope>provided</scope>
         </dependency>
-        <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <version>${junit.version}</version>
-            <scope>test</scope>
-        </dependency>
     </dependencies>
 
 </project>

dongtai-api-gather/dongtai-api-gather-spring-api/pom.xml

@@ -30,12 +30,6 @@
             <version>${project.version}</version>
             <scope>provided</scope>
         </dependency>
-        <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <version>${junit.version}</version>
-            <scope>test</scope>
-        </dependency>
     </dependencies>
 
     <build>

dongtai-api-gather/dongtai-api-gather-spring-api/src/main/java/io/dongtai/iast/api/gather/spring/extractor/SpringMVCApiExtractor.java

@@ -2,6 +2,7 @@
 
 import io.dongtai.iast.api.gather.spring.convertor.RequestMappingHandlerMappingConvertor;
 import io.dongtai.iast.api.openapi.domain.OpenApi;
+import io.dongtai.iast.common.utils.ExceptionUtil;
 import io.dongtai.log.DongTaiLog;
 import org.springframework.web.context.WebApplicationContext;
 import org.springframework.web.servlet.mvc.method.annotation.RequestMappingHandlerMapping;
@@ -72,7 +73,11 @@ private List<RequestMappingHandlerMapping> findRequestMappingHandlerMapping(WebA
                 }
             }
         } catch (Throwable e) {
-            DongTaiLog.debug("try use BeanFactoryUtils find RequestMappingHandlerMapping exception", e);
+            // 仅在出现预期外错误的时候才打印日志
+            String s = ExceptionUtil.getPrintStackTraceString(e);
+            if (!s.contains("java.lang.NoSuchMethodException: org.springframework.beans.factory.BeanFactoryUtils.beansOfTypeIncludingAncestors()")) {
+                DongTaiLog.debug("try use BeanFactoryUtils throw RequestMappingHandlerMapping exception", e);
+            }
         }
 
         // 没有工具类，就只从自己里面找

dongtai-api-gather/pom.xml

@@ -18,4 +18,13 @@
         <module>dongtai-api-gather-dubbo-api</module>
     </modules>
 
+    <dependencies>
+        <dependency>
+            <groupId>io.dongtai.iast</groupId>
+            <artifactId>dongtai-common</artifactId>
+            <version>${revision}</version>
+            <scope>compile</scope>
+        </dependency>
+    </dependencies>
+
 </project>

dongtai-common/pom.xml

@@ -37,12 +37,6 @@
             <artifactId>fastjson2</artifactId>
             <version>${fastjson2.version}</version>
         </dependency>
-
-        <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <scope>test</scope>
-        </dependency>
     </dependencies>
 
     <build>

dongtai-common/src/main/java/io/dongtai/iast/common/utils/ExceptionUtil.java

@@ -0,0 +1,27 @@
+package io.dongtai.iast.common.utils;
+
+import java.io.PrintWriter;
+import java.io.StringWriter;
+
+/**
+ * 处理异常相关的公共逻辑提取到这里
+ *
+ * @author CC11001100
+ * @since 1.13.2
+ */
+public class ExceptionUtil {
+
+    /**
+     * 把printStackTrace会打印的内容以字符串的形式返回
+     *
+     * @param e
+     * @return
+     */
+    public static String getPrintStackTraceString(Throwable e) {
+        StringWriter stringWriter = new StringWriter();
+        PrintWriter printWriter = new PrintWriter(stringWriter);
+        e.printStackTrace(printWriter);
+        return stringWriter.toString();
+    }
+
+}

dongtai-common/src/test/java/io/dongtai/iast/common/utils/ExceptionUtilTest.java

@@ -0,0 +1,20 @@
+package io.dongtai.iast.common.utils;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+import static org.junit.Assert.*;
+
+/**
+ * @author CC11001100
+ */
+public class ExceptionUtilTest {
+
+    @Test
+    public void getPrintStackTraceString() {
+        Exception e = new Exception();
+        String printStackTraceString = ExceptionUtil.getPrintStackTraceString(e);
+        Assert.assertNotNull(printStackTraceString);
+    }
+
+}

dongtai-core/pom.xml

@@ -145,12 +145,6 @@
     </build>
 
     <dependencies>
-        <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <version>${junit.version}</version>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>io.dongtai.iast</groupId>
             <artifactId>dongtai-spy</artifactId>

dongtai-log/pom.xml

@@ -38,23 +38,4 @@
         </plugins>
     </build>
 
-    <dependencies>
-        <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <version>${junit.version}</version>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.openjdk.jmh</groupId>
-            <artifactId>jmh-core</artifactId>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.openjdk.jmh</groupId>
-            <artifactId>jmh-generator-annprocess</artifactId>
-            <scope>test</scope>
-        </dependency>
-    </dependencies>
-
 </project>