Code Quality: Added a logger to send exception reports to Sentry (#15618)

Author: hishitetsu

src/Files.App/Helpers/Application/AppLifecycleHelper.cs

@@ -5,6 +5,7 @@
 using Files.App.Helpers.Application;
 using Files.App.Services.SizeProvider;
 using Files.App.Storage.Storables;
+using Files.App.Utils.Logger;
 using Files.App.ViewModels.Settings;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
@@ -138,6 +139,7 @@ public static IHost ConfigureHost()
 				.UseEnvironment(AppLifecycleHelper.AppEnvironment.ToString())
 				.ConfigureLogging(builder => builder
 					.AddProvider(new FileLoggerProvider(Path.Combine(ApplicationData.Current.LocalFolder.Path, "debug.log")))
+					.AddProvider(new SentryLoggerProvider())
 					.SetMinimumLevel(LogLevel.Information))
 				.ConfigureServices(services => services
 					// Settings services
@@ -265,14 +267,6 @@ public static void HandleAppUnhandledException(Exception? ex, bool showToastNoti
 				ex.Data[Mechanism.HandledKey] = false;
 				ex.Data[Mechanism.MechanismKey] = "Application.UnhandledException";
 
-				SentrySdk.CaptureException(ex, scope =>
-				{
-					scope.User.Id = generalSettingsService.UserId;
-				});
-
-
-				SentrySdk.CaptureException(ex);
-
 				formattedException.AppendLine($">>>> HRESULT: {ex.HResult}");
 
 				if (ex.Message is not null)

src/Files.App/Utils/Logger/SentryLogger.cs

@@ -0,0 +1,46 @@
+// Copyright (c) 2024 Files Community
+// Licensed under the MIT License. See the LICENSE.
+
+using Microsoft.Extensions.Logging;
+using Sentry;
+
+namespace Files.App.Utils.Logger
+{
+	public sealed class SentryLogger : ILogger
+	{
+
+		public IDisposable? BeginScope<TState>(TState state) where TState : notnull
+		{
+			return null;
+		}
+
+		public bool IsEnabled(LogLevel logLevel)
+		{
+			return true;
+		}
+
+		public void Log<TState>(LogLevel logLevel, EventId eventId, TState state, Exception? exception, Func<TState, Exception?, string> formatter)
+		{
+			if (exception is null)
+				return;
+
+			var generalSettingsService = Ioc.Default.GetRequiredService<IGeneralSettingsService>();
+
+			var level = logLevel switch
+			{
+				LogLevel.Debug => SentryLevel.Debug,
+				LogLevel.Information => SentryLevel.Info,
+				LogLevel.Warning => SentryLevel.Warning,
+				LogLevel.Error => SentryLevel.Error,
+				LogLevel.Critical => SentryLevel.Fatal,
+				_ => SentryLevel.Debug
+			};
+
+			SentrySdk.CaptureException(exception, scope =>
+			{
+				scope.User.Id = generalSettingsService?.UserId;
+				scope.Level = level;
+			});
+		}
+	}
+}

src/Files.App/Utils/Logger/SentryLoggerProvider.cs

@@ -0,0 +1,19 @@
+// Copyright (c) 2024 Files Community
+// Licensed under the MIT License. See the LICENSE.
+
+using Microsoft.Extensions.Logging;
+
+namespace Files.App.Utils.Logger
+{
+	public sealed class SentryLoggerProvider : ILoggerProvider
+	{
+		public ILogger CreateLogger(string categoryName)
+		{
+			return new SentryLogger();
+		}
+
+		public void Dispose()
+		{
+		}
+	}
+}

src/Files.App/ViewModels/UserControls/InfoPaneViewModel.cs

@@ -153,12 +153,6 @@ private async void ContentPageContext_PropertyChanged(object? sender, PropertyCh
 						// Handle exception in case WinUI Windows is closed
 						// (see https://github.com/files-community/Files/issues/15599)
 
-						SentrySdk.CaptureException(ex, scope =>
-						{
-							scope.User.Id = generalSettingsService.UserId;
-							scope.Level = SentryLevel.Warning;
-						});
-
 						App.Logger.LogWarning(ex, ex.Message);
 					}
 

src/Files.App/Views/MainPage.xaml.cs

@@ -462,12 +462,6 @@ private void LoadPaneChanged()
 				// Handle exception in case WinUI Windows is closed
 				// (see https://github.com/files-community/Files/issues/15599)
 
-				SentrySdk.CaptureException(ex, scope =>
-				{
-					scope.User.Id = generalSettingsService.UserId;
-					scope.Level = SentryLevel.Warning;
-				});
-
 				App.Logger.LogWarning(ex, ex.Message);
 			}
 		}