add coroutine exception handler

Author: wenxi-zeng

Analytics-CSharp/Analytics-CSharp.csproj

@@ -32,9 +32,9 @@
     <Folder Include="Segment\Analytics\Utilities\" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Coroutine.NET" Version="0.5.0" />
+    <PackageReference Include="Coroutine.NET" Version="1.0.0" />
     <PackageReference Include="Serialization.NET" Version="0.1.0" />
-    <PackageReference Include="Sovran.NET" Version="0.4.0" />
+    <PackageReference Include="Sovran.NET" Version="1.0.0" />
   </ItemGroup>
   <ItemGroup>
     <AssemblyAttribute Include="System.Runtime.CompilerServices.InternalsVisibleTo">

Analytics-CSharp/Segment/Analytics/Analytics.cs

@@ -33,7 +33,7 @@ public partial class Analytics : ISubscriber
         public Analytics(Configuration configuration)
         {
             this.configuration = configuration;
-            analyticsScope = new Scope();
+            analyticsScope = new Scope(configuration.exceptionHandler);
             if (configuration.userSynchronizeDispatcher)
             {
                 IDispatcher dispatcher = new SynchronizeDispatcher();
@@ -48,7 +48,7 @@ public Analytics(Configuration configuration)
                 analyticsDispatcher = new Dispatcher(new LimitedConcurrencyLevelTaskScheduler(Environment.ProcessorCount));
             }
 
-            store = new Store(configuration.userSynchronizeDispatcher);
+            store = new Store(configuration.userSynchronizeDispatcher, configuration.exceptionHandler);
             storage = new Storage(store, configuration.writeKey, configuration.persistentDataPath, fileIODispatcher);
             timeline = new Timeline();
 

Analytics-CSharp/Segment/Analytics/Configuration.cs

@@ -1,4 +1,6 @@
 
+using Segment.Concurrent;
+
 namespace Segment.Analytics
 {
     public class Configuration
@@ -21,6 +23,8 @@ public class Configuration
 
         public bool userSynchronizeDispatcher { get; }
 
+        public ICoroutineExceptionHandler exceptionHandler { get; }
+        
         /// <summary>
         /// Configuration that analytics can use
         /// </summary>
@@ -38,7 +42,8 @@ public class Configuration
         /// <param name="autoAddSegmentDestination">automatically add SegmentDestination plugin, defaults to <c>true</c></param>
         /// <param name="userSynchronizeDispatcher">forcing everything to run synchronously, used for unit tests </param>
         /// <param name="apiHost">set a default apiHost to which Segment sends events, defaults to <c>api.segment.io/v1</c></param>
-        /// <param name="cdnHost">et a default cdnHost to which Segment fetches settings, defaults to <c>cdn-settings.segment.com/v1</c></param>
+        /// <param name="cdnHost">set a default cdnHost to which Segment fetches settings, defaults to <c>cdn-settings.segment.com/v1</c></param>
+        /// <param name="exceptionHandler">set a an exception handler to handle errors happened in async methods within the analytics scope</param>
         public Configuration(string writeKey,
             string persistentDataPath,
             int flushAt = 20,
@@ -47,7 +52,8 @@ public Configuration(string writeKey,
             bool autoAddSegmentDestination = true,
             bool userSynchronizeDispatcher = false,
             string apiHost = null,
-            string cdnHost = null)
+            string cdnHost = null,
+            ICoroutineExceptionHandler exceptionHandler = null)
         {
             this.writeKey = writeKey;
             this.persistentDataPath = persistentDataPath;
@@ -58,6 +64,7 @@ public Configuration(string writeKey,
             this.userSynchronizeDispatcher = userSynchronizeDispatcher;
             this.apiHost = apiHost;
             this.cdnHost = cdnHost;
+            this.exceptionHandler = exceptionHandler;
         }
     }