Skip to content

Commit cfdecaf

Browse files
committed
sling: don't add unused method arguments to instrumentation
1 parent 906bef0 commit cfdecaf

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

instrumentation/sling/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/sling/SlingSafeMethodsServletInstrumentation.java

-3
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
import net.bytebuddy.matcher.ElementMatcher;
2121
import org.apache.sling.api.SlingHttpServletRequest;
2222
import javax.servlet.ServletRequest;
23-
import javax.servlet.ServletResponse;
2423

2524
public class SlingSafeMethodsServletInstrumentation implements TypeInstrumentation {
2625
@Override
@@ -50,7 +49,6 @@ public static class ServiceServletAdvice {
5049
@Advice.OnMethodEnter(suppress = Throwable.class)
5150
public static void onEnter(
5251
@Advice.Argument(0) ServletRequest request,
53-
@Advice.Argument(1) ServletResponse response,
5452
@Advice.Local("otelContext") Context context,
5553
@Advice.Local("otelScope") Scope scope) {
5654

@@ -86,7 +84,6 @@ public static void onEnter(
8684
@Advice.OnMethodExit(suppress = Throwable.class, onThrowable = Throwable.class)
8785
public static void onExit(
8886
@Advice.Argument(0) ServletRequest request,
89-
@Advice.Argument(1) ServletResponse response,
9087
@Advice.Thrown Throwable throwable,
9188
@Advice.Local("otelContext") Context context,
9289
@Advice.Local("otelScope") Scope scope) {

0 commit comments

Comments
 (0)