Manually handle sending of crashes

TinyInsights/ApplicationInsightsProvider.cs

@@ -11,7 +11,7 @@
 public class ApplicationInsightsProvider : IInsightsProvider, ILogger
 {
     public string? ConnectionString { get; set; }
     private static ApplicationInsightsProvider? provider;
     private const string userIdKey = nameof(userIdKey);
 
     private const string crashLogFilename = "crashes.mauiinsights";
@@ -23,6 +23,7 @@
 
     public bool IsTrackErrorsEnabled { get; set; } = true;
     public bool IsTrackCrashesEnabled { get; set; } = true;
+    public bool WriteCrashes { get; set; } = true;
     public bool IsTrackPageViewsEnabled { get; set; } = true;
     public bool IsAutoTrackPageViewsEnabled { get; set; } = true;
     public bool IsTrackEventsEnabled { get; set; } = true;
@@ -106,7 +107,10 @@
             Application.Current.PageDisappearing += weakOnDisappearingHandler.Handler;
         }
 
-        Task.Run(SendCrashes);
+        if (WriteCrashes)
+        {
+            Task.Run(SendCrashes);
+        }
 
         IsInitialized = true;
     }
@@ -163,7 +167,36 @@
             // Add any global properties, the user has already added
             foreach (var property in _globalProperties)
             {
-                client.Context.GlobalProperties[property.Key] = property.Value;
+                switch(property.Key)
+                {
+                    case "Cloud.RoleName":
+                        client.Context.Cloud.RoleName = property.Value;
+                        break;
+
+                    case "Cloud.RoleInstance":
+                        client.Context.Cloud.RoleInstance = property.Value;
+                        break;
+
+                    case "Device.OperatingSystem":
+                        client.Context.Device.OperatingSystem = property.Value;
+                        break;
+
+                    case "Device.Model":
+                        client.Context.Device.Model = property.Value;
+                        break;
+
+                    case "Device.Type":
+                        client.Context.Device.Type = property.Value;
+                        break;
+
+                    case "Device.Id":
+                        client.Context.Device.Id = property.Value;
+                        break;
+
+                    default:
+                        client.Context.GlobalProperties[property.Key] = property.Value;
+                        break;
+                }
             }
 
             client.Context.GlobalProperties.TryAdd("Language", CultureInfo.CurrentUICulture.TwoLetterISOLanguageName);
@@ -173,7 +206,6 @@
             client.Context.GlobalProperties.TryAdd("OperatingSystemVersion", DeviceInfo.VersionString);
             client.Context.Session.Id = Guid.NewGuid().ToString();
 
-
             return client;
         }
         catch (Exception)
@@ -187,7 +219,9 @@
 
     public void UpsertGlobalProperty(string key, string value)
     {
-        if (Client is null)
+        _globalProperties[key] = value;
+
+        if(Client is null)
         {
             return;
         }
@@ -196,29 +230,32 @@
         {
             case "Cloud.RoleName":
                 Client.Context.Cloud.RoleName = value;
-                return;
+                break;
+
             case "Cloud.RoleInstance":
                 Client.Context.Cloud.RoleInstance = value;
-                return;
+                break;
 
             case "Device.OperatingSystem":
                 Client.Context.Device.OperatingSystem = value;
-                return;
+                break;
+
             case "Device.Model":
                 Client.Context.Device.Model = value;
-                return;
+                break;
+
             case "Device.Type":
                 Client.Context.Device.Type = value;
-                return;
+                break;
+
             case "Device.Id":
                 Client.Context.Device.Id = value;
-                return;
-        }
-
-        _globalProperties[key] = value;
-
-        Client.Context.GlobalProperties[key] = value;
+                break;
 
+            default:
+                Client.Context.GlobalProperties[key] = value;
+                break;
+        }
     }
 
     public void RemoveGlobalProperty(string key)
@@ -273,7 +310,7 @@
         Client.Context.Session.Id = Guid.NewGuid().ToString();
     }
 
-    private async Task SendCrashes()
+    public async Task SendCrashes()
     {
         try
         {
@@ -321,6 +358,29 @@
         }
     }
 
+    public bool HasCrashed()
+    {
+        try
+        {
+            var path = Path.Combine(logPath, crashLogFilename);
+
+            if(!File.Exists(path))
+            {
+                return false;
+            }
+
+            var json = File.ReadAllText(path);
+
+            var crashes = string.IsNullOrWhiteSpace(json) ? null : JsonSerializer.Deserialize<List<Crash>>(json);
+
+            return crashes is null ? false : crashes.Count != 0;
+        }
+        catch(Exception)
+        {
+            return false;
+        }
+    }
+
     private List<Crash>? ReadCrashes()
     {
         try
@@ -346,7 +406,7 @@
         return null;
     }
 
-    private void ResetCrashes()
+    public void ResetCrashes()
     {
         try
         {
@@ -447,6 +507,11 @@
                 return;
             }
 
+            if (!IsTrackPageViewsEnabled)
+            {
+                return;
+            }
+
             if (EnableConsoleLogging)
                 Console.WriteLine($"TinyInsights: tracking page view {viewName}");
 

TinyInsights/IInsights.cs

@@ -3,16 +3,19 @@ namespace TinyInsights;
 public interface IInsights
 {
     void AddProvider(IInsightsProvider provider);
+
     IReadOnlyList<IInsightsProvider> GetProviders();
 
     void UpsertGlobalProperty(string key, string value);
+
     void RemoveGlobalProperty(string key);
 
     Task TrackErrorAsync(Exception ex, Dictionary<string, string>? properties = null);
 
     Task TrackPageViewAsync(string viewName, Dictionary<string, string>? properties = null, TimeSpan? duration = null);
 
     Task TrackEventAsync(string eventName, Dictionary<string, string>? properties = null);
+
     Task TrackErrorAsync(Exception ex, ErrorSeverity severity, Dictionary<string, string>? properties = null);
 
     Task TrackDependencyAsync(string dependencyType, string dependencyName, string data, DateTimeOffset startTime, TimeSpan duration, bool success, int resultCode = 0, Exception? exception = null);
@@ -25,5 +28,12 @@ Task TrackDependencyAsync(string dependencyType, string dependencyName, string d
     void OverrideAnonymousUserId(string userId);
 
     void GenerateNewAnonymousUserId();
+
     void CreateNewSession();
+
+    bool HasCrashed();
+
+    Task SendCrashes();
+
+    void ResetCrashes();
 }

TinyInsights/IInsightsProvider.cs

@@ -3,7 +3,8 @@ namespace TinyInsights;
 public interface IInsightsProvider
 {
     bool IsTrackErrorsEnabled { get; set; }
-    public bool IsTrackCrashesEnabled { get; set; }
+    bool IsTrackCrashesEnabled { get; set; }
+    bool WriteCrashes { get; set; }
     bool IsTrackPageViewsEnabled { get; set; }
     bool IsAutoTrackPageViewsEnabled { get; set; }
     bool IsTrackEventsEnabled { get; set; }
@@ -14,6 +15,7 @@ public interface IInsightsProvider
     void Initialize();
 
     void UpsertGlobalProperty(string key, string value);
+
     void RemoveGlobalProperty(string key);
 
     Task TrackErrorAsync(Exception ex, Dictionary<string, string>? properties = null);
@@ -29,5 +31,12 @@ public interface IInsightsProvider
     void OverrideAnonymousUserId(string userId);
 
     string GenerateNewAnonymousUserId();
+
     void CreateNewSession();
+
+    bool HasCrashed();
+
+    Task SendCrashes();
+
+    void ResetCrashes();
 }

TinyInsights/Insights.cs

@@ -157,4 +157,34 @@ public void CreateNewSession()
             provider.CreateNewSession();
         }
     }
+
+    public bool HasCrashed()
+    {
+        foreach(var provider in insightsProviders)
+        {
+            bool hasCrashed = provider.HasCrashed();
+            if(hasCrashed)
+            {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
+    public async Task SendCrashes()
+    {
+        foreach(var provider in insightsProviders)
+        {
+            await provider.SendCrashes();
+        }
+    }
+
+    public void ResetCrashes()
+    {
+        foreach(var provider in insightsProviders)
+        {
+            provider.ResetCrashes();
+        }
+    }
 }