Skip to content

add logger.UnredactedProto helper#1578

Merged
paulwe merged 1 commit into
mainfrom
paulwe/unredacted-proto-logger
May 22, 2026
Merged

add logger.UnredactedProto helper#1578
paulwe merged 1 commit into
mainfrom
paulwe/unredacted-proto-logger

Conversation

@paulwe
Copy link
Copy Markdown
Contributor

@paulwe paulwe commented May 22, 2026

Summary

  • Adds logger.UnredactedProto, a sibling to logger.Proto that ignores the (logger.redact) field option and emits the real field values.
  • Intended for log sinks that are not part of system logs (e.g. project-scoped observability event logs that customers consume). System logs still use logger.Proto and remain redacted.
  • Threads a redact flag through protoMarshaller / protoMapMarshaller / protoListMarshaller / marshalProtoField so the behavior propagates into nested messages, lists, and maps.

Test plan

  • go build ./logger/...
  • go test ./logger/...
  • Downstream consumers (cloud, cloud-io) bumped to a version containing this symbol and swap their observability-reporter event-log call sites over.

Provides a sibling to logger.Proto that ignores the (logger.redact) field
option. Intended for project-scoped observability event logs where the
real field values are part of the event payload customers consume.
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 22, 2026

🦋 Changeset detected

Latest commit: 309f8bc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@livekit/protocol Patch
github.com/livekit/protocol Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@paulwe paulwe merged commit e8f795f into main May 22, 2026
10 checks passed
@paulwe paulwe deleted the paulwe/unredacted-proto-logger branch May 22, 2026 22:54
@github-actions github-actions Bot mentioned this pull request May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants