Skip to content

Commit 42953b1

Browse files
committed
fix(appsec): set t before effectivelyBlocked() in Tomcat filenames and content blocking paths
1 parent be80399 commit 42953b1

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • dd-java-agent/instrumentation/tomcat/tomcat-appsec/tomcat-appsec-7.0/src/main/java/datadog/trace/instrumentation/tomcat7

dd-java-agent/instrumentation/tomcat/tomcat-appsec/tomcat-appsec-7.0/src/main/java/datadog/trace/instrumentation/tomcat7/ParsePartsInstrumentation.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,8 +142,8 @@ static void after(
142142
BlockResponseFunction brf = reqCtx.getBlockResponseFunction();
143143
if (brf != null) {
144144
brf.tryCommitBlockingResponse(reqCtx.getTraceSegment(), rba);
145-
reqCtx.getTraceSegment().effectivelyBlocked();
146145
t = new BlockingException("Blocked request (multipart file upload)");
146+
reqCtx.getTraceSegment().effectivelyBlocked();
147147
}
148148
}
149149
}
@@ -163,8 +163,8 @@ static void after(
163163
BlockResponseFunction brf = reqCtx.getBlockResponseFunction();
164164
if (brf != null) {
165165
brf.tryCommitBlockingResponse(reqCtx.getTraceSegment(), rba);
166-
reqCtx.getTraceSegment().effectivelyBlocked();
167166
t = new BlockingException("Blocked request (multipart file upload content)");
167+
reqCtx.getTraceSegment().effectivelyBlocked();
168168
}
169169
}
170170
}

0 commit comments

Comments
 (0)