ChatManager improvements

- SET THE CHANNELS IN CHANGE SETTINGS OMG
- Log ProcessMessage failure when they happen instead of at server shutdown
This commit is contained in:
Jordan Brown
2021-10-14 10:25:53 -04:00
parent d13c792f29
commit 99b5f7585e
@@ -316,6 +316,7 @@ namespace Tgstation.Server.Host.Components.Chat
Name = newSettings.Name,
ReconnectionInterval = newSettings.ReconnectionInterval,
Provider = newSettings.Provider,
Channels = newSettings.Channels,
});
}
@@ -822,7 +823,15 @@ namespace Tgstation.Server.Host.Components.Chat
{
var localActiveProcessingTask = activeProcessingTask;
using (LogContext.PushProperty("ChatMessage", messageNumber))
await ProcessMessage(completedMessageTaskKvp.Key, message, cancellationToken).ConfigureAwait(false);
try
{
await ProcessMessage(completedMessageTaskKvp.Key, message, cancellationToken).ConfigureAwait(false);
}
catch (Exception ex)
{
logger.LogError(ex, "Error processing message {messageNumber}!", messageNumber);
}
await localActiveProcessingTask.ConfigureAwait(false);
}