cyberpresidentvanellope/tgstation-server
1
0
Fork 0
mirror of https://github.com/tgstation/tgstation-server.git synced 2026-08-23 21:16:52 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1609 from tgstation/DumpOnHealthCheckRestartFix

Browse Source 
Prevent DumpOnHealthCheckRestart from crashing the monitor
This commit is contained in:
Jordan Dominion
2023-07-23 14:33:58 -04:00
committed by GitHub
parent 144b813eaf 5b17368e1b
commit fee685ddef
1 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Tgstation.Server.Host/Components/Watchdog/WatchdogBase.cs
+15 -1
View File
@@ -4,7 +4,10 @@ using System.Linq;
using System.Threading;
using System.Threading.Tasks;
using BetterWin32Errors;
using Microsoft.Extensions.Logging;
using Serilog.Context;
using Tgstation.Server.Api.Models;
@@ -1086,7 +1089,18 @@ namespace Tgstation.Server.Host.Components.Watchdog
if (ActiveLaunchParameters.DumpOnHealthCheckRestart.Value)
{
Logger.LogDebug("DumpOnHealthCheckRestart enabled.");
await CreateDump(cancellationToken);
try
{
await CreateDump(cancellationToken);
}
catch (JobException ex)
{
Logger.LogWarning(ex, "Creating dump failed!");
}
catch (Win32Exception ex)
{
Logger.LogWarning(ex, "Creating dump failed!");
}
}
else
Logger.LogTrace("DumpOnHealthCheckRestart disabled.");