Don't set MarshalManagedExceptionMode.UnwindNativeCode for CoreCLR

Author: jamescrosswell

src/Sentry.Bindings.Cocoa/buildTransitive/Sentry.Bindings.Cocoa.targets

@@ -49,4 +49,17 @@
     </ItemGroup>
   </Target>
 
+  <!--
+   Ensure the AppDomain.CurrentDomain.UnhandledException gets triggered - see:
+    https://github.com/xamarin/xamarin-macios/issues/15252
+   Not supported on the CoreCLR - see:
+    https://github.com/xamarin/xamarin-macios/blob/0a81da28e4217624a54ccb83541e1eb8ba651fca/runtime/runtime.m#L2480-L2491
+
+   The condition is a bit hacky - it relies on:
+    https://github.com/xamarin/xamarin-macios/blob/0ab074390689880fe9269804a0156341d30251f6/dotnet/targets/Xamarin.Shared.Sdk.targets#L1004-L1006
+  -->
+  <PropertyGroup Condition="'$(_XamarinRuntime)' != 'CoreCLR'">
+    <MtouchExtraArgs>--marshal-managed-exceptions=unwindnativecode</MtouchExtraArgs>
+  </PropertyGroup>
+
 </Project>

src/Sentry/Platforms/Cocoa/SentrySdk.cs

@@ -1,6 +1,8 @@
 using Sentry.Cocoa;
 using Sentry.Cocoa.Extensions;
 using Sentry.Extensibility;
+using Sentry.Internal;
+using Sentry.PlatformAbstractions;
 
 // ReSharper disable once CheckNamespace
 namespace Sentry;
@@ -10,10 +12,10 @@ public static partial class SentrySdk
     private static void InitSentryCocoaSdk(SentryOptions options)
     {
         options.LogDebug("Initializing native SDK");
-        // Workaround for https://github.com/xamarin/xamarin-macios/issues/15252
         ObjCRuntime.Runtime.MarshalManagedException += (_, args) =>
         {
-            args.ExceptionMode = ObjCRuntime.MarshalManagedExceptionMode.UnwindNativeCode;
+            // This MAY be overriden in Sentry.Bindings.Cocoa.targets
+            options.LogDebug($"MarshalManagedExceptionMode: {args.ExceptionMode}");
         };
 
         // Set default release and distribution