Skip to content

Conversation

limbonaut
Copy link
Collaborator

@limbonaut limbonaut commented Aug 7, 2025

This PR adds option logger/messages_as_breadcrumbs which is enabled by default. This allows disabling messages as breadcrumbs by a simple toggle. Since some projects print() heavily into console, this option should be quite useful.

Screenshot 2025-08-07 at 19 15 49

@limbonaut limbonaut changed the title feat: Add option to disable loggin messages as breadcrumbs feat: Add option to disable logging messages as breadcrumbs Aug 7, 2025
Copy link
Contributor

github-actions bot commented Aug 7, 2025

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 99906fd

@jpnurmi
Copy link
Collaborator

jpnurmi commented Aug 8, 2025

The Native SDK's Qt integration does something similar and forwards all Qt logging output to Sentry as breadcrumbs. Personally, I find it quite noisy and annoying, which makes me wonder if the feature would be better off as opt-in... How do other SDKs handle this? 🤔

@mujacica
Copy link

mujacica commented Aug 8, 2025

@jpnurmi I think this is going to change with the upcoming Sentry Logs and logger changes. That should give us a possibility to configure these things in much more detail.

@limbonaut
Copy link
Collaborator Author

This only affects projects that add a lot of print() statements in code. By default, this adds close to zero noise. Since it's project-dependent how much noise it adds, I think leaving it ON by default is good for discoverability.

@limbonaut limbonaut merged commit ea8d8db into main Aug 8, 2025
85 of 86 checks passed
@limbonaut limbonaut deleted the feat/option-logger-messages-as-breadcrumbs branch August 8, 2025 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add option to disable logging messages as breadcrumbs
3 participants