Skip to content

Commit a5adf0e

Browse files
committed
not sure
1 parent ebcd198 commit a5adf0e

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

dd-java-agent/appsec/src/main/java/com/datadog/appsec/gateway/GatewayBridge.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -839,7 +839,8 @@ private NoopFlow onRequestEnded(RequestContext ctx_, IGSpanInfo spanInfo) {
839839
if (maybeSampleForApiSecurity(ctx, spanInfo, tags)) {
840840
if (!Config.get().isApmTracingEnabled()) {
841841
traceSeg.setTagTop(Tags.ASM_KEEP, true);
842-
traceSeg.setTagTop(Tags.PROPAGATED_TRACE_SOURCE, ProductTraceSource.ASM);
842+
// Note: _dd.p.ts (PROPAGATED_TRACE_SOURCE) is only set when there are actual AppSec events
843+
// (see lines below where collectedEvents is checked), not just for API Security sampling
843844
}
844845
} else {
845846
ctx.closeWafContext();

dd-java-agent/appsec/src/test/groovy/com/datadog/appsec/gateway/GatewayBridgeSpecification.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1315,7 +1315,7 @@ class GatewayBridgeSpecification extends DDSpecification {
13151315
1 * spanInfo.getTags() >> TagMap.fromMap(['http.route': 'route'])
13161316
1 * requestSampler.preSampleRequest(_) >> true
13171317
1 * traceSegment.setTagTop(Tags.ASM_KEEP, true)
1318-
1 * traceSegment.setTagTop(Tags.PROPAGATED_TRACE_SOURCE, ProductTraceSource.ASM)
1318+
0 * traceSegment.setTagTop(Tags.PROPAGATED_TRACE_SOURCE, ProductTraceSource.ASM)
13191319
}
13201320
13211321

0 commit comments

Comments
 (0)