Handle dynamic screenshots exceptions (Closes #1204) (#1206)

Author: AlexMacocian

Daybreak/Services/Screenshots/ScreenshotService.cs

@@ -1,14 +1,18 @@
 using System.Collections.Concurrent;
 using System.Drawing;
+using System.Extensions.Core;
 using System.IO;
 using Daybreak.Shared.Models;
 using Daybreak.Shared.Models.ColorPalette;
 using Daybreak.Shared.Services.Screenshots;
+using Microsoft.Extensions.Logging;
 using static Daybreak.Shared.Models.Themes.Theme;
 
 namespace Daybreak.Services.Screenshots;
 
-public sealed class ScreenshotService : IScreenshotService
+public sealed class ScreenshotService(
+    ILogger<ScreenshotService> logger)
+    : IScreenshotService
 {
     private const string GuildWarsFolder = "Guild Wars";
     private const string ScreensFolder = "Screens";
@@ -17,6 +21,8 @@ public sealed class ScreenshotService : IScreenshotService
     private static readonly ColorThiefDotNet.ColorThief ColorThief = new();
     private static readonly ConcurrentDictionary<string, ScreenshotEntry> EntryCache = [];
 
+    private readonly ILogger<ScreenshotService> logger = logger;
+
     public Task<ScreenshotEntry?> GetRandomScreenshot(CancellationToken cancellationToken)
     {
         return Task.Factory.StartNew(this.GetRandomScreenshotInternal, cancellationToken, TaskCreationOptions.LongRunning, TaskScheduler.Current).Unwrap();
@@ -41,27 +47,38 @@ public sealed class ScreenshotService : IScreenshotService
             return default;
         }
 
-        return GetEntryFromPath(selectedScreenshot);
+        return this.GetEntryFromPath(selectedScreenshot);
     }
 
-    private static ScreenshotEntry? GetEntryFromPath(string path)
+    private ScreenshotEntry? GetEntryFromPath(string path)
     {
+        var scopedLogger = this.logger.CreateScopedLogger();
         if (EntryCache.TryGetValue(path, out var cachedEntry))
         {
+            scopedLogger.LogDebug("Screenshot entry found in cache for path {path}", path);
             return cachedEntry;
         }
 
-        using var bitmap = GetBitmap(path);
-        if (bitmap is null)
+        try
         {
+            using var bitmap = GetBitmap(path);
+            if (bitmap is null)
+            {
+                return default;
+            }
+
+            var dominantColor = ColorThief.GetColor(bitmap, quality: 2);
+            var closestAccent = GetClosestAccentColor(Color.FromArgb(dominantColor.Color.A, dominantColor.Color.R, dominantColor.Color.G, dominantColor.Color.B));
+            var entry = new ScreenshotEntry(path, closestAccent, dominantColor.IsDark ? LightDarkMode.Dark : LightDarkMode.Light);
+            EntryCache[path] = entry;
+            scopedLogger.LogDebug("Screenshot entry created and cached for path {path}", path);
+            return entry;
+        }
+        catch(Exception e)
+        {
+            scopedLogger.LogError(e, "Failed to get screenshot entry from path {path}", path);
             return default;
         }
-
-        var dominantColor = ColorThief.GetColor(bitmap, quality: 2);
-        var closestAccent = GetClosestAccentColor(Color.FromArgb(dominantColor.Color.A, dominantColor.Color.R, dominantColor.Color.G, dominantColor.Color.B));
-        var entry = new ScreenshotEntry(path, closestAccent, dominantColor.IsDark ? LightDarkMode.Dark : LightDarkMode.Light);
-        EntryCache[path] = entry;
-        return entry;
     }
 
     private static AccentColor GetClosestAccentColor(Color color)