fix: Android - make sure logcat.log is attached to unhandled event (SIGSEGV Segfault)

CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## Unreleased
+
+### Fixes
+
+- Android: Fix for SIGSEGV reported by Sentry when a NullReferenceException is thrown in Release mode ([#3694](https://github.com/getsentry/sentry-dotnet/pull/3694))
+  - To turn it on, set: `options.HandlerStrategy = NdkHandlerStrategy.SENTRY_HANDLER_STRATEGY_CHAIN_AT_START;`
+
 ## 4.12.1
 
 ### Fixes

src/Sentry/Platforms/Android/NdkHandlerStrategy.cs

@@ -0,0 +1,30 @@
+namespace Sentry
+{
+    /// <summary>
+    /// Handler strategy
+    /// </summary>
+    ///
+    public enum NdkHandlerStrategy
+    {
+        /// <summary>
+        /// default handler strategy -> value 0
+        /// </summary>
+        SENTRY_HANDLER_STRATEGY_DEFAULT,
+        /// <summary>
+        /// Handle strategy chain at start -> value 1
+        /// </summary>
+        SENTRY_HANDLER_STRATEGY_CHAIN_AT_START,
+
+
+    }
+    /// <summary>
+    /// Extension class of strategy containing relevant its actions
+    /// </summary>
+    public static class  HandlerStrategyExtension
+    {
+        internal static JavaSdk.Android.Core.NdkHandlerStrategy ToJava(this NdkHandlerStrategy strategy)
+        {
+            return JavaSdk.Android.Core.NdkHandlerStrategy.Values()![(int)strategy];
+        }
+    }
+}

src/Sentry/Platforms/Android/SentrySdk.cs

@@ -59,6 +59,9 @@ private static void InitSentryAndroidSdk(SentryOptions options)
 
         // Define the configuration for the Android SDK
         SentryAndroidOptions? nativeOptions = null;
+
+        var handlerStrategy = options.HandlerStrategy.ToJava();
+
         var configuration = new OptionsConfigurationCallback(o =>
         {
             // Capture the android options reference on the outer scope
@@ -87,6 +90,7 @@ private static void InitSentryAndroidSdk(SentryOptions options)
             o.ServerName = options.ServerName;
             o.SessionTrackingIntervalMillis = (long)options.AutoSessionTrackingInterval.TotalMilliseconds;
             o.ShutdownTimeoutMillis = (long)options.ShutdownTimeout.TotalMilliseconds;
+            o.SetNativeHandlerStrategy(handlerStrategy);
 
             if (options.CacheDirectoryPath is { } cacheDirectoryPath)
             {

src/Sentry/SentryOptions.cs

@@ -77,6 +77,20 @@ public bool IsGlobalModeEnabled
         set => _isGlobalModeEnabled = value;
     }
 #endif
+    /// <summary>
+    /// Strategy used by the handler
+    /// </summary>
+    /// <code>
+    /// SentrySdk.Init(options =>
+    /// {
+    /// Sentry
+    /// // ...
+    /// options.HandlerStrategy = NdkHandlerStrategy.SENTRY_HANDLER_STRATEGY_DEFAULT;
+    /// // ... other options
+    /// }
+    /// </code>
+    /// <seealso href="https://github.com/dotnet/android/issues/9055#issuecomment-2261347912"/>
+    public NdkHandlerStrategy HandlerStrategy { get; set; } = NdkHandlerStrategy.SENTRY_HANDLER_STRATEGY_DEFAULT;
 
     /// <summary>
     /// A scope set outside of Sentry SDK. If set, the global parameters from the SDK's scope will be sent to the observed scope.<br/>

test/Sentry.Maui.Tests/SentryMauiOptionsTests.cs

@@ -56,4 +56,31 @@ public void AttachScreenshots_Default()
         var options = new SentryMauiOptions();
         Assert.False(options.AttachScreenshot);
     }
+
+    [Fact]
+    public void HandlerStrategy_Default()
+    {
+        // Arrange
+        var options = new SentryMauiOptions();
+        var expected = NdkHandlerStrategy.SENTRY_HANDLER_STRATEGY_DEFAULT;
+
+        // Assert
+        // testing default state here so nothing to assert
+
+        // Act
+        Assert.Equal(expected, options.HandlerStrategy);
+    }
+    [Fact]
+    public void HandlerStrategy_set()
+    {
+        // Arrange
+        var options = new SentryMauiOptions();
+        var notExpected = NdkHandlerStrategy.SENTRY_HANDLER_STRATEGY_DEFAULT;
+
+        // Assert
+        options.HandlerStrategy = NdkHandlerStrategy.SENTRY_HANDLER_STRATEGY_CHAIN_AT_START;
+
+        // Act
+        Assert.NotEqual(notExpected, options.HandlerStrategy);
+    }
 }