Merge branch '2.3' into 2.4-pr

Application/GameEventHandler.cs

@@ -1,8 +1,6 @@
 using SharedLibraryCore;
 using SharedLibraryCore.Events;
 using SharedLibraryCore.Interfaces;
-using System.Collections.Generic;
-using System.Linq;
 using System.Threading;
 
 namespace IW4MAdmin.Application
@@ -17,6 +15,11 @@ namespace IW4MAdmin.Application
 
         public void AddEvent(GameEvent gameEvent)
         {
+#if DEBUG
+            ThreadPool.GetMaxThreads(out int workerThreads, out int n);
+            ThreadPool.GetAvailableThreads(out int availableThreads, out int m);
+            gameEvent.Owner.Logger.WriteDebug($"There are {workerThreads - availableThreads} active threading tasks");
+#endif
             Manager.OnServerEvent?.Invoke(gameEvent.Owner, new GameEventArgs(null, false, gameEvent));
         }
     }

Application/IO/GameLogEventDetection.cs

@@ -72,7 +72,6 @@ namespace IW4MAdmin.Application.IO
             foreach (var ev in events)
             {
                 _server.Manager.GetEventHandler().AddEvent(ev);
-                await ev.WaitAsync(Utilities.DefaultCommandTimeout, ev.Owner.Manager.CancellationToken);
             }
 
             previousFileSize = fileSize;

Application/IW4MServer.cs

@@ -648,8 +648,12 @@ namespace IW4MAdmin
                         // because we don't want to try to fill up a slot that's not empty yet
                         waiterList.Add(e);
                     }
+
                     // wait for all the disconnect tasks to finish
-                    await Task.WhenAll(waiterList.Select(e => e.WaitAsync(Utilities.DefaultCommandTimeout, Manager.CancellationToken)));
+                    foreach (var waiter in waiterList)
+                    {
+                        waiter.Wait();
+                    }
 
                     waiterList.Clear();
                     // this are our new connecting clients
@@ -673,7 +677,10 @@ namespace IW4MAdmin
                     }
 
                     // wait for all the connect tasks to finish
-                    await Task.WhenAll(waiterList.Select(e => e.WaitAsync(Utilities.DefaultCommandTimeout, Manager.CancellationToken)));
+                    foreach (var waiter in waiterList)
+                    {
+                        waiter.Wait();
+                    }
 
                     waiterList.Clear();
                     // these are the clients that have updated
@@ -690,7 +697,10 @@ namespace IW4MAdmin
                         waiterList.Add(e);
                     }
 
-                    await Task.WhenAll(waiterList.Select(e => e.WaitAsync(Utilities.DefaultCommandTimeout, Manager.CancellationToken)));
+                    foreach (var waiter in waiterList)
+                    {
+                        waiter.Wait();
+                    }
 
                     if (ConnectionErrors > 0)
                     {

Application/Misc/Logger.cs

@@ -54,9 +54,9 @@ namespace IW4MAdmin.Application
             }
         }
 
-        async Task Write(string msg, LogType type)
+        void Write(string msg, LogType type)
         {
-            await OnLogWriting.WaitAsync();
+            OnLogWriting.Wait();
 
             string stringType = type.ToString();
             msg = msg.StripColors();
@@ -74,7 +74,7 @@ namespace IW4MAdmin.Application
 #if DEBUG
                 // lets keep it simple and dispose of everything quickly as logging wont be that much (relatively)
                 Console.WriteLine(LogLine);
-                await File.AppendAllTextAsync(FileName, $"{LogLine}{Environment.NewLine}");
+                File.AppendAllText(FileName, $"{LogLine}{Environment.NewLine}");
                 //Debug.WriteLine(msg);
 #else
                 if (type == LogType.Error || type == LogType.Verbose)

SharedLibraryCore/Events/GameEvent.cs

@@ -235,5 +235,11 @@ namespace SharedLibraryCore
                 return this;
             });
         }
+
+        public GameEvent Wait()
+        {
+            OnProcessed.Wait();
+            return this;
+        }
     }
 }

SharedLibraryCore/Services/ClientService.cs

@@ -146,6 +146,19 @@ namespace SharedLibraryCore.Services
                     .Where(_client => _client.AliasLinkId == oldAliasLink.AliasLinkId)
                     .ForEachAsync(_client => _client.AliasLinkId = newAliasLink.AliasLinkId);
 
+                // we also need to update all the penalties or they get deleted
+                // scenario
+                // link1 joins with ip1
+                // link2 joins with ip2,
+                // link2 receives penalty
+                // link2 joins with ip1
+                // pre existing link for link2 detected
+                // link2 is deleted
+                // link2 penalties are orphaned
+                await context.Penalties
+                    .Where(_penalty => _penalty.LinkId == oldAliasLink.AliasLinkId)
+                    .ForEachAsync(_penalty => _penalty.LinkId = newAliasLink.AliasLinkId);
+
                 entity.AliasLink = newAliasLink;
                 entity.AliasLinkId = newAliasLink.AliasLinkId;
 

WebfrontCore/Controllers/ClientController.cs

@@ -104,9 +104,7 @@ namespace WebfrontCore.Controllers
         public async Task<IActionResult> PrivilegedAsync()
         {
             var admins = (await Manager.GetClientService().GetPrivilegedClients())
-                .GroupBy(a => a.AliasLinkId)
-                .Select(_client => _client.OrderByDescending(_c => _c.LastConnection).First())
-                .OrderByDescending(_client => _client.Level);
+                .OrderBy(_client => _client.Name);
 
             var adminsDict = new Dictionary<EFClient.Permission, IList<ClientInfo>>();