From e70290975d524874a8742a675cbad00c385225d6 Mon Sep 17 00:00:00 2001 From: rootdarkarchon Date: Thu, 11 May 2023 01:07:12 +0200 Subject: [PATCH] reduce logging for player null --- MareSynchronos/PlayerData/Handlers/PairHandler.cs | 8 +------- MareSynchronos/WebAPI/Files/FileTransferOrchestrator.cs | 2 +- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/MareSynchronos/PlayerData/Handlers/PairHandler.cs b/MareSynchronos/PlayerData/Handlers/PairHandler.cs index f71d2c7..d6563c5 100644 --- a/MareSynchronos/PlayerData/Handlers/PairHandler.cs +++ b/MareSynchronos/PlayerData/Handlers/PairHandler.cs @@ -364,12 +364,6 @@ public sealed class PairHandler : DisposableMediatorSubscriberBase Logger.LogDebug("[{applicationId}] Application finished", _applicationId); } - catch (ArgumentNullException ex) - { - Logger.LogWarning(ex, "[{applicationId}] Cancelled, player turned null during application", _applicationId); - IsVisible = false; - _applyLastReceivedDataOnVisible = true; - } catch (Exception ex) { if (ex is AggregateException aggr && aggr.InnerExceptions.Any(e => e is ArgumentNullException)) @@ -377,7 +371,7 @@ public sealed class PairHandler : DisposableMediatorSubscriberBase IsVisible = false; _applyLastReceivedDataOnVisible = true; _cachedData = charaData; - Logger.LogWarning(aggr, "[{applicationId}] Cancelled, player turned null during application", _applicationId); + Logger.LogDebug("[{applicationId}] Cancelled, player turned null during application", _applicationId); } else { diff --git a/MareSynchronos/WebAPI/Files/FileTransferOrchestrator.cs b/MareSynchronos/WebAPI/Files/FileTransferOrchestrator.cs index c69b539..6d99c12 100644 --- a/MareSynchronos/WebAPI/Files/FileTransferOrchestrator.cs +++ b/MareSynchronos/WebAPI/Files/FileTransferOrchestrator.cs @@ -109,7 +109,7 @@ public class FileTransferOrchestrator : DisposableMediatorSubscriberBase } catch (Exception ex) { - Logger.LogCritical(ex, "Error during SendRequestInternal for {uri}", requestMessage.RequestUri); + Logger.LogWarning(ex, "Error during SendRequestInternal for {uri}", requestMessage.RequestUri); throw; } }