Sitelet https://github.com/getsentry/sentry-elixir/pull/1136
Skip to content

fix: use LoggerUtils for diagnostics - #1136

Merged
solnic merged 7 commits into
masterfrom
fix/scheduler-log-domain
Jul 31, 2026
Merged

solnic merged 7 commits into
masterfrom
fix/scheduler-log-domain

Conversation

@solnic

@solnic solnic commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

For diagnostic logging we must use LoggerUtils because it ensures :sentry domain is used, and that is one of the excluded domains by default in the structured logging feature.

@solnic solnic changed the title fix(scheduler): use LoggerUtils for diagnostics fix: use LoggerUtils for diagnostics Jul 31, 2026
@solnic
solnic force-pushed the fix/scheduler-log-domain branch from 1ee9e43 to c9f20e6 Compare July 31, 2026 10:24
@solnic
solnic marked this pull request as ready for review July 31, 2026 10:28

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c9f20e6. Configure here.

Comment thread lib/sentry/telemetry/scheduler.ex Outdated
Comment thread lib/sentry/opentelemetry/span_processor.ex Outdated
Comment thread lib/sentry/telemetry/scheduler.ex
@solnic
solnic force-pushed the fix/scheduler-log-domain branch from c9f20e6 to e8c1ca6 Compare July 31, 2026 11:36
@solnic
solnic force-pushed the fix/scheduler-log-domain branch from e8c1ca6 to 3a4fa22 Compare July 31, 2026 12:20
@solnic
solnic merged commit 6b0558e into master Jul 31, 2026
16 checks passed
@solnic
solnic deleted the fix/scheduler-log-domain branch July 31, 2026 12:34
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.

2 participants