fix OnlineSettings.json not created when non-existing or failed to read

Aurora-RGB · Oct 2, 2024 · 8ab750f · 8ab750f
1 parent 7260619
commit 8ab750f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/Project-Aurora/Project-Aurora/Modules/OnlineConfigs/OnlineConfigsRepository.cs b/Project-Aurora/Project-Aurora/Modules/OnlineConfigs/OnlineConfigsRepository.cs
@@ -67,8 +67,16 @@ public static async Task<OnlineSettingsMeta> GetOnlineSettingsOnline()
     {
         await using var stream = GetJsonStream(cachePath);
 
-        var deserialize = await JsonSerializer.DeserializeAsync<T>(stream, JsonSerializerOptions);
-        return deserialize ?? new T();
+        try
+        {
+            var deserialize = await JsonSerializer.DeserializeAsync<T>(stream, JsonSerializerOptions);
+            return deserialize ?? new T();
+        }
+        catch (Exception e)
+        {
+            Global.logger.Error($"Error parsing local json: {cachePath}", e);
+        }
+        return new T();
     }
 
     private static Stream GetJsonStream(string cachePath)

diff --git a/Project-Aurora/Project-Aurora/Modules/OnlineConfiguration.cs b/Project-Aurora/Project-Aurora/Modules/OnlineConfiguration.cs
@@ -98,7 +98,7 @@ private async Task DownloadAndExtract()
 
         try
         {
-            await ExtractSettings();
+            await DownloadAndExtractRepository();
         }
         catch (Exception e)
         {
@@ -136,7 +136,7 @@ private async Task Refresh()
         }
     }
 
-    private async Task ExtractSettings()
+    private async Task DownloadAndExtractRepository()
     {
         const string zipUrl = "https://github.com/Aurora-RGB/Online-Settings/archive/refs/heads/master.zip";