Exclude method handles exceptions from telemetry logs (#7208)

DataDog · Jun 17, 2024 · 5cf2f32 · 5cf2f32
1 parent 841d933
commit 5cf2f32
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/internal-api/src/main/java/datadog/trace/util/MethodHandles.java b/internal-api/src/main/java/datadog/trace/util/MethodHandles.java
@@ -1,5 +1,6 @@
 package datadog.trace.util;
 
+import datadog.trace.api.telemetry.LogCollector;
 import java.lang.invoke.MethodHandle;
 import java.lang.reflect.Constructor;
 import java.lang.reflect.Field;
@@ -49,6 +50,7 @@ public MethodHandle privateFieldGetter(Class<?> clazz, String fieldName) {
 
               } catch (Throwable t) {
                 log.debug(
+                    LogCollector.EXCLUDE_TELEMETRY,
                     "Could not get private field {} getter from class {}",
                     fieldName,
                     clazz.getName(),
@@ -84,6 +86,7 @@ public MethodHandle privateFieldSetter(Class<?> clazz, String fieldName) {
 
               } catch (Throwable t) {
                 log.debug(
+                    LogCollector.EXCLUDE_TELEMETRY,
                     "Could not get private field {} setter from class {}",
                     fieldName,
                     clazz.getName(),
@@ -119,6 +122,7 @@ public MethodHandle constructor(Class<?> clazz, Class<?>... parameterTypes) {
 
               } catch (Throwable t) {
                 log.debug(
+                    LogCollector.EXCLUDE_TELEMETRY,
                     "Could not get constructor accepting {} from class {}",
                     Arrays.toString(parameterTypes),
                     clazz.getName(),
@@ -154,6 +158,7 @@ public MethodHandle method(Class<?> clazz, String methodName, Class<?>... parame
 
               } catch (Throwable t) {
                 log.debug(
+                    LogCollector.EXCLUDE_TELEMETRY,
                     "Could not get method {} accepting {} from class {}",
                     methodName,
                     Arrays.toString(parameterTypes),
@@ -191,7 +196,11 @@ public MethodHandle method(Class<?> clazz, Predicate<Method> filter) {
                 return null;
 
               } catch (Throwable t) {
-                log.debug("Could not find desired method in class {}", clazz, t);
+                log.debug(
+                    LogCollector.EXCLUDE_TELEMETRY,
+                    "Could not find desired method in class {}",
+                    clazz,
+                    t);
                 return null;
               }
             });
@@ -201,7 +210,7 @@ private Class<?> loadClass(String className) {
     try {
       return classLoader.loadClass(className);
     } catch (Throwable t) {
-      log.debug("Could not load class {}", className, t);
+      log.debug(LogCollector.EXCLUDE_TELEMETRY, "Could not load class {}", className, t);
       return null;
     }
   }