open-telemetry
/
opentelemetry-java-instrumentation
mirror of https://github.com/open-telemetry/opentelemetry-java-instrumentation.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Workaround module bug by skipping LogManager$Cleaner instrumentation

This commit is contained in:
Andrew Kent 2019-01-09 09:20:07 -08:00
parent 4a6bd7a097
commit cc8db0b11f
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
dd-java-agent/agent-tooling/src/main/java/datadog/trace/agent/tooling/AgentInstaller.java
View File

@ -80,7 +80,13 @@ public class AgentInstaller {
named("java.net.URL") named("java.net.URL")
.or(named("java.net.HttpURLConnection")) .or(named("java.net.HttpURLConnection"))
.or(nameStartsWith("java.util.concurrent.")) .or(nameStartsWith("java.util.concurrent."))
.or(nameStartsWith("java.util.logging."))))) .or(
nameStartsWith("java.util.logging.")
// Concurrent instrumentation modifies the strucutre of
// Cleaner class incompaibly with java9+ modules.
// Working around until a long-term fix for modules can be
// put in place.
.and(not(named("java.util.logging.LogManager$Cleaner")))))))
.or(nameStartsWith("com.sun.").and(not(nameStartsWith("com.sun.messaging.")))) .or(nameStartsWith("com.sun.").and(not(nameStartsWith("com.sun.messaging."))))
.or( .or(
nameStartsWith("sun.") nameStartsWith("sun.")