Skip to content

Commit 7127442

Browse files
committed
fix logs
1 parent 7d9ad68 commit 7127442

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

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

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -856,6 +856,9 @@ private NoopFlow onRequestEnded(RequestContext ctx_, IGSpanInfo spanInfo) {
856856
log.info(
857857
"[APPSEC-57815] Setting ASM_KEEP=true (API Security sampled, APM tracing disabled)");
858858
traceSeg.setTagTop(Tags.ASM_KEEP, true);
859+
// Verify the tag was set and check sampling priority
860+
Object asmKeepAfterSet = traceSeg.getTagTop(Tags.ASM_KEEP);
861+
log.info("[APPSEC-57815] ASM_KEEP after setTagTop: {}", asmKeepAfterSet);
859862
// Note: _dd.p.ts (PROPAGATED_TRACE_SOURCE) is only set when there are actual AppSec events
860863
// (see lines below where collectedEvents is checked), not just for API Security sampling
861864
}
@@ -952,11 +955,12 @@ private NoopFlow onRequestEnded(RequestContext ctx_, IGSpanInfo spanInfo) {
952955
);
953956
}
954957

955-
// Log final state of propagation tags
956-
Object finalPropagatedTs = tags.get(Tags.PROPAGATED_TRACE_SOURCE);
957-
Object finalAsmKeep = tags.get(Tags.ASM_KEEP);
958+
// Log final state of propagation tags from TraceSegment (not from spanInfo.getTags() which is
959+
// immutable)
960+
Object finalPropagatedTs = traceSeg.getTagTop(Tags.PROPAGATED_TRACE_SOURCE);
961+
Object finalAsmKeep = traceSeg.getTagTop(Tags.ASM_KEEP);
958962
log.info(
959-
"[APPSEC-57815] Request ended - final state: _dd.p.ts={}, _dd.appsec.keep={}",
963+
"[APPSEC-57815] Request ended - final state from TraceSegment: _dd.p.ts={}, _dd.appsec.keep={}",
960964
finalPropagatedTs,
961965
finalAsmKeep);
962966

0 commit comments

Comments
 (0)