Merge pull request #968 from dremin/log-fixes

Misc logging fixes
dremin · Dec 1, 2024 · d8d56a7 · d8d56a7
2 parents 74228bd + ce8e09b
commit d8d56a7
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/RetroBar/Utilities/ManagedShellLogger.cs b/RetroBar/Utilities/ManagedShellLogger.cs
@@ -39,17 +39,24 @@ private void SetupLogging()
 
             SetupFileLog();
 
-            ShellLogger.Attach(new ConsoleLog());
+            ShellLogger.Attach(new ConsoleLog(), true);
         }
 
         private void SetupFileLog()
         {
             DeleteOldLogFiles();
 
-            _fileLog = new FileLog(Path.Combine(_logPath, $"{_logName}.{_logExt}"));
-            _fileLog?.Open();
+            try
+            {
+                _fileLog = new FileLog(Path.Combine(_logPath, $"{_logName}.{_logExt}"));
+                _fileLog?.Open();
 
-            ShellLogger.Attach(_fileLog);
+                ShellLogger.Attach(_fileLog);
+            }
+            catch (Exception ex)
+            {
+                ShellLogger.Error($"Unable to open and attach file logger: {ex.Message}");
+            }
         }
 
         private void DeleteOldLogFiles()