Change default log level to warning

[philipphofmann]

Description

The default SDK log level is debug. When you turn on option.debug = true the SDK logs too much information, so warnings and errors get lost. Instead the SDK should log only warnings and above when option.debug = true. Users can manually set the diagnosticLevel to debug if required. When we detect a debug build, we turn on debug = true, so you get only the warnings and errors. We still need to discuss if we're going to handle this as a breaking change or not, and we should create a meta issue for this to align across all SDKs.

This is also required to surface severe SDK misconfiguration or other errors clearly to the user such as: #4566

🛑 Blocked by

Preview Give feedback

1. feat(develop-docs): SDK Debug Mode sentry-docs#12100
   +0
   

[kahest]

related: getsentry/sentry-native#888

[philipphofmann]

@philprime, we still should change the default log level to warning. #4707 only changes always logging fatal message. I would like to keep this open.

[armcknight]

I just saw that PR, i'm not sure I'd overload a level to essentially indicate "ignore levels", and instead would have added another config parameter like SentryLog.configure(isDebug: false, diagnosticLevel: .whatever, logAllLevels: true). Did you discuss an option like this?

[philprime]

@armcknight not sure if I understand your proposition. In my PR the logAlwaysLevel is defined as a constant and can not be configured.