Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set options.debug = true if we can reliably detect that the app runs in a debug build #4463

Open
Tracked by #201
krystofwoldrich opened this issue Jan 17, 2025 · 1 comment

Comments

@krystofwoldrich
Copy link
Member

krystofwoldrich commented Jan 17, 2025

Sentry RN has quite verbose logs, we should only consider this together with changing the default log level from debug to error or something similar not as verbose as debug.

We also need to consider the way Sentry JS uses log levels, as a significant amount of logs is coming from it.

The flag should be propagated to the native SDKs.

@lucas-zimerman
Copy link
Collaborator

lucas-zimerman commented Feb 12, 2025

@krystofwoldrich Detecting when we are running on a debug build surely is a todo task.
Diagnostic Level is missing from Sentry JavaScript for filtering the logs, but it is available on the Native SDKs.
We can set it to error on the native SDKs but it will have no impact on the JS SDK.

Do you think this is enough for this task? For the JS Logs, it requires changes to the JavaScript SDK. I can make the change but it mostly likely will only be merged on V9.

@lucas-zimerman lucas-zimerman moved this from In Progress to Todo in Mobile & Cross Platform SDK Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

No branches or pull requests

2 participants