chore(tracer): log when baggage extract limits are hit

mtoffl01 · claude · mtoffl01 · commit b037ef9301b6 · 2026-05-04T14:33:21.000-04:00
Split the combined limit check so we can emit a distinct warning for
the items vs. bytes case when entries are dropped.

Co-Authored-By: Claude Opus 4.7 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/ddtrace/tracer/textmap.go b/ddtrace/tracer/textmap.go
@@ -1449,7 +1449,12 @@ func (*propagatorBaggage) extractTextMap(reader TextMapReader) (*SpanContext, er
 		if ctr > 0 {
 			itemBytes++ // comma separator
 		}
-		if ctr >= baggageMaxItems || byteCount+itemBytes > baggageMaxBytes {
+		if ctr >= baggageMaxItems {
+			log.Warn("baggage item count exceeded limit (%d), dropping remaining items", baggageMaxItems)
+			break
+		}
+		if byteCount+itemBytes > baggageMaxBytes {
+			log.Warn("baggage byte limit exceeded (%d), dropping remaining items", baggageMaxBytes)
 			break
 		}
 		k, v, ok := strings.Cut(kv, "=")

Original file line number	Diff line number	Diff line change
`@@ -1449,7 +1449,12 @@ func (propagatorBaggage) extractTextMap(reader TextMapReader) (SpanContext, er`
`1449`	`1449`	`if ctr > 0 {`
`1450`	`1450`	`itemBytes++ // comma separator`
`1451`	`1451`	`}`
`1452`		`- if ctr >= baggageMaxItems \|\| byteCount+itemBytes > baggageMaxBytes {`
	`1452`	`+ if ctr >= baggageMaxItems {`
	`1453`	`+ log.Warn("baggage item count exceeded limit (%d), dropping remaining items", baggageMaxItems)`
	`1454`	`+ break`
	`1455`	`+ }`
	`1456`	`+ if byteCount+itemBytes > baggageMaxBytes {`
	`1457`	`+ log.Warn("baggage byte limit exceeded (%d), dropping remaining items", baggageMaxBytes)`
`1453`	`1458`	`break`
`1454`	`1459`	`}`
`1455`	`1460`	`k, v, ok := strings.Cut(kv, "=")`