fix: catch json exceptions for config

Fooxboy · Apr 28, 2024 · 2bf2dab · 2bf2dab
1 parent d55a1af
commit 2bf2dab
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/MusicX/Services/ConfigService.cs b/MusicX/Services/ConfigService.cs
@@ -78,6 +78,10 @@ public async Task<ConfigModel> GetConfig()
                     await using var stream = File.OpenRead(_configPath);
                     config = await JsonSerializer.DeserializeAsync<ConfigModel>(stream, _configSerializerOptions);
                 }
+                catch (JsonException e)
+                {
+                    _logger.Error(e, "Failed to read config");
+                }
                 finally
                 {
                     ConfigSemaphore.Release();
@@ -102,6 +106,10 @@ public async Task SetConfig(ConfigModel config)
                 await using var stream = File.Create(_configPath);
                 await JsonSerializer.SerializeAsync(stream, config, _configSerializerOptions);
             }
+            catch (JsonException e)
+            {
+                _logger.Error(e, "Failed to write config");
+            }
             finally
             {
                 ConfigSemaphore.Release();