fix: forward enable logs/metrics options to native layer#2662
fix: forward enable logs/metrics options to native layer#2662bitsandfoxes merged 3 commits intomainfrom
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 14683c4. Configure here.
| #endif | ||
|
|
||
| Logger?.LogDebug("Setting EnableLogs: {0}", options.EnableLogs); | ||
| sentry_options_set_enable_logs(cOptions, options.EnableLogs ? 1 : 0); |
There was a problem hiding this comment.
Missing native stub for sentry_options_set_enable_logs
High Severity
The new sentry_options_set_enable_logs P/Invoke in SentryNativeBridge.cs is called unconditionally but lacks a corresponding stub in sentry_native_stubs.c. On Switch builds using the stub library, this causes an EntryPointNotFoundException during initialization, which disables all native Sentry features like scope synchronization and crash reporting.
Reviewed by Cursor Bugbot for commit 14683c4. Configure here.


Follow up on #2660 (and probably other native bumps)
The enable logs and metrics options need to be properly propagated to the native layers.