Skip to content

Commit 1173824

Browse files
committed
Revert changes not needed anymore after self-review
1 parent 6662850 commit 1173824

File tree

3 files changed

+4
-14
lines changed

3 files changed

+4
-14
lines changed

instrumentation/internal/internal-class-loader/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/internal/classloader/ClassLoaderInstrumentationModule.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,13 +10,11 @@
1010
import com.google.auto.service.AutoService;
1111
import io.opentelemetry.javaagent.extension.instrumentation.InstrumentationModule;
1212
import io.opentelemetry.javaagent.extension.instrumentation.TypeInstrumentation;
13-
import io.opentelemetry.javaagent.extension.instrumentation.internal.ExperimentalInstrumentationModule;
1413
import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
1514
import java.util.List;
1615

1716
@AutoService(InstrumentationModule.class)
18-
public class ClassLoaderInstrumentationModule extends InstrumentationModule
19-
implements ExperimentalInstrumentationModule {
17+
public class ClassLoaderInstrumentationModule extends InstrumentationModule {
2018
public ClassLoaderInstrumentationModule() {
2119
super("internal-class-loader");
2220
}

instrumentation/internal/internal-class-loader/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/internal/classloader/ResourceInjectionInstrumentation.java

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88
import static io.opentelemetry.javaagent.extension.matcher.AgentElementMatchers.extendsClass;
99
import static net.bytebuddy.matcher.ElementMatchers.isMethod;
1010
import static net.bytebuddy.matcher.ElementMatchers.named;
11-
import static net.bytebuddy.matcher.ElementMatchers.namedOneOf;
12-
import static net.bytebuddy.matcher.ElementMatchers.not;
1311
import static net.bytebuddy.matcher.ElementMatchers.returns;
1412
import static net.bytebuddy.matcher.ElementMatchers.takesArguments;
1513

@@ -34,12 +32,7 @@ public class ResourceInjectionInstrumentation implements TypeInstrumentation {
3432

3533
@Override
3634
public ElementMatcher<TypeDescription> typeMatcher() {
37-
return extendsClass(named("java.lang.ClassLoader"))
38-
.and(
39-
not(
40-
namedOneOf(
41-
"io.opentelemetry.javaagent.bootstrap.AgentClassLoader",
42-
"io.opentelemetry.javaagent.tooling.instrumentation.indy.InstrumentationModuleClassLoader")));
35+
return extendsClass(named("java.lang.ClassLoader"));
4336
}
4437

4538
@Override

javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/instrumentation/indy/InstrumentationModuleClassLoader.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -150,9 +150,8 @@ public synchronized void installModule(InstrumentationModule module) {
150150
className -> BytecodeWithUrl.create(className, agentOrExtensionCl)));
151151
installInjectedClasses(classesToInject);
152152
if (module instanceof ExperimentalInstrumentationModule) {
153-
ExperimentalInstrumentationModule experimentalModule =
154-
(ExperimentalInstrumentationModule) module;
155-
hiddenAgentPackages.addAll(experimentalModule.agentPackagesToHide());
153+
hiddenAgentPackages.addAll(
154+
((ExperimentalInstrumentationModule) module).agentPackagesToHide());
156155
}
157156
}
158157

0 commit comments

Comments
 (0)