fix: better logging (AscensionGameDev#2091)

Author: pandinocoder

Intersect.Network/LiteNetLib/AesGcmAlgorithm.cs

@@ -134,7 +134,11 @@ public override EncryptionResult TryDecrypt(ReadOnlySpan<byte> cipherdata, out R
             Log.Debug($"cipherdataView({cipherdataView.Length})={Convert.ToHexString(cipherdataView)}");
             Log.Debug($"key={Convert.ToHexString(_key.Span)}");
 #endif
-            Log.Error(exception);
+            var configuredLogLevel = Options.Instance?.Logging.Level ?? LogLevel.Diagnostic;
+            if (configuredLogLevel < LogLevel.Debug)
+            {
+                Log.Error(exception);
+            }
             plaindata = default;
             return EncryptionResult.Error;
         }
@@ -207,7 +211,11 @@ public override EncryptionResult TryEncrypt(ReadOnlySpan<byte> plaindata, out Re
         }
         catch (Exception exception)
         {
-            Log.Error(exception);
+            var configuredLogLevel = Options.Instance?.Logging.Level ?? LogLevel.Diagnostic;
+            if (configuredLogLevel < LogLevel.Debug)
+            {
+                Log.Error(exception);
+            }
             cipherdata = default;
             return EncryptionResult.Error;
         }

Intersect.Server.Core/Database/IntersectDbContext.cs

@@ -6,6 +6,8 @@
 using Intersect.Logging;
 using Intersect.Reflection;
 using Intersect.Server.Core;
+using Intersect.Server.Database.PlayerData.Players;
+using Intersect.Server.Entities;
 using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.ChangeTracking;
 using Microsoft.EntityFrameworkCore.Infrastructure;
@@ -207,13 +209,42 @@ public override int SaveChanges(bool acceptAllChangesOnSuccess)
         }
         catch (DbUpdateConcurrencyException concurrencyException)
         {
+            static Guid GetUserIdFrom(object entry)
+            {
+                return entry switch
+                {
+                    IPlayerOwned playerOwned => playerOwned.Player?.User?.Id ?? playerOwned.Player?.UserId ?? default,
+                    Player player => player.User?.Id ?? player.UserId,
+                    _ => default,
+                };
+            }
+
             var concurrencyErrors = new StringBuilder();
+            var userIds = concurrencyException.Entries.Select(GetUserIdFrom)
+                .Where(id => id != default)
+                .Distinct()
+                .ToArray();
+            if (userIds.Length == 1)
+            {
+                concurrencyErrors.AppendLine($"This failure was wholly caused by user {userIds.First()}");
+            }
+            else
+            {
+                concurrencyErrors.AppendLine($"Involved users: {string.Join(", ", userIds)}");
+            }
+
             foreach (var entry in concurrencyException.Entries)
             {
                 concurrencyErrors.AppendLine(
                     $"[{entry.State}] {entry.Entity.GetFullishName()} ({entry.GetFullishName()})"
                 );
 
+                var userId = GetUserIdFrom(entry.Entity);
+                if (userId != default)
+                {
+                    concurrencyErrors.AppendLine($"User Id {userId}");
+                }
+
                 var proposedValues = entry.CurrentValues;
                 var databaseValues = entry.GetDatabaseValues();