Do not crash the API if config fails to save

BattleBit-Community-Servers · Oct 2, 2023 · 9175734 · 9175734
1 parent d8c9086
commit 9175734
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/BattleBitAPIRunner/Program.cs b/BattleBitAPIRunner/Program.cs
@@ -547,10 +547,17 @@ private void ModuleConfiguration_OnSavingRequest(object? sender, BattleBitModule
                 return; // nothing to save
             }
 
-            File.WriteAllText(filePath, JsonSerializer.Serialize(configurationValue, configurationValue.GetType(), new JsonSerializerOptions()
+            try
             {
-                WriteIndented = true
-            }));
+                File.WriteAllText(filePath, JsonSerializer.Serialize(configurationValue, configurationValue.GetType(), new JsonSerializerOptions()
+                {
+                    WriteIndented = true
+                }));
+            }
+            catch (Exception ex)
+            {
+                this.logger.Error($"Failed to save configuration {property.Name} for module {module.GetType().Name}.", ex);
+            }
         }
 
         private void ModuleConfiguration_OnLoadingRequest(object? sender, BattleBitModule module, PropertyInfo property, string serverName)