Skip to content

Commit 6b718f4

Browse files
authored
Review fixes for jetty-12.0:javaagent (#18087)
Co-authored-by: otelbot <197425009+otelbot@users.noreply.github.com>
1 parent d4216b7 commit 6b718f4

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

instrumentation/jetty/jetty-12.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jetty/v12_0/Jetty12ServerInstrumentation.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ private AdviceScope(Context context, Scope scope) {
5353
}
5454

5555
@Nullable
56-
public static AdviceScope start(Object source, Request request, Response response) {
56+
public static AdviceScope start(Request request, Response response) {
5757
Context parentContext = Context.current();
5858
if (!helper().shouldStart(parentContext, request)) {
5959
return null;
@@ -76,10 +76,8 @@ public void end(Request request, Response response, @Nullable Throwable throwabl
7676
@Nullable
7777
@Advice.OnMethodEnter(suppress = Throwable.class, inline = false)
7878
public static AdviceScope onEnter(
79-
@Advice.This Object source,
80-
@Advice.Argument(0) Request request,
81-
@Advice.Argument(1) Response response) {
82-
return AdviceScope.start(source, request, response);
79+
@Advice.Argument(0) Request request, @Advice.Argument(1) Response response) {
80+
return AdviceScope.start(request, response);
8381
}
8482

8583
@Advice.OnMethodExit(onThrowable = Throwable.class, suppress = Throwable.class, inline = false)

0 commit comments

Comments
 (0)