Skip to content

Commit 3f3a0a0

Browse files
committed
changefeedccl: mark description as safe for log redaction
Previously, logs in maybeLogBehindSpan would mark the description string (i.e., coordinator or aggregator) as redactable. This is not sensitive information, so it should not be redacted. Epic: none Release note: none
1 parent 8a15c13 commit 3f3a0a0

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pkg/ccl/changefeedccl/changefeed_processors.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2052,12 +2052,12 @@ func maybeLogBehindSpan(
20522052

20532053
if frontierChanged && slowLogEveryN.ShouldProcess(now) {
20542054
log.Dev.Infof(ctx, "%s new resolved timestamp %s is behind by %s",
2055-
description, frontierTS, resolvedBehind)
2055+
redact.Safe(description), frontierTS, resolvedBehind)
20562056
}
20572057

20582058
if slowLogEveryN.ShouldProcess(now) {
20592059
s := frontier.PeekFrontierSpan()
2060-
log.Dev.Infof(ctx, "%s span %s is behind by %s", description, s, resolvedBehind)
2060+
log.Dev.Infof(ctx, "%s span %s is behind by %s", redact.Safe(description), s, resolvedBehind)
20612061
}
20622062
}
20632063

0 commit comments

Comments
 (0)