Merge pull request #453 from microsoft/fix/unhandled-crashes-fix

IOS new thread exception crash report fix

Author: MatkovIvan

Assets/AppCenter/Plugins/AppCenterSDK/Crashes/Shared/Crashes.cs

@@ -78,11 +78,15 @@ public static void OnHandleUnresolvedException(object sender, UnhandledException
             if (exception != null)
             {
                 Debug.Log("Unhandled exception: " + exception.ToString());
+#if UNITY_IOS && !UNITY_EDITOR
+                TrackErrorWithAttachments(exception);
+#else
                 lock (_unhandledExceptions)
                 {
                     _unhandledExceptions.Enqueue(exception);
                 }
                 UnityCoroutineHelper.StartCoroutine(SendUnhandledExceptionReports);
+#endif
             }
         }
 #endif
@@ -326,18 +330,25 @@ private static IEnumerator SendUnhandledExceptionReports()
                 }
                 if (exception != null)
                 {
-                    var exceptionWrapper = CreateWrapperException(exception);
-                    var errorId = CrashesInternal.TrackException(exceptionWrapper.GetRawObject(), null, null);
-                    if (_enableErrorAttachmentsCallbacks)
-                    {
-                        SendErrorAttachments(errorId);
-                    }
+                    TrackErrorWithAttachments(exception);
                 }
                 yield return null; // report remaining exceptions on next frames
             }
         }
 #endif
 
+        private static void TrackErrorWithAttachments(Exception exception)
+        {
+            var exceptionWrapper = CreateWrapperException(exception);
+            var errorId = CrashesInternal.TrackException(exceptionWrapper.GetRawObject(), null, null);
+
+            // If the main thread is not crashed, attachments should be sent.
+            if (_enableErrorAttachmentsCallbacks)
+            {
+                SendErrorAttachments(errorId);
+            }
+        }
+
         private static WrapperException CreateWrapperException(Exception exception)
         {
             var exceptionWrapper = new WrapperException();

CHANGELOG.md

@@ -1,5 +1,13 @@
 # App Center SDK for Unity Change Log
 
+## Release 3.2.1 (in development)
+
+### App Center
+
+#### iOS
+
+* **[Fix]** Fix reporting crashes caused by a thread exception.
+
 ## Release 3.2.0
 
 ### App Center