Skip to content

Scrub message-less structlog records in Sentry - #458

Merged
lesnik512 merged 2 commits into
mainfrom
fix/sentry-missing-log-message
Nov 10, 2026
Merged

lesnik512 merged 2 commits into
mainfrom
fix/sentry-missing-log-message

Conversation

@lesnik512

Copy link
Copy Markdown
Member

Summary

  • Scrub message-less structlog records rendered by lite-bootstrap before sending them as Sentry issues or breadcrumbs. Use [No message] as the Sentry message.
  • Keep the original stdout JSON. Independently configured JSON renderers do not carry the record marker, so their message-less records remain outside this fix.

Refs #450.

Verification

  • Confirmed the new envelope test fails on the password leak before the fix.
  • just test-ci passed with 666 tests and 100% line coverage.
  • Type and lint checks passed. The envelope regression also passed with sentry-sdk 2.70.0 and structlog 21.3.0.

@lesnik512
lesnik512 merged commit a1e065b into main Nov 10, 2026
35 checks passed
@lesnik512
lesnik512 deleted the fix/sentry-missing-log-message branch November 10, 2026 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant