EllieBotDevs/EllieHub
2
0
Fork 1
Code Issues Pull requests Projects Releases 9 Packages Wiki Activity

Fixed error when flushing all logs

Browse source
This commit is contained in:
Toastie 2025-01-07 19:06:33 +13:00
parent f0030af0ae
commit 516e768aba
Signed by: toastie_t0ast
GPG key ID: 0861BE54AD481DC7
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
EllieHub/Features/BotConfig/Services/LogWriter.cs
View file

@ -1,6 +1,7 @@
using EllieHub.Features.AppConfig.Models;
using EllieHub.Features.BotConfig.Models;
using EllieHub.Features.BotConfig.Services.Abstractions;
using System.Collections.Concurrent;
using System.Diagnostics.CodeAnalysis;
using System.Text;
@ -11,7 +12,7 @@ namespace EllieHub.Features.BotConfig.Services;
/// </summary>
public sealed class LogWriter : ILogWriter
{
private readonly Dictionary<Guid, StringBuilder> _botLogs = [];
private readonly ConcurrentDictionary<Guid, StringBuilder> _botLogs = [];
private readonly ReadOnlyAppSettings _appConfig;
/// <inheritdoc/>
@ -27,7 +28,11 @@ public sealed class LogWriter : ILogWriter
/// <inheritdoc/>
public async Task<bool> FlushAllAsync(bool removeFromMemory = false, CancellationToken cToken = default)
{
var result = await Task.WhenAll(_botLogs.Keys.Select(x => FlushAsync(x, removeFromMemory, cToken)));
var result = await Task.WhenAll(_botLogs.Keys.Select(x => FlushAsync(x, false, cToken)));
if (removeFromMemory)
_botLogs.Clear();
return result.Any(x => x);
}
@ -38,7 +43,7 @@ public sealed class LogWriter : ILogWriter
return false;
if (removeFromMemory)
_botLogs.Remove(botId);
_botLogs.Remove(botId, out _);
if (logStringBuilder.Length is 0)
return false;