From c65c79207a06203d7dea0bf543815a7bf678bb07 Mon Sep 17 00:00:00 2001 From: Tyler Benson Date: Thu, 14 Feb 2019 08:45:02 -0800 Subject: [PATCH] Remove `isSynthetic()` ignore MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit There’s concern that this will cause classes to be skipped that we do want to instrument. --- .../main/java/datadog/trace/agent/tooling/AgentInstaller.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/dd-java-agent/agent-tooling/src/main/java/datadog/trace/agent/tooling/AgentInstaller.java b/dd-java-agent/agent-tooling/src/main/java/datadog/trace/agent/tooling/AgentInstaller.java index 687a151e1b..54dcf22ae3 100644 --- a/dd-java-agent/agent-tooling/src/main/java/datadog/trace/agent/tooling/AgentInstaller.java +++ b/dd-java-agent/agent-tooling/src/main/java/datadog/trace/agent/tooling/AgentInstaller.java @@ -69,8 +69,6 @@ public class AgentInstaller { .or(ElementMatchers.isAnnotation()) // Unlikely to ever need to instrument an enum: .or(ElementMatchers.isEnum()) - // Exclude generated classes like proxies: - .or(ElementMatchers.isSynthetic()) .or( nameStartsWith("datadog.trace.") // FIXME: We should remove this once