fix introduced bug :)

- reset recoil state on map change
- refactor config
- remove m21 from chest detection
- allow ignored client ids

.gitignore

@@ -244,5 +244,3 @@ launchSettings.json
 /Tests/ApplicationTests/Files/GameEvents.json
 /Tests/ApplicationTests/Files/replay.json
 /GameLogServer/game_log_server_env
-.idea/*
-*.db

Application/API/Master/IMasterApi.cs

@@ -1,7 +1,5 @@
-using System;
+using System.Collections.Generic;
-using System.Collections.Generic;
 using System.Threading.Tasks;
-using IW4MAdmin.Application.Misc;
 using Newtonsoft.Json;
 using RestEase;
 using SharedLibraryCore.Helpers;
@@ -37,13 +35,6 @@ namespace IW4MAdmin.Application.API.Master
         public string Message { get; set; }
     }
 
-    public class PluginSubscriptionContent
-    {
-        public string Content { get; set; }
-        public PluginType Type { get; set; }
-    }
-
-
     /// <summary>
     /// Defines the capabilities of the master API
     /// </summary>
@@ -72,8 +63,5 @@ namespace IW4MAdmin.Application.API.Master
 
         [Get("localization/{languageTag}")]
         Task<SharedLibraryCore.Localization.Layout> GetLocalization([Path("languageTag")] string languageTag);
-
-        [Get("plugin_subscriptions")]
-        Task<IEnumerable<PluginSubscriptionContent>> GetPluginSubscription([Query("instance_id")] Guid instanceId, [Query("subscription_id")] string subscription_id);
     }
 }

Application/Application.csproj

@@ -5,7 +5,7 @@
     <TargetFramework>netcoreapp3.1</TargetFramework>
     <MvcRazorExcludeRefAssembliesFromPublish>false</MvcRazorExcludeRefAssembliesFromPublish>
     <PackageId>RaidMax.IW4MAdmin.Application</PackageId>
-    <Version>2020.0.0.0</Version>
+    <Version>2.3.2.0</Version>
     <Authors>RaidMax</Authors>
     <Company>Forever None</Company>
     <Product>IW4MAdmin</Product>
@@ -25,13 +25,13 @@
 
   <ItemGroup>
     <PackageReference Include="Jint" Version="3.0.0-beta-1632" />
-    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="3.1.10">
+    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="3.1.7">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>
-    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="3.1.10" />
+    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="3.1.7" />
-    <PackageReference Include="RestEase" Version="1.5.1" />
+    <PackageReference Include="RestEase" Version="1.5.0" />
-    <PackageReference Include="System.Text.Encoding.CodePages" Version="5.0.0" />
+    <PackageReference Include="System.Text.Encoding.CodePages" Version="4.7.1" />
   </ItemGroup>
 
   <PropertyGroup>
@@ -39,6 +39,7 @@
     <ConcurrentGarbageCollection>true</ConcurrentGarbageCollection>
     <TieredCompilation>true</TieredCompilation>
     <LangVersion>Latest</LangVersion>
+    <StartupObject></StartupObject>
   </PropertyGroup>
 
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Prerelease|AnyCPU'">
@@ -58,9 +59,6 @@
     <None Update="DefaultSettings.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
-    <None Update="Configuration\LoggingConfiguration.json">
-      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
-    </None>
   </ItemGroup>
 
   <Target Name="PreBuild" BeforeTargets="PreBuildEvent">

Application/ApplicationManager.cs

@@ -1,4 +1,5 @@
-using IW4MAdmin.Application.EventParsers;
+using IW4MAdmin.Application.API.Master;
+using IW4MAdmin.Application.EventParsers;
 using IW4MAdmin.Application.Extensions;
 using IW4MAdmin.Application.Misc;
 using IW4MAdmin.Application.RconParsers;
@@ -8,9 +9,11 @@ using SharedLibraryCore.Configuration;
 using SharedLibraryCore.Configuration.Validation;
 using SharedLibraryCore.Database;
 using SharedLibraryCore.Database.Models;
+using SharedLibraryCore.Dtos;
 using SharedLibraryCore.Exceptions;
 using SharedLibraryCore.Helpers;
 using SharedLibraryCore.Interfaces;
+using SharedLibraryCore.QueryHelper;
 using SharedLibraryCore.Services;
 using System;
 using System.Collections;
@@ -21,13 +24,7 @@ using System.Reflection;
 using System.Text;
 using System.Threading;
 using System.Threading.Tasks;
-using IW4MAdmin.Application.Migration;
-using Microsoft.Extensions.DependencyInjection;
-using Microsoft.Extensions.Logging;
-using Serilog.Context;
 using static SharedLibraryCore.GameEvent;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
-using ObsoleteLogger = SharedLibraryCore.Interfaces.ILogger;
 
 namespace IW4MAdmin.Application
 {
@@ -35,7 +32,7 @@ namespace IW4MAdmin.Application
     {
         private readonly ConcurrentBag<Server> _servers;
         public List<Server> Servers => _servers.OrderByDescending(s => s.ClientNum).ToList();
-        [Obsolete] public ObsoleteLogger Logger => _serviceProvider.GetRequiredService<ObsoleteLogger>();
+        public ILogger Logger => GetLogger(0);
         public bool IsRunning { get; private set; }
         public bool IsInitialized { get; private set; }
         public DateTime StartTime { get; private set; }
@@ -53,9 +50,11 @@ namespace IW4MAdmin.Application
         private readonly ILogger _logger;
         private readonly List<MessageToken> MessageTokens;
         private readonly ClientService ClientSvc;
+        readonly AliasService AliasSvc;
         readonly PenaltyService PenaltySvc;
         public IConfigurationHandler<ApplicationConfiguration> ConfigHandler;
         readonly IPageList PageList;
+        private readonly Dictionary<long, ILogger> _loggers = new Dictionary<long, ILogger>();
         private readonly IMetaService _metaService;
         private readonly TimeSpan _throttleTimeout = new TimeSpan(0, 1, 0);
         private readonly CancellationTokenSource _tokenSource;
@@ -69,33 +68,30 @@ namespace IW4MAdmin.Application
         private readonly IScriptCommandFactory _scriptCommandFactory;
         private readonly IMetaRegistration _metaRegistration;
         private readonly IScriptPluginServiceResolver _scriptPluginServiceResolver;
-        private readonly IServiceProvider _serviceProvider;
-        private readonly ChangeHistoryService _changeHistoryService;
-        private readonly ApplicationConfiguration _appConfig;
-        public ConcurrentDictionary<long, GameEvent> ProcessingEvents { get; } = new ConcurrentDictionary<long, GameEvent>();
 
-        public ApplicationManager(ILogger<ApplicationManager> logger, IMiddlewareActionHandler actionHandler, IEnumerable<IManagerCommand> commands,
+        public ApplicationManager(ILogger logger, IMiddlewareActionHandler actionHandler, IEnumerable<IManagerCommand> commands,
             ITranslationLookup translationLookup, IConfigurationHandler<CommandConfiguration> commandConfiguration,
             IConfigurationHandler<ApplicationConfiguration> appConfigHandler, IGameServerInstanceFactory serverInstanceFactory,
             IEnumerable<IPlugin> plugins, IParserRegexFactory parserRegexFactory, IEnumerable<IRegisterEvent> customParserEvents,
             IEventHandler eventHandler, IScriptCommandFactory scriptCommandFactory, IDatabaseContextFactory contextFactory, IMetaService metaService,
-            IMetaRegistration metaRegistration, IScriptPluginServiceResolver scriptPluginServiceResolver, ClientService clientService, IServiceProvider serviceProvider,
+            IMetaRegistration metaRegistration, IScriptPluginServiceResolver scriptPluginServiceResolver)
-            ChangeHistoryService changeHistoryService, ApplicationConfiguration appConfig, PenaltyService penaltyService)
         {
             MiddlewareActionHandler = actionHandler;
             _servers = new ConcurrentBag<Server>();
             MessageTokens = new List<MessageToken>();
-            ClientSvc = clientService;
+            ClientSvc = new ClientService(contextFactory);
-            PenaltySvc = penaltyService;
+            AliasSvc = new AliasService();
+            PenaltySvc = new PenaltyService();
             ConfigHandler = appConfigHandler;
             StartTime = DateTime.UtcNow;
             PageList = new PageList();
-            AdditionalEventParsers = new List<IEventParser>() { new BaseEventParser(parserRegexFactory, logger, _appConfig) };
+            AdditionalEventParsers = new List<IEventParser>() { new BaseEventParser(parserRegexFactory, logger, appConfigHandler.Configuration()) };
-            AdditionalRConParsers = new List<IRConParser>() { new BaseRConParser(serviceProvider.GetRequiredService<ILogger<BaseRConParser>>(), parserRegexFactory) };
+            AdditionalRConParsers = new List<IRConParser>() { new BaseRConParser(parserRegexFactory) };
             TokenAuthenticator = new TokenAuthentication();
             _logger = logger;
             _metaService = metaService;
             _tokenSource = new CancellationTokenSource();
+            _loggers.Add(0, logger);
             _commands = commands.ToList();
             _translationLookup = translationLookup;
             _commandConfiguration = commandConfiguration;
@@ -106,9 +102,6 @@ namespace IW4MAdmin.Application
             _scriptCommandFactory = scriptCommandFactory;
             _metaRegistration = metaRegistration;
             _scriptPluginServiceResolver = scriptPluginServiceResolver;
-            _serviceProvider = serviceProvider;
-            _changeHistoryService = changeHistoryService;
-            _appConfig = appConfig;
             Plugins = plugins;
         }
 
@@ -116,7 +109,9 @@ namespace IW4MAdmin.Application
 
         public async Task ExecuteEvent(GameEvent newEvent)
         {
-            ProcessingEvents.TryAdd(newEvent.Id, newEvent);
+#if DEBUG == true
+            Logger.WriteDebug($"Entering event process for {newEvent.Id}");
+#endif
 
             // the event has failed already
             if (newEvent.Failed)
@@ -129,17 +124,22 @@ namespace IW4MAdmin.Application
                 await newEvent.Owner.ExecuteEvent(newEvent);
 
                 // save the event info to the database
-                await _changeHistoryService.Add(newEvent);
+                var changeHistorySvc = new ChangeHistoryService();
+                await changeHistorySvc.Add(newEvent);
+
+#if DEBUG
+                Logger.WriteDebug($"Processed event with id {newEvent.Id}");
+#endif
             }
 
             catch (TaskCanceledException)
             {
-                _logger.LogDebug("Received quit signal for event id {eventId}, so we are aborting early", newEvent.Id);
+                Logger.WriteInfo($"Received quit signal for event id {newEvent.Id}, so we are aborting early");
             }
 
             catch (OperationCanceledException)
             {
-                _logger.LogDebug("Received quit signal for event id {eventId}, so we are aborting early", newEvent.Id);
+                Logger.WriteInfo($"Received quit signal for event id {newEvent.Id}, so we are aborting early");
             }
 
             // this happens if a plugin requires login
@@ -152,58 +152,31 @@ namespace IW4MAdmin.Application
             catch (NetworkException ex)
             {
                 newEvent.FailReason = EventFailReason.Exception;
-                using (LogContext.PushProperty("Server", newEvent.Owner?.ToString()))
+                Logger.WriteError(ex.Message);
-                {
+                Logger.WriteDebug(ex.GetExceptionInfo());
-                    _logger.LogError(ex, ex.Message);
-                }
             }
 
             catch (ServerException ex)
             {
                 newEvent.FailReason = EventFailReason.Exception;
-                using (LogContext.PushProperty("Server", newEvent.Owner?.ToString()))
+                Logger.WriteWarning(ex.Message);
-                {
-                    _logger.LogError(ex, ex.Message);
-                }
             }
 
             catch (Exception ex)
             {
                 newEvent.FailReason = EventFailReason.Exception;
-                Console.WriteLine(Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_EXCEPTION"].FormatExt(newEvent.Owner));
+                Logger.WriteError(Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_EXCEPTION"].FormatExt(newEvent.Owner));
-                using (LogContext.PushProperty("Server", newEvent.Owner?.ToString()))
+                Logger.WriteDebug(ex.GetExceptionInfo());
-                {
-                    _logger.LogError(ex, "Unexpected exception");
-                }
-            }
-
-            skip:
-            if (newEvent.Type == EventType.Command && newEvent.ImpersonationOrigin == null)
-            {
-                var correlatedEvents =
-                    ProcessingEvents.Values.Where(ev =>
-                            ev.CorrelationId == newEvent.CorrelationId && ev.Id != newEvent.Id)
-                        .ToList();
-
-                await Task.WhenAll(correlatedEvents.Select(ev =>
-                    ev.WaitAsync(Utilities.DefaultCommandTimeout, CancellationToken)));
-                newEvent.Output.AddRange(correlatedEvents.SelectMany(ev => ev.Output));
-
-                foreach (var correlatedEvent in correlatedEvents)
-                {
-                    ProcessingEvents.Remove(correlatedEvent.Id, out _);
-                }
-            }
-
-            // we don't want to remove events that are correlated to command
-            if (ProcessingEvents.Values.ToList()?.Count(gameEvent => gameEvent.CorrelationId == newEvent.CorrelationId) == 1)
-            {
-                ProcessingEvents.Remove(newEvent.Id, out _);
             }
 
+        skip:
             // tell anyone waiting for the output that we're done
             newEvent.Complete();
             OnGameEventExecuted?.Invoke(this, newEvent);
+
+#if DEBUG == true
+            Logger.WriteDebug($"Exiting event process for {newEvent.Id}");
+#endif
         }
 
         public IList<Server> GetServers()
@@ -253,14 +226,17 @@ namespace IW4MAdmin.Application
                         try
                         {
                             await server.ProcessUpdatesAsync(_tokenSource.Token);
+
+                            if (server.Throttled)
+                            {
+                                await Task.Delay((int)_throttleTimeout.TotalMilliseconds, _tokenSource.Token);
+                            }
                         }
 
                         catch (Exception e)
                         {
-                            using (LogContext.PushProperty("Server", server.ToString()))
+                            Logger.WriteWarning($"Failed to update status for {server}");
-                            {
+                            Logger.WriteDebug(e.GetExceptionInfo());
-                                _logger.LogError(e, "Failed to update status");
-                            }
                         }
 
                         finally
@@ -269,7 +245,12 @@ namespace IW4MAdmin.Application
                         }
                     }));
                 }
-
+#if DEBUG
+                Logger.WriteDebug($"{runningUpdateTasks.Count} servers queued for stats updates");
+                ThreadPool.GetMaxThreads(out int workerThreads, out int n);
+                ThreadPool.GetAvailableThreads(out int availableThreads, out int m);
+                Logger.WriteDebug($"There are {workerThreads - availableThreads} active threading tasks");
+#endif
                 try
                 {
                     await Task.Delay(ConfigHandler.Configuration().RConPollRate, _tokenSource.Token);
@@ -308,8 +289,8 @@ namespace IW4MAdmin.Application
 
                             catch (Exception ex)
                             {
-                                Console.WriteLine(Utilities.CurrentLocalization.LocalizationIndex["PLUGIN_IMPORTER_ERROR"].FormatExt(scriptPlugin.Name));
+                                Logger.WriteError(Utilities.CurrentLocalization.LocalizationIndex["PLUGIN_IMPORTER_ERROR"].FormatExt(scriptPlugin.Name));
-                                _logger.LogError(ex, "Could not properly load plugin {plugin}", scriptPlugin.Name);
+                                Logger.WriteDebug(ex.Message);
                             }
                         };
                     }
@@ -322,29 +303,32 @@ namespace IW4MAdmin.Application
 
                 catch (Exception ex)
                 {
-                    _logger.LogError(ex, $"{_translationLookup["SERVER_ERROR_PLUGIN"]} {plugin.Name}");
+                    Logger.WriteError($"{_translationLookup["SERVER_ERROR_PLUGIN"]} {plugin.Name}");
+                    Logger.WriteDebug(ex.GetExceptionInfo());
                 }
             }
             #endregion
 
             #region CONFIG
+            var config = ConfigHandler.Configuration();
+
             // copy over default config if it doesn't exist
-            if (!_appConfig.Servers?.Any() ?? true)
+            if (config == null)
             {
                 var defaultConfig = new BaseConfigurationHandler<DefaultConfiguration>("DefaultSettings").Configuration();
-                //ConfigHandler.Set((ApplicationConfiguration)new ApplicationConfiguration().Generate());
+                ConfigHandler.Set((ApplicationConfiguration)new ApplicationConfiguration().Generate());
-                //var newConfig = ConfigHandler.Configuration();
+                var newConfig = ConfigHandler.Configuration();
 
-                _appConfig.AutoMessages = defaultConfig.AutoMessages;
+                newConfig.AutoMessages = defaultConfig.AutoMessages;
-                _appConfig.GlobalRules = defaultConfig.GlobalRules;
+                newConfig.GlobalRules = defaultConfig.GlobalRules;
-                _appConfig.Maps = defaultConfig.Maps;
+                newConfig.Maps = defaultConfig.Maps;
-                _appConfig.DisallowedClientNames = defaultConfig.DisallowedClientNames;
+                newConfig.DisallowedClientNames = defaultConfig.DisallowedClientNames;
-                _appConfig.QuickMessages = defaultConfig.QuickMessages;
+                newConfig.QuickMessages = defaultConfig.QuickMessages;
 
-                //if (newConfig.Servers == null)
+                if (newConfig.Servers == null)
                 {
-                    ConfigHandler.Set(_appConfig);
+                    ConfigHandler.Set(newConfig);
-                    _appConfig.Servers = new ServerConfiguration[1];
+                    newConfig.Servers = new ServerConfiguration[1];
 
                     do
                     {
@@ -359,29 +343,30 @@ namespace IW4MAdmin.Application
                             serverConfig.AddEventParser(parser);
                         }
 
-                        _appConfig.Servers = _appConfig.Servers.Where(_servers => _servers != null).Append((ServerConfiguration)serverConfig.Generate()).ToArray();
+                        newConfig.Servers = newConfig.Servers.Where(_servers => _servers != null).Append((ServerConfiguration)serverConfig.Generate()).ToArray();
                     } while (Utilities.PromptBool(_translationLookup["SETUP_SERVER_SAVE"]));
 
+                    config = newConfig;
                     await ConfigHandler.Save();
                 }
             }
 
             else
             {
-                if (string.IsNullOrEmpty(_appConfig.Id))
+                if (string.IsNullOrEmpty(config.Id))
                 {
-                    _appConfig.Id = Guid.NewGuid().ToString();
+                    config.Id = Guid.NewGuid().ToString();
                     await ConfigHandler.Save();
                 }
 
-                if (string.IsNullOrEmpty(_appConfig.WebfrontBindUrl))
+                if (string.IsNullOrEmpty(config.WebfrontBindUrl))
                 {
-                    _appConfig.WebfrontBindUrl = "http://0.0.0.0:1624";
+                    config.WebfrontBindUrl = "http://0.0.0.0:1624";
                     await ConfigHandler.Save();
                 }
 
                 var validator = new ApplicationConfigurationValidator();
-                var validationResult = validator.Validate(_appConfig);
+                var validationResult = validator.Validate(config);
 
                 if (!validationResult.IsValid)
                 {
@@ -392,7 +377,7 @@ namespace IW4MAdmin.Application
                     };
                 }
 
-                foreach (var serverConfig in _appConfig.Servers)
+                foreach (var serverConfig in config.Servers)
                 {
                     Migration.ConfigurationMigration.ModifyLogPath020919(serverConfig);
 
@@ -414,27 +399,26 @@ namespace IW4MAdmin.Application
                 }
             }
 
-            if (_appConfig.Servers.Length == 0)
+            if (config.Servers.Length == 0)
             {
                 throw new ServerException("A server configuration in IW4MAdminSettings.json is invalid");
             }
 
             Encoding.RegisterProvider(CodePagesEncodingProvider.Instance);
-            Utilities.EncodingType = Encoding.GetEncoding(!string.IsNullOrEmpty(_appConfig.CustomParserEncoding) ? _appConfig.CustomParserEncoding : "windows-1252");
+            Utilities.EncodingType = Encoding.GetEncoding(!string.IsNullOrEmpty(config.CustomParserEncoding) ? config.CustomParserEncoding : "windows-1252");
 
             #endregion
 
             #region DATABASE
-            _logger.LogInformation("Beginning database migration sync");
+            using (var db = new DatabaseContext(GetApplicationSettings().Configuration()?.ConnectionString,
-            Console.WriteLine(_translationLookup["MANAGER_MIGRATION_START"]);
+                GetApplicationSettings().Configuration()?.DatabaseProvider))
-            await ContextSeed.Seed(_serviceProvider.GetRequiredService<IDatabaseContextFactory>(), _tokenSource.Token);
+            {
-            await DatabaseHousekeeping.RemoveOldRatings(_serviceProvider.GetRequiredService<IDatabaseContextFactory>(), _tokenSource.Token);
+                await new ContextSeed(db).Seed();
-            _logger.LogInformation("Finished database migration sync");
+            }
-            Console.WriteLine(_translationLookup["MANAGER_MIGRATION_END"]);
             #endregion
 
             #region COMMANDS
-            if (await ClientSvc.HasOwnerAsync(_tokenSource.Token))
+            if (ClientSvc.GetOwners().Result.Count > 0)
             {
                 _commands.RemoveAll(_cmd => _cmd.GetType() == typeof(OwnerCommand));
             }
@@ -456,8 +440,8 @@ namespace IW4MAdmin.Application
 
             // this is because I want to store the command prefix in IW4MAdminSettings, but can't easily
             // inject it to all the places that need it
-            cmdConfig.CommandPrefix = _appConfig.CommandPrefix;
+            cmdConfig.CommandPrefix = config.CommandPrefix;
-            cmdConfig.BroadcastCommandPrefix = _appConfig.BroadcastCommandPrefix;
+            cmdConfig.BroadcastCommandPrefix = config.BroadcastCommandPrefix;
 
             foreach (var cmd in commandsToAddToConfig)
             {
@@ -488,7 +472,6 @@ namespace IW4MAdmin.Application
             }
             #endregion
 
-            Console.WriteLine(_translationLookup["MANAGER_COMMUNICATION_INFO"]);
             await InitializeServers();
         }
 
@@ -504,17 +487,13 @@ namespace IW4MAdmin.Application
                 {
                     // todo: this might not always be an IW4MServer
                     var ServerInstance = _serverInstanceFactory.CreateServer(Conf, this) as IW4MServer;
-                    using (LogContext.PushProperty("Server", ServerInstance.ToString()))
+                    await ServerInstance.Initialize();
-                    {
-                        _logger.LogInformation("Beginning server communication initialization");
-                        await ServerInstance.Initialize();
 
-                        _servers.Add(ServerInstance);
+                    _servers.Add(ServerInstance);
-                        Console.WriteLine(Utilities.CurrentLocalization.LocalizationIndex["MANAGER_MONITORING_TEXT"].FormatExt(ServerInstance.Hostname.StripColors()));
-                        _logger.LogInformation("Finishing initialization and now monitoring [{server}]", ServerInstance.Hostname, ServerInstance.ToString());
-                    }
 
+                    Logger.WriteVerbose(Utilities.CurrentLocalization.LocalizationIndex["MANAGER_MONITORING_TEXT"].FormatExt(ServerInstance.Hostname));
                     // add the start event for this server
+
                     var e = new GameEvent()
                     {
                         Type = GameEvent.EventType.Start,
@@ -528,11 +507,13 @@ namespace IW4MAdmin.Application
 
                 catch (ServerException e)
                 {
-                    Console.WriteLine(Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_UNFIXABLE"].FormatExt($"[{Conf.IPAddress}:{Conf.Port}]"));
+                    Logger.WriteError(Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_UNFIXABLE"].FormatExt($"[{Conf.IPAddress}:{Conf.Port}]"));
-                    using (LogContext.PushProperty("Server", $"{Conf.IPAddress}:{Conf.Port}"))
+
+                    if (e.GetType() == typeof(DvarException))
                     {
-                        _logger.LogError(e, "Unexpected exception occurred during initialization");
+                        Logger.WriteDebug($"{e.Message} {(e.GetType() == typeof(DvarException) ? $"({Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_DVAR_HELP"]})" : "")}");
                     }
+
                     lastException = e;
                 }
             }
@@ -567,10 +548,20 @@ namespace IW4MAdmin.Application
             Stop();
         }
 
-        [Obsolete]
+        public ILogger GetLogger(long serverId)
-        public ObsoleteLogger GetLogger(long serverId)
         {
-            return _serviceProvider.GetRequiredService<ObsoleteLogger>();
+            if (_loggers.ContainsKey(serverId))
+            {
+                return _loggers[serverId];
+            }
+
+            else
+            {
+                var newLogger = new Logger($"IW4MAdmin-Server-{serverId}");
+
+                _loggers.Add(serverId, newLogger);
+                return newLogger;
+            }
         }
 
         public IList<MessageToken> GetMessageTokens()
@@ -589,6 +580,11 @@ namespace IW4MAdmin.Application
             return ClientSvc;
         }
 
+        public AliasService GetAliasService()
+        {
+            return AliasSvc;
+        }
+
         public PenaltyService GetPenaltyService()
         {
             return PenaltySvc;
@@ -611,7 +607,7 @@ namespace IW4MAdmin.Application
 
         public IRConParser GenerateDynamicRConParser(string name)
         {
-            return new DynamicRConParser(_serviceProvider.GetRequiredService<ILogger<BaseRConParser>>(), _parserRegexFactory)
+            return new DynamicRConParser(_parserRegexFactory)
             {
                 Name = name
             };

Application/BuildScripts/PostBuild.bat

@@ -4,9 +4,11 @@ set TargetDir=%3
 set OutDir=%4
 set Version=%5
 
+echo %Version% > "%SolutionDir%DEPLOY\version.txt"
+
 echo Copying dependency configs
 copy "%SolutionDir%WebfrontCore\%OutDir%*.deps.json" "%TargetDir%"
-copy "%SolutionDir%SharedLibraryCore\%OutDir%*.deps.json" "%TargetDir%"
+copy "%SolutionDir%SharedLibaryCore\%OutDir%*.deps.json" "%TargetDir%"
 
 if not exist "%TargetDir%Plugins" (
 	echo "Making plugin dir"
@@ -14,3 +16,12 @@ if not exist "%TargetDir%Plugins" (
 )
 
 xcopy /y "%SolutionDir%Build\Plugins" "%TargetDir%Plugins\"
+
+echo Copying plugins for publish
+del %SolutionDir%BUILD\Plugins\Tests.dll
+xcopy /Y "%SolutionDir%BUILD\Plugins" "%SolutionDir%Publish\Windows\Plugins\"
+xcopy /Y "%SolutionDir%BUILD\Plugins" "%SolutionDir%Publish\WindowsPrerelease\Plugins\"
+
+echo Copying script plugins for publish
+xcopy /Y "%SolutionDir%Plugins\ScriptPlugins" "%SolutionDir%Publish\Windows\Plugins\"
+xcopy /Y "%SolutionDir%Plugins\ScriptPlugins" "%SolutionDir%Publish\WindowsPrerelease\Plugins\"

Application/Configuration/LoggingConfiguration.json

@@ -1,43 +0,0 @@
-{
-  "Serilog": {
-    "Using": [
-      "Serilog.Sinks.File"
-    ],
-    "MinimumLevel": "Information",
-    "WriteTo": [
-      {
-        "Name": "File",
-        "Args": {
-          "path": "Log/IW4MAdmin-Application.log",
-          "rollingInterval": "Day",
-          "outputTemplate": "[{Timestamp:yyyy-MM-dd HH:mm:ss.fff} {Server} {Level:u3}] {Message:lj}{NewLine}{Exception}"
-        }
-      }
-    ],
-    "Enrich": [
-      "FromLogContext",
-      "WithMachineName",
-      "WithThreadId"
-    ],
-    "Destructure": [
-      {
-        "Name": "ToMaximumDepth",
-        "Args": {
-          "maximumDestructuringDepth": 4
-        }
-      },
-      {
-        "Name": "ToMaximumStringLength",
-        "Args": {
-          "maximumStringLength": 1000
-        }
-      },
-      {
-        "Name": "ToMaximumCollectionCount",
-        "Args": {
-          "maximumCollectionCount": 24
-        }
-      }
-    ]
-  }
-}

Application/EventParsers/BaseEventParser.cs

@@ -5,9 +5,7 @@ using SharedLibraryCore.Interfaces;
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using Microsoft.Extensions.Logging;
 using static SharedLibraryCore.Server;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.EventParsers
 {
@@ -93,25 +91,16 @@ namespace IW4MAdmin.Application.EventParsers
         public virtual GameEvent GenerateGameEvent(string logLine)
         {
             var timeMatch = Configuration.Time.PatternMatcher.Match(logLine);
-            var gameTime = 0L;
+            int gameTime = 0;
 
             if (timeMatch.Success)
             {
-                if (timeMatch.Values[0].Contains(":"))
+                gameTime = timeMatch
-                {
+                    .Values
-                    gameTime = timeMatch
+                    .Skip(2)
-                        .Values
+                    // this converts the timestamp into seconds passed
-                        .Skip(2)
+                    .Select((_value, index) => int.Parse(_value.ToString()) * (index == 0 ? 60 : 1))
-                        // this converts the timestamp into seconds passed
+                    .Sum();
-                        .Select((_value, index) => long.Parse(_value.ToString()) * (index == 0 ? 60 : 1))
-                        .Sum();
-         
-                }
-                else
-                {
-                    gameTime = long.Parse(timeMatch.Values[0]);
-                }
-                
                 // we want to strip the time from the log line
                 logLine = logLine.Substring(timeMatch.Values.First().Length);
             }
@@ -359,7 +348,7 @@ namespace IW4MAdmin.Application.EventParsers
 
                 catch (Exception e)
                 {
-                    _logger.LogError(e, $"Could not handle custom event generation");
+                    _logger.WriteWarning($"Could not handle custom event generation - {e.GetExceptionInfo()}");
                 }
             }
 

Application/EventParsers/DynamicEventParser.cs

@@ -1,6 +1,5 @@
 using SharedLibraryCore.Configuration;
 using SharedLibraryCore.Interfaces;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.EventParsers
 {

Application/Extensions/StartupExtensions.cs

@@ -1,101 +0,0 @@
-using System;
-using System.IO;
-using System.Runtime.InteropServices;
-using Microsoft.Data.Sqlite;
-using Microsoft.EntityFrameworkCore;
-using Microsoft.Extensions.Configuration;
-using Microsoft.Extensions.DependencyInjection;
-using Microsoft.Extensions.Logging;
-using Serilog;
-using Serilog.Events;
-using SharedLibraryCore;
-using SharedLibraryCore.Configuration;
-using SharedLibraryCore.Database.MigrationContext;
-using ILogger = Serilog.ILogger;
-
-namespace IW4MAdmin.Application.Extensions
-{
-    public static class StartupExtensions
-    {
-        private static ILogger _defaultLogger = null;
-
-        public static IServiceCollection AddBaseLogger(this IServiceCollection services,
-            ApplicationConfiguration appConfig)
-        {
-            if (_defaultLogger == null)
-            {
-                var configuration = new ConfigurationBuilder()
-                    .AddJsonFile(Path.Join(Utilities.OperatingDirectory, "Configuration", "LoggingConfiguration.json"))
-                    .Build();
-
-                var loggerConfig = new LoggerConfiguration()
-                    .ReadFrom.Configuration(configuration)
-                    .MinimumLevel.Override("Microsoft", LogEventLevel.Warning);
-
-
-                if (Utilities.IsDevelopment)
-                {
-                    loggerConfig = loggerConfig.WriteTo.Console(
-                            outputTemplate:
-                            "[{Timestamp:yyyy-MM-dd HH:mm:ss.fff} {Server} {Level:u3}] {Message:lj}{NewLine}{Exception}")
-                        .MinimumLevel.Override("Microsoft", LogEventLevel.Warning)
-                        .MinimumLevel.Debug();
-                }
-
-                _defaultLogger = loggerConfig.CreateLogger();
-            }
-
-            services.AddLogging(builder => builder.AddSerilog(_defaultLogger, dispose: true));
-            services.AddSingleton(new LoggerFactory()
-                .AddSerilog(_defaultLogger, true));
-            return services;
-        }
-
-        public static IServiceCollection AddDatabaseContextOptions(this IServiceCollection services,
-            ApplicationConfiguration appConfig)
-        {
-            var activeProvider = appConfig.DatabaseProvider?.ToLower();
-
-            if (string.IsNullOrEmpty(appConfig.ConnectionString) || activeProvider == "sqlite")
-            {
-                var currentPath = Utilities.OperatingDirectory;
-                currentPath = !RuntimeInformation.IsOSPlatform(OSPlatform.Windows)
-                    ? $"{Path.DirectorySeparatorChar}{currentPath}"
-                    : currentPath;
-
-                var connectionStringBuilder = new SqliteConnectionStringBuilder
-                    {DataSource = Path.Join(currentPath, "Database", "Database.db")};
-                var connectionString = connectionStringBuilder.ToString();
-
-                var builder = new DbContextOptionsBuilder<SqliteDatabaseContext>()
-                    .UseSqlite(connectionString);
-
-                services.AddSingleton((DbContextOptions) builder.Options);
-                return services;
-            }
-
-            switch (activeProvider)
-            {
-                case "mysql":
-                    var appendTimeout = !appConfig.ConnectionString.Contains("default command timeout",
-                        StringComparison.InvariantCultureIgnoreCase);
-                    services.AddSingleton(sp => (DbContextOptions) new DbContextOptionsBuilder<MySqlDatabaseContext>()
-                        .UseMySql(appConfig.ConnectionString + (appendTimeout ? ";default command timeout=0" : ""),
-                            mysqlOptions => mysqlOptions.EnableRetryOnFailure())
-                        .UseLoggerFactory(sp.GetRequiredService<ILoggerFactory>()).Options);
-                    return services;
-                case "postgresql":
-                    appendTimeout = !appConfig.ConnectionString.Contains("Command Timeout",
-                        StringComparison.InvariantCultureIgnoreCase);
-                    services.AddSingleton(sp =>
-                        (DbContextOptions) new DbContextOptionsBuilder<PostgresqlDatabaseContext>()
-                            .UseNpgsql(appConfig.ConnectionString + (appendTimeout ? ";Command Timeout=0" : ""),
-                                postgresqlOptions => postgresqlOptions.EnableRetryOnFailure())
-                            .UseLoggerFactory(sp.GetRequiredService<ILoggerFactory>()).Options);
-                    return services;
-                default:
-                    throw new ArgumentException($"No context available for {appConfig.DatabaseProvider}");
-            }
-        }
-    }
-}

Application/Factories/DatabaseContextFactory.cs

@@ -1,8 +1,4 @@
-using System;
+using SharedLibraryCore.Database;
-using Microsoft.EntityFrameworkCore;
-using SharedLibraryCore.Configuration;
-using SharedLibraryCore.Database;
-using SharedLibraryCore.Database.MigrationContext;
 using SharedLibraryCore.Interfaces;
 
 namespace IW4MAdmin.Application.Factories
@@ -12,15 +8,6 @@ namespace IW4MAdmin.Application.Factories
     /// </summary>
     public class DatabaseContextFactory : IDatabaseContextFactory
     {
-        private readonly DbContextOptions _contextOptions;
-        private readonly string _activeProvider;
-
-        public DatabaseContextFactory(ApplicationConfiguration appConfig, DbContextOptions contextOptions)
-        {
-            _contextOptions = contextOptions;
-            _activeProvider = appConfig.DatabaseProvider?.ToLower();
-        }
-
         /// <summary>
         /// creates a new database context
         /// </summary>
@@ -28,35 +15,7 @@ namespace IW4MAdmin.Application.Factories
         /// <returns></returns>
         public DatabaseContext CreateContext(bool? enableTracking = true)
         {
-            var context = BuildContext();
+            return enableTracking.HasValue ? new DatabaseContext(disableTracking: !enableTracking.Value) : new DatabaseContext();
-
-            enableTracking ??= true;
-
-            if (enableTracking.Value)
-            {
-                context.ChangeTracker.AutoDetectChangesEnabled = true;
-                context.ChangeTracker.LazyLoadingEnabled = true;
-                context.ChangeTracker.QueryTrackingBehavior = QueryTrackingBehavior.TrackAll;
-            }
-            else
-            {
-                context.ChangeTracker.AutoDetectChangesEnabled = false;
-                context.ChangeTracker.LazyLoadingEnabled = false;
-                context.ChangeTracker.QueryTrackingBehavior = QueryTrackingBehavior.NoTracking;
-            }
-
-            return context;
-        }
-
-        private DatabaseContext BuildContext()
-        {
-            return _activeProvider switch
-            {
-                "sqlite" => new SqliteDatabaseContext(_contextOptions),
-                "mysql" => new MySqlDatabaseContext(_contextOptions),
-                "postgresql" => new PostgresqlDatabaseContext(_contextOptions),
-                _ => throw new ArgumentException($"No context found for {_activeProvider}")
-            };
         }
     }
 }

Application/Factories/GameLogReaderFactory.cs

@@ -2,7 +2,6 @@
 using Microsoft.Extensions.DependencyInjection;
 using SharedLibraryCore.Interfaces;
 using System;
-using Microsoft.Extensions.Logging;
 
 namespace IW4MAdmin.Application.Factories
 {
@@ -20,12 +19,12 @@ namespace IW4MAdmin.Application.Factories
             var baseUri = logUris[0];
             if (baseUri.Scheme == Uri.UriSchemeHttp)
             {
-                return new GameLogReaderHttp(logUris, eventParser, _serviceProvider.GetRequiredService<ILogger<GameLogReaderHttp>>());
+                return new GameLogReaderHttp(logUris, eventParser, _serviceProvider.GetRequiredService<ILogger>());
             }
 
             else if (baseUri.Scheme == Uri.UriSchemeFile)
             {
-                return new GameLogReader(baseUri.LocalPath, eventParser, _serviceProvider.GetRequiredService<ILogger<GameLogReader>>());
+                return new GameLogReader(baseUri.LocalPath, eventParser, _serviceProvider.GetRequiredService<ILogger>());
             }
 
             throw new NotImplementedException($"No log reader implemented for Uri scheme \"{baseUri.Scheme}\"");

Application/Factories/GameServerInstanceFactory.cs

@@ -1,8 +1,7 @@
-using System;
+using SharedLibraryCore;
-using Microsoft.Extensions.DependencyInjection;
-using SharedLibraryCore;
 using SharedLibraryCore.Configuration;
 using SharedLibraryCore.Interfaces;
+using System.Collections;
 
 namespace IW4MAdmin.Application.Factories
 {
@@ -12,21 +11,21 @@ namespace IW4MAdmin.Application.Factories
     internal class GameServerInstanceFactory : IGameServerInstanceFactory
     {
         private readonly ITranslationLookup _translationLookup;
+        private readonly IRConConnectionFactory _rconConnectionFactory;
+        private readonly IGameLogReaderFactory _gameLogReaderFactory;
         private readonly IMetaService _metaService;
-        private readonly IServiceProvider _serviceProvider;
 
         /// <summary>
         /// base constructor
         /// </summary>
         /// <param name="translationLookup"></param>
         /// <param name="rconConnectionFactory"></param>
-        public GameServerInstanceFactory(ITranslationLookup translationLookup, 
+        public GameServerInstanceFactory(ITranslationLookup translationLookup, IRConConnectionFactory rconConnectionFactory, IGameLogReaderFactory gameLogReaderFactory, IMetaService metaService)
-            IMetaService metaService,
-            IServiceProvider serviceProvider)
         {
             _translationLookup = translationLookup;
+            _rconConnectionFactory = rconConnectionFactory;
+            _gameLogReaderFactory = gameLogReaderFactory;
             _metaService = metaService;
-            _serviceProvider = serviceProvider;
         }
 
         /// <summary>
@@ -37,7 +36,7 @@ namespace IW4MAdmin.Application.Factories
         /// <returns></returns>
         public Server CreateServer(ServerConfiguration config, IManager manager)
         {
-            return new IW4MServer(config, _translationLookup, _metaService, _serviceProvider, _serviceProvider.GetRequiredService<IClientNoticeMessageFormatter>());
+            return new IW4MServer(manager, config, _translationLookup, _rconConnectionFactory, _gameLogReaderFactory, _metaService);
         }
     }
 }

Application/Factories/RConConnectionFactory.cs

@@ -1,7 +1,6 @@
 using IW4MAdmin.Application.RCon;
 using SharedLibraryCore.Interfaces;
 using System.Text;
-using Microsoft.Extensions.Logging;
 
 namespace IW4MAdmin.Application.Factories
 {
@@ -11,13 +10,13 @@ namespace IW4MAdmin.Application.Factories
     internal class RConConnectionFactory : IRConConnectionFactory
     {
         private static readonly Encoding gameEncoding = Encoding.GetEncoding("windows-1252");
-        private readonly ILogger<RConConnection> _logger;
+        private readonly ILogger _logger;
        
         /// <summary>
         /// Base constructor
         /// </summary>
         /// <param name="logger"></param>
-        public RConConnectionFactory(ILogger<RConConnection> logger)
+        public RConConnectionFactory(ILogger logger)
         {
             _logger = logger;
         }

Application/Factories/ScriptCommandFactory.cs

@@ -6,8 +6,6 @@ using SharedLibraryCore.Interfaces;
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using Microsoft.Extensions.DependencyInjection;
-using Microsoft.Extensions.Logging;
 using static SharedLibraryCore.Database.Models.EFClient;
 
 namespace IW4MAdmin.Application.Factories
@@ -17,20 +15,17 @@ namespace IW4MAdmin.Application.Factories
     /// </summary>
     public class ScriptCommandFactory : IScriptCommandFactory
     {
-        private readonly CommandConfiguration _config;
+        private CommandConfiguration _config;
         private readonly ITranslationLookup _transLookup;
-        private readonly IServiceProvider _serviceProvider;
 
-        public ScriptCommandFactory(CommandConfiguration  config, ITranslationLookup transLookup, IServiceProvider serviceProvider)
+        public ScriptCommandFactory(CommandConfiguration  config, ITranslationLookup transLookup)
         {
             _config = config;
             _transLookup = transLookup;
-            _serviceProvider = serviceProvider;
         }
 
         /// <inheritdoc/>
-        public IManagerCommand CreateScriptCommand(string name, string alias, string description, string permission, 
+        public IManagerCommand CreateScriptCommand(string name, string alias, string description, string permission, bool isTargetRequired, IEnumerable<(string, bool)> args, Action<GameEvent> executeAction)
-            bool isTargetRequired, IEnumerable<(string, bool)> args, Action<GameEvent> executeAction)
         {
             var permissionEnum = Enum.Parse<Permission>(permission);
             var argsArray = args.Select(_arg => new CommandArgument
@@ -39,8 +34,7 @@ namespace IW4MAdmin.Application.Factories
                 Required = _arg.Item2
             }).ToArray();
 
-            return new ScriptCommand(name, alias, description, isTargetRequired, permissionEnum, argsArray, executeAction,
+            return new ScriptCommand(name, alias, description, isTargetRequired, permissionEnum, argsArray, executeAction, _config, _transLookup);
-                _config, _transLookup, _serviceProvider.GetRequiredService<ILogger<ScriptCommand>>());
         }
     }
 }

Application/GameEventHandler.cs

@@ -1,18 +1,19 @@
 using IW4MAdmin.Application.Misc;
+using Newtonsoft.Json;
 using SharedLibraryCore;
 using SharedLibraryCore.Events;
 using SharedLibraryCore.Interfaces;
+using System;
+using System.Collections.Generic;
 using System.Linq;
+using System.Threading;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application
 {
     public class GameEventHandler : IEventHandler
     {
         private readonly EventLog _eventLog;
-        private readonly ILogger _logger;
         private static readonly GameEvent.EventType[] overrideEvents = new[]
         {
             GameEvent.EventType.Connect,
@@ -21,23 +22,34 @@ namespace IW4MAdmin.Application
             GameEvent.EventType.Stop
         };
 
-        public GameEventHandler(ILogger<GameEventHandler> logger)
+        public GameEventHandler()
         {
             _eventLog = new EventLog();
-            _logger = logger;
         }
 
         public void HandleEvent(IManager manager, 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
             if (manager.IsRunning || overrideEvents.Contains(gameEvent.Type))
             {
+#if DEBUG
+                gameEvent.Owner.Logger.WriteDebug($"Adding event with id {gameEvent.Id}");
+#endif
+
                 EventApi.OnGameEvent(gameEvent);
                 Task.Factory.StartNew(() => manager.ExecuteEvent(gameEvent));
             }
+#if DEBUG
             else
             {
-                _logger.LogDebug("Skipping event as we're shutting down {eventId}", gameEvent.Id);
+                gameEvent.Owner.Logger.WriteDebug($"Skipping event as we're shutting down {gameEvent.Id}");
             }
+#endif
         }
     }
 }

Application/IO/GameLogEventDetection.cs

@@ -3,9 +3,6 @@ using SharedLibraryCore.Interfaces;
 using System;
 using System.Linq;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using Serilog.Context;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.IO
 {
@@ -15,14 +12,12 @@ namespace IW4MAdmin.Application.IO
         private readonly Server _server;
         private readonly IGameLogReader _reader;
         private readonly bool _ignoreBots;
-        private readonly ILogger _logger;
 
-        public GameLogEventDetection(ILogger<GameLogEventDetection> logger, IW4MServer server, Uri[] gameLogUris, IGameLogReaderFactory gameLogReaderFactory)
+        public GameLogEventDetection(Server server, Uri[] gameLogUris, IGameLogReaderFactory gameLogReaderFactory)
         {
             _reader = gameLogReaderFactory.CreateGameLogReader(gameLogUris, server.EventParser);
             _server = server;
             _ignoreBots = server?.Manager.GetApplicationSettings().Configuration().IgnoreBots ?? false;
-            _logger = logger;
         }
 
         public async Task PollForChanges()
@@ -38,17 +33,15 @@ namespace IW4MAdmin.Application.IO
 
                     catch (Exception e)
                     {
-                        using(LogContext.PushProperty("Server", _server.ToString()))
+                        _server.Logger.WriteWarning($"Failed to update log event for {_server.EndPoint}");
-                        {
+                        _server.Logger.WriteDebug(e.GetExceptionInfo());
-                            _logger.LogError(e, "Failed to update log event for {endpoint}", _server.EndPoint);
-                        }
                     }
                 }
 
                 await Task.Delay(_reader.UpdateInterval, _server.Manager.CancellationToken);
             }
 
-            _logger.LogDebug("Stopped polling for changes");
+            _server.Logger.WriteDebug("Stopped polling for changes");
         }
 
         public async Task UpdateLogEvents()
@@ -75,6 +68,9 @@ namespace IW4MAdmin.Application.IO
             {
                 try
                 {
+#if DEBUG
+                    _server.Logger.WriteVerbose(gameEvent.Data);
+#endif
                     gameEvent.Owner = _server;
 
                     // we don't want to add the event if ignoreBots is on and the event comes from a bot
@@ -106,14 +102,10 @@ namespace IW4MAdmin.Application.IO
 
                 catch (InvalidOperationException)
                 {
-                    if (_ignoreBots)
+                    if (!_ignoreBots)
                     {
-                        continue;
+                        _server.Logger.WriteWarning("Could not find client in client list when parsing event line");
-                    }
+                        _server.Logger.WriteDebug(gameEvent.Data);
-                    
-                    using(LogContext.PushProperty("Server", _server.ToString()))
-                    {
-                        _logger.LogError("Could not find client in client list when parsing event line {data}", gameEvent.Data);
                     }
                 }
             }

Application/IO/GameLogReader.cs

@@ -6,8 +6,6 @@ using System.IO;
 using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.IO
 {
@@ -21,7 +19,7 @@ namespace IW4MAdmin.Application.IO
 
         public int UpdateInterval => 300;
 
-        public GameLogReader(string logFile, IEventParser parser, ILogger<GameLogReader> logger)
+        public GameLogReader(string logFile, IEventParser parser, ILogger logger)
         {
             _logFile = logFile;
             _parser = parser;
@@ -75,7 +73,9 @@ namespace IW4MAdmin.Application.IO
 
                 catch (Exception e)
                 {
-                    _logger.LogError(e, "Could not properly parse event line {@eventLine}", eventLine);
+                    _logger.WriteWarning("Could not properly parse event line");
+                    _logger.WriteDebug(e.Message);
+                    _logger.WriteDebug(eventLine);
                 }
             }
 

Application/IO/GameLogReaderHttp.cs

@@ -6,8 +6,6 @@ using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.IO
 {
@@ -22,7 +20,7 @@ namespace IW4MAdmin.Application.IO
         private readonly string _safeLogPath;
         private string lastKey = "next";
 
-        public GameLogReaderHttp(Uri[] gameLogServerUris, IEventParser parser, ILogger<GameLogReaderHttp> logger)
+        public GameLogReaderHttp(Uri[] gameLogServerUris, IEventParser parser, ILogger logger)
         {
             _eventParser = parser;
             _logServerApi = RestClient.For<IGameLogServer>(gameLogServerUris[0].ToString());
@@ -42,7 +40,7 @@ namespace IW4MAdmin.Application.IO
 
             if (!response.Success && string.IsNullOrEmpty(lastKey))
             {
-                _logger.LogError("Could not get log server info of {logPath}", _safeLogPath);
+                _logger.WriteError($"Could not get log server info of {_safeLogPath}");
                 return events;
             }
 
@@ -64,7 +62,9 @@ namespace IW4MAdmin.Application.IO
 
                     catch (Exception e)
                     {
-                        _logger.LogError(e, "Could not properly parse event line from http {eventLine}", eventLine);
+                        _logger.WriteError("Could not properly parse event line from http");
+                        _logger.WriteDebug(e.Message);
+                        _logger.WriteDebug(eventLine);
                     }
                 }
             }

Application/Localization/Configure.cs

@@ -6,22 +6,15 @@ using System.Collections.Generic;
 using System.Globalization;
 using System.IO;
 using System.Text;
-using Microsoft.Extensions.Logging;
-using SharedLibraryCore.Configuration;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Localization
 {
-    public static class Configure
+    public class Configure
     {
-        public static ITranslationLookup Initialize(ILogger logger, IMasterApi apiInstance, ApplicationConfiguration applicationConfiguration)
+        public static ITranslationLookup Initialize(bool useLocalTranslation, IMasterApi apiInstance, string customLocale = null)
         {
-            var useLocalTranslation = applicationConfiguration?.UseLocalTranslations ?? true;
+            string currentLocale = string.IsNullOrEmpty(customLocale) ? CultureInfo.CurrentCulture.Name : customLocale;
-            var customLocale = applicationConfiguration?.EnableCustomLocale ?? false
+            string[] localizationFiles = Directory.GetFiles(Path.Join(Utilities.OperatingDirectory, "Localization"), $"*.{currentLocale}.json");
-                ? (applicationConfiguration.CustomLocale ?? "en-US")
-                : "en-US";
-            var currentLocale = string.IsNullOrEmpty(customLocale) ? CultureInfo.CurrentCulture.Name : customLocale;
-            var localizationFiles = Directory.GetFiles(Path.Join(Utilities.OperatingDirectory, "Localization"), $"*.{currentLocale}.json");
 
             if (!useLocalTranslation)
             {
@@ -32,10 +25,9 @@ namespace IW4MAdmin.Application.Localization
                     return localization.LocalizationIndex;
                 }
 
-                catch (Exception ex)
+                catch (Exception)
                 {
                     // the online localization failed so will default to local files
-                    logger.LogWarning(ex, "Could not download latest translations");
                 }
             }
 
@@ -63,20 +55,18 @@ namespace IW4MAdmin.Application.Localization
             {
                 var localizationContents = File.ReadAllText(filePath, Encoding.UTF8);
                 var eachLocalizationFile = Newtonsoft.Json.JsonConvert.DeserializeObject<SharedLibraryCore.Localization.Layout>(localizationContents);
-                if (eachLocalizationFile == null)
-                {
-                    continue;
-                }
 
                 foreach (var item in eachLocalizationFile.LocalizationIndex.Set)
                 {
                     if (!localizationDict.TryAdd(item.Key, item.Value))
                     {
-                        logger.LogError("Could not add locale string {key} to localization", item.Key);
+                        Program.ServerManager.GetLogger(0).WriteError($"Could not add locale string {item.Key} to localization");
                     }
                 }
             }
 
+            string localizationFile = $"{Path.Join(Utilities.OperatingDirectory, "Localization")}{Path.DirectorySeparatorChar}IW4MAdmin.{currentLocale}-{currentLocale.ToUpper()}.json";
+
             Utilities.CurrentLocalization = new SharedLibraryCore.Localization.Layout(localizationDict)
             {
                 LocalizationName = currentLocale,

Application/Main.cs

@@ -1,6 +1,7 @@
 using IW4MAdmin.Application.API.Master;
 using IW4MAdmin.Application.EventParsers;
 using IW4MAdmin.Application.Factories;
+using IW4MAdmin.Application.Helpers;
 using IW4MAdmin.Application.Meta;
 using IW4MAdmin.Application.Migration;
 using IW4MAdmin.Application.Misc;
@@ -23,10 +24,6 @@ using System.Linq;
 using System.Text;
 using System.Threading;
 using System.Threading.Tasks;
-using IW4MAdmin.Application.Extensions;
-using IW4MAdmin.Application.Localization;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application
 {
@@ -68,10 +65,7 @@ namespace IW4MAdmin.Application
         private static async void OnCancelKey(object sender, ConsoleCancelEventArgs e)
         {
             ServerManager?.Stop();
-            if (ApplicationTask != null)
+            await ApplicationTask;
-            {
-                await ApplicationTask;
-            }
         }
 
         /// <summary>
@@ -82,33 +76,29 @@ namespace IW4MAdmin.Application
         {
         restart:
             ITranslationLookup translationLookup = null;
-            var logger = BuildDefaultLogger<Program>(new ApplicationConfiguration());
-            Utilities.DefaultLogger = logger;
-            IServiceCollection services = null;
-            logger.LogInformation("Begin IW4MAdmin startup. Version is {version} {@args}", Version, args);
-            
             try
             {
                 // do any needed housekeeping file/folder migrations
                 ConfigurationMigration.MoveConfigFolder10518(null);
                 ConfigurationMigration.CheckDirectories();
-                logger.LogDebug("Configuring services...");
+
-                services = ConfigureServices(args);
+                var services = ConfigureServices(args);
                 serviceProvider = services.BuildServiceProvider();
                 var versionChecker = serviceProvider.GetRequiredService<IMasterCommunication>();
                 ServerManager = (ApplicationManager)serviceProvider.GetRequiredService<IManager>();
                 translationLookup = serviceProvider.GetRequiredService<ITranslationLookup>();
 
+                ServerManager.Logger.WriteInfo(Utilities.CurrentLocalization.LocalizationIndex["MANAGER_VERSION"].FormatExt(Version));
+
                 await versionChecker.CheckVersion();
                 await ServerManager.Init();
             }
 
             catch (Exception e)
             {
-                string failMessage = translationLookup == null ? "Failed to initialize IW4MAdmin" : translationLookup["MANAGER_INIT_FAIL"];
+                string failMessage = translationLookup == null ? "Failed to initalize IW4MAdmin" : translationLookup["MANAGER_INIT_FAIL"];
                 string exitMessage = translationLookup == null ? "Press enter to exit..." : translationLookup["MANAGER_EXIT"];
 
-                logger.LogCritical(e, "Failed to initialize IW4MAdmin");
                 Console.WriteLine(failMessage);
 
                 while (e.InnerException != null)
@@ -141,14 +131,13 @@ namespace IW4MAdmin.Application
 
             try
             {
-                ApplicationTask = RunApplicationTasksAsync(logger, services);
+                ApplicationTask = RunApplicationTasksAsync();
                 await ApplicationTask;
             }
 
             catch (Exception e) 
             {
-                logger.LogCritical(e, "Failed to launch IW4MAdmin");
+                string failMessage = translationLookup == null ? "Failed to initalize IW4MAdmin" : translationLookup["MANAGER_INIT_FAIL"];
-                string failMessage = translationLookup == null ? "Failed to launch IW4MAdmin" : translationLookup["MANAGER_INIT_FAIL"];
                 Console.WriteLine($"{failMessage}: {e.GetExceptionInfo()}");
             }
 
@@ -164,15 +153,15 @@ namespace IW4MAdmin.Application
         /// runs the core application tasks
         /// </summary>
         /// <returns></returns>
-        private static async Task RunApplicationTasksAsync(ILogger logger, IServiceCollection services)
+        private static async Task RunApplicationTasksAsync()
         {
             var webfrontTask = ServerManager.GetApplicationSettings().Configuration().EnableWebFront ?
-                WebfrontCore.Program.Init(ServerManager, serviceProvider, services, ServerManager.CancellationToken) :
+                WebfrontCore.Program.Init(ServerManager, serviceProvider, ServerManager.CancellationToken) :
                 Task.CompletedTask;
 
             // we want to run this one on a manual thread instead of letting the thread pool handle it,
             // because we can't exit early from waiting on console input, and it prevents us from restarting
-            var inputThread = new Thread(async () => await ReadConsoleInput(logger));
+            var inputThread = new Thread(async () => await ReadConsoleInput());
             inputThread.Start();
 
             var tasks = new[]
@@ -182,11 +171,9 @@ namespace IW4MAdmin.Application
                 serviceProvider.GetRequiredService<IMasterCommunication>().RunUploadStatus(ServerManager.CancellationToken)
             };
 
-            logger.LogDebug("Starting webfront and input tasks");
             await Task.WhenAll(tasks);
 
-            logger.LogInformation("Shutdown completed successfully");
+            ServerManager.Logger.WriteVerbose(Utilities.CurrentLocalization.LocalizationIndex["MANAGER_SHUTDOWN_SUCCESS"]);
-            Console.Write(Utilities.CurrentLocalization.LocalizationIndex["MANAGER_SHUTDOWN_SUCCESS"]);
         }
 
 
@@ -194,11 +181,11 @@ namespace IW4MAdmin.Application
         /// reads input from the console and executes entered commands on the default server
         /// </summary>
         /// <returns></returns>
-        private static async Task ReadConsoleInput(ILogger logger)
+        private static async Task ReadConsoleInput()
         {
             if (Console.IsInputRedirected)
             {
-                logger.LogInformation("Disabling console input as it has been redirected");
+                ServerManager.Logger.WriteInfo("Disabling console input as it has been redirected");
                 return;
             }
 
@@ -234,29 +221,68 @@ namespace IW4MAdmin.Application
             { }
         }
 
-        private static IServiceCollection HandlePluginRegistration(ApplicationConfiguration appConfig, 
+        /// <summary>
-            IServiceCollection serviceCollection, 
+        /// Configures the dependency injection services
-            IMasterApi masterApi)
+        /// </summary>
+        private static IServiceCollection ConfigureServices(string[] args)
         {
-            var defaultLogger = BuildDefaultLogger<Program>(appConfig);
+            var defaultLogger = new Logger("IW4MAdmin-Manager");
-            var pluginServiceProvider = new ServiceCollection()
+            var pluginImporter = new PluginImporter(defaultLogger);
-                .AddBaseLogger(appConfig)
+
-                .AddSingleton(appConfig)
+            var serviceCollection = new ServiceCollection();
-                .AddSingleton(masterApi)
+            serviceCollection.AddSingleton<IServiceCollection>(_serviceProvider => serviceCollection)
-                .AddSingleton<IRemoteAssemblyHandler, RemoteAssemblyHandler>()
+                .AddSingleton(new BaseConfigurationHandler<ApplicationConfiguration>("IW4MAdminSettings") as IConfigurationHandler<ApplicationConfiguration>)
+                .AddSingleton(new BaseConfigurationHandler<CommandConfiguration>("CommandConfiguration") as IConfigurationHandler<CommandConfiguration>)
+                .AddSingleton(_serviceProvider => _serviceProvider.GetRequiredService<IConfigurationHandler<ApplicationConfiguration>>().Configuration() ?? new ApplicationConfiguration())
+                .AddSingleton(_serviceProvider => _serviceProvider.GetRequiredService<IConfigurationHandler<CommandConfiguration>>().Configuration() ?? new CommandConfiguration())
+                .AddSingleton<ILogger>(_serviceProvider => defaultLogger)
                 .AddSingleton<IPluginImporter, PluginImporter>()
-                .BuildServiceProvider();
+                .AddSingleton<IMiddlewareActionHandler, MiddlewareActionHandler>()
+                .AddSingleton<IRConConnectionFactory, RConConnectionFactory>()
+                .AddSingleton<IGameServerInstanceFactory, GameServerInstanceFactory>()
+                .AddSingleton<IConfigurationHandlerFactory, ConfigurationHandlerFactory>()
+                .AddSingleton<IParserRegexFactory, ParserRegexFactory>()
+                .AddSingleton<IDatabaseContextFactory, DatabaseContextFactory>()
+                .AddSingleton<IGameLogReaderFactory, GameLogReaderFactory>()
+                .AddSingleton<IScriptCommandFactory, ScriptCommandFactory>()
+                .AddSingleton<IAuditInformationRepository, AuditInformationRepository>()
+                .AddSingleton<IEntityService<EFClient>, ClientService>()
+                .AddSingleton<IMetaService, MetaService>()
+                .AddSingleton<IMetaRegistration, MetaRegistration>()
+                .AddSingleton<IScriptPluginServiceResolver, ScriptPluginServiceResolver>()
+                .AddSingleton<IResourceQueryHelper<ClientPaginationRequest, ReceivedPenaltyResponse>, ReceivedPenaltyResourceQueryHelper>()
+                .AddSingleton<IResourceQueryHelper<ClientPaginationRequest, AdministeredPenaltyResponse>, AdministeredPenaltyResourceQueryHelper>()
+                .AddSingleton<IResourceQueryHelper<ClientPaginationRequest, UpdatedAliasResponse>, UpdatedAliasResourceQueryHelper>()
+                .AddSingleton<IResourceQueryHelper<ChatSearchQuery, MessageResponse>, ChatResourceQueryHelper>()
+                .AddTransient<IParserPatternMatcher, ParserPatternMatcher>()
+                .AddSingleton(_serviceProvider =>
+                {
+                    var config = _serviceProvider.GetRequiredService<IConfigurationHandler<ApplicationConfiguration>>().Configuration();
+                    return Localization.Configure.Initialize(useLocalTranslation: config?.UseLocalTranslations ?? false,
+                        apiInstance: _serviceProvider.GetRequiredService<IMasterApi>(),
+                        customLocale: config?.EnableCustomLocale ?? false ? (config.CustomLocale ?? "en-US") : "en-US");
+                })
+                .AddSingleton<IManager, ApplicationManager>()
+                .AddSingleton(_serviceProvider => RestClient
+                    .For<IMasterApi>(Utilities.IsDevelopment ? new Uri("http://127.0.0.1:8080") : _serviceProvider
+                    .GetRequiredService<IConfigurationHandler<ApplicationConfiguration>>().Configuration()?.MasterUrl ??
+                    new ApplicationConfiguration().MasterUrl))
+                .AddSingleton<IMasterCommunication, MasterCommunication>();
 
-            var pluginImporter = pluginServiceProvider.GetRequiredService<IPluginImporter>();
+            if (args.Contains("serialevents"))
-            
+            {
-            // we need to register the rest client with regular collection
+                serviceCollection.AddSingleton<IEventHandler, SerialGameEventHandler>();
-            serviceCollection.AddSingleton(masterApi);
+            }
+            else
+            {
+                serviceCollection.AddSingleton<IEventHandler, GameEventHandler>();
+            }
 
             // register the native commands
             foreach (var commandType in typeof(SharedLibraryCore.Commands.QuitCommand).Assembly.GetTypes()
                         .Where(_command => _command.BaseType == typeof(Command)))
             {
-                defaultLogger.LogDebug("Registered native command type {name}", commandType.Name);
+                defaultLogger.WriteInfo($"Registered native command type {commandType.Name}");
                 serviceCollection.AddSingleton(typeof(IManagerCommand), commandType);
             }
 
@@ -264,14 +290,14 @@ namespace IW4MAdmin.Application
             var pluginImplementations = pluginImporter.DiscoverAssemblyPluginImplementations();
             foreach (var pluginType in pluginImplementations.Item1)
             {
-                defaultLogger.LogDebug("Registered plugin type {name}", pluginType.FullName);
+                defaultLogger.WriteInfo($"Registered plugin type {pluginType.FullName}");
                 serviceCollection.AddSingleton(typeof(IPlugin), pluginType);
             }
 
             // register the plugin commands
             foreach (var commandType in pluginImplementations.Item2)
             {
-                defaultLogger.LogDebug("Registered plugin command type {name}", commandType.FullName);
+                defaultLogger.WriteInfo($"Registered plugin command type {commandType.FullName}");
                 serviceCollection.AddSingleton(typeof(IManagerCommand), commandType);
             }
 
@@ -295,96 +321,5 @@ namespace IW4MAdmin.Application
 
             return serviceCollection;
         }
-        
-
-        /// <summary>
-        /// Configures the dependency injection services
-        /// </summary>
-        private static IServiceCollection ConfigureServices(string[] args)
-        {
-            // setup the static resources (config/master api/translations)
-            var serviceCollection = new ServiceCollection();
-            var appConfigHandler = new BaseConfigurationHandler<ApplicationConfiguration>("IW4MAdminSettings");
-            var appConfig = appConfigHandler.Configuration();
-            var masterUri = Utilities.IsDevelopment
-                ? new Uri("http://127.0.0.1:8080")
-                : appConfig?.MasterUrl ?? new ApplicationConfiguration().MasterUrl;
-            var masterRestClient = RestClient.For<IMasterApi>(masterUri);
-            var translationLookup =  Configure.Initialize(Utilities.DefaultLogger, masterRestClient, appConfig);
-
-            if (appConfig == null)
-            {
-                appConfig = (ApplicationConfiguration) new ApplicationConfiguration().Generate();
-                appConfigHandler.Set(appConfig);
-                appConfigHandler.Save();
-            }
-            
-            // build the dependency list
-            HandlePluginRegistration(appConfig, serviceCollection, masterRestClient);
-
-            serviceCollection
-                .AddBaseLogger(appConfig)
-                .AddSingleton<IServiceCollection>(_serviceProvider => serviceCollection)
-                .AddSingleton((IConfigurationHandler<ApplicationConfiguration>) appConfigHandler)
-                .AddSingleton(
-                    new BaseConfigurationHandler<CommandConfiguration>("CommandConfiguration") as
-                        IConfigurationHandler<CommandConfiguration>)
-                .AddSingleton(appConfig)
-                .AddSingleton(_serviceProvider =>
-                    _serviceProvider.GetRequiredService<IConfigurationHandler<CommandConfiguration>>()
-                        .Configuration() ?? new CommandConfiguration())
-                .AddSingleton<IPluginImporter, PluginImporter>()
-                .AddSingleton<IMiddlewareActionHandler, MiddlewareActionHandler>()
-                .AddSingleton<IRConConnectionFactory, RConConnectionFactory>()
-                .AddSingleton<IGameServerInstanceFactory, GameServerInstanceFactory>()
-                .AddSingleton<IConfigurationHandlerFactory, ConfigurationHandlerFactory>()
-                .AddSingleton<IParserRegexFactory, ParserRegexFactory>()
-                .AddSingleton<IDatabaseContextFactory, DatabaseContextFactory>()
-                .AddSingleton<IGameLogReaderFactory, GameLogReaderFactory>()
-                .AddSingleton<IScriptCommandFactory, ScriptCommandFactory>()
-                .AddSingleton<IAuditInformationRepository, AuditInformationRepository>()
-                .AddSingleton<IEntityService<EFClient>, ClientService>()
-                .AddSingleton<IMetaService, MetaService>()
-                .AddSingleton<ClientService>()
-                .AddSingleton<PenaltyService>()
-                .AddSingleton<ChangeHistoryService>()
-                .AddSingleton<IMetaRegistration, MetaRegistration>()
-                .AddSingleton<IScriptPluginServiceResolver, ScriptPluginServiceResolver>()
-                .AddSingleton<IResourceQueryHelper<ClientPaginationRequest, ReceivedPenaltyResponse>,
-                    ReceivedPenaltyResourceQueryHelper>()
-                .AddSingleton<IResourceQueryHelper<ClientPaginationRequest, AdministeredPenaltyResponse>,
-                    AdministeredPenaltyResourceQueryHelper>()
-                .AddSingleton<IResourceQueryHelper<ClientPaginationRequest, UpdatedAliasResponse>,
-                    UpdatedAliasResourceQueryHelper>()
-                .AddSingleton<IResourceQueryHelper<ChatSearchQuery, MessageResponse>, ChatResourceQueryHelper>()
-                .AddTransient<IParserPatternMatcher, ParserPatternMatcher>()
-                .AddSingleton<IRemoteAssemblyHandler, RemoteAssemblyHandler>()
-                .AddSingleton<IMasterCommunication, MasterCommunication>()
-                .AddSingleton<IManager, ApplicationManager>()
-                .AddSingleton<SharedLibraryCore.Interfaces.ILogger, Logger>()
-                .AddSingleton<IClientNoticeMessageFormatter, ClientNoticeMessageFormatter>()
-                .AddSingleton(translationLookup)
-                .AddDatabaseContextOptions(appConfig);
-
-            if (args.Contains("serialevents"))
-            {
-                serviceCollection.AddSingleton<IEventHandler, SerialGameEventHandler>();
-            }
-            else
-            {
-                serviceCollection.AddSingleton<IEventHandler, GameEventHandler>();
-            }
-
-            return serviceCollection;
-        }
-       
-        private static ILogger BuildDefaultLogger<T>(ApplicationConfiguration appConfig)
-        {
-            var collection = new ServiceCollection()
-                .AddBaseLogger(appConfig)
-                .BuildServiceProvider();
-
-            return collection.GetRequiredService<ILogger<T>>();
-        }
     }
 }

Application/Meta/AdministeredPenaltyResourceQueryHelper.cs

@@ -1,13 +1,11 @@
 using System.Linq;
 using System.Threading.Tasks;
 using Microsoft.EntityFrameworkCore;
-using Microsoft.Extensions.Logging;
 using SharedLibraryCore.Database.Models;
 using SharedLibraryCore.Dtos.Meta.Responses;
 using SharedLibraryCore.Helpers;
 using SharedLibraryCore.Interfaces;
 using SharedLibraryCore.QueryHelper;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Meta
 {
@@ -20,7 +18,7 @@ namespace IW4MAdmin.Application.Meta
         private readonly ILogger _logger;
         private readonly IDatabaseContextFactory _contextFactory;
 
-        public AdministeredPenaltyResourceQueryHelper(ILogger<AdministeredPenaltyResourceQueryHelper> logger, IDatabaseContextFactory contextFactory)
+        public AdministeredPenaltyResourceQueryHelper(ILogger logger, IDatabaseContextFactory contextFactory)
         {
             _contextFactory = contextFactory;
             _logger = logger;
@@ -28,7 +26,7 @@ namespace IW4MAdmin.Application.Meta
 
         public async Task<ResourceQueryHelperResult<AdministeredPenaltyResponse>> QueryResource(ClientPaginationRequest query)
         {
-            await using var ctx = _contextFactory.CreateContext(enableTracking: false);
+            using var ctx = _contextFactory.CreateContext(enableTracking: false);
 
             var iqPenalties = ctx.Penalties.AsNoTracking()
                 .Where(_penalty => query.ClientId == _penalty.PunisherId)

Application/Meta/MetaRegistration.cs

@@ -6,8 +6,6 @@ using SharedLibraryCore.QueryHelper;
 using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Meta
 {
@@ -21,7 +19,7 @@ namespace IW4MAdmin.Application.Meta
         private readonly IResourceQueryHelper<ClientPaginationRequest, AdministeredPenaltyResponse> _administeredPenaltyHelper;
         private readonly IResourceQueryHelper<ClientPaginationRequest, UpdatedAliasResponse> _updatedAliasHelper;
 
-        public MetaRegistration(ILogger<MetaRegistration> logger, IMetaService metaService, ITranslationLookup transLookup, IEntityService<EFClient> clientEntityService,
+        public MetaRegistration(ILogger logger, IMetaService metaService, ITranslationLookup transLookup, IEntityService<EFClient> clientEntityService,
             IResourceQueryHelper<ClientPaginationRequest, ReceivedPenaltyResponse> receivedPenaltyHelper,
             IResourceQueryHelper<ClientPaginationRequest, AdministeredPenaltyResponse> administeredPenaltyHelper,
             IResourceQueryHelper<ClientPaginationRequest, UpdatedAliasResponse> updatedAliasHelper)
@@ -84,7 +82,7 @@ namespace IW4MAdmin.Application.Meta
 
             if (client == null)
             {
-                _logger.LogWarning("No client found with id {clientId} when generating profile meta", request.ClientId);
+                _logger.WriteWarning($"No client found with id {request.ClientId} when generating profile meta");
                 return metaList;
             }
 

Application/Meta/ReceivedPenaltyResourceQueryHelper.cs

@@ -2,14 +2,12 @@
 using System.Linq;
 using System.Threading.Tasks;
 using Microsoft.EntityFrameworkCore;
-using Microsoft.Extensions.Logging;
 using SharedLibraryCore;
 using SharedLibraryCore.Database.Models;
 using SharedLibraryCore.Dtos.Meta.Responses;
 using SharedLibraryCore.Helpers;
 using SharedLibraryCore.Interfaces;
 using SharedLibraryCore.QueryHelper;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Meta
 {
@@ -22,7 +20,7 @@ namespace IW4MAdmin.Application.Meta
         private readonly ILogger _logger;
         private readonly IDatabaseContextFactory _contextFactory;
 
-        public ReceivedPenaltyResourceQueryHelper(ILogger<ReceivedPenaltyResourceQueryHelper> logger, IDatabaseContextFactory contextFactory)
+        public ReceivedPenaltyResourceQueryHelper(ILogger logger, IDatabaseContextFactory contextFactory)
         {
             _contextFactory = contextFactory;
             _logger = logger;
@@ -31,7 +29,7 @@ namespace IW4MAdmin.Application.Meta
         public async Task<ResourceQueryHelperResult<ReceivedPenaltyResponse>> QueryResource(ClientPaginationRequest query)
         {
             var linkedPenaltyType = Utilities.LinkedPenaltyTypes();
-            await using var ctx = _contextFactory.CreateContext(enableTracking: false);
+            using var ctx = _contextFactory.CreateContext(enableTracking: false);
 
             var linkId = await ctx.Clients.AsNoTracking()
                     .Where(_client => _client.ClientId == query.ClientId)

Application/Meta/UpdatedAliasResourceQueryHelper.cs

@@ -6,8 +6,6 @@ using SharedLibraryCore.Interfaces;
 using SharedLibraryCore.QueryHelper;
 using System.Linq;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Meta
 {
@@ -20,7 +18,7 @@ namespace IW4MAdmin.Application.Meta
         private readonly ILogger _logger;
         private readonly IDatabaseContextFactory _contextFactory;
 
-        public UpdatedAliasResourceQueryHelper(ILogger<UpdatedAliasResourceQueryHelper> logger, IDatabaseContextFactory contextFactory)
+        public UpdatedAliasResourceQueryHelper(ILogger logger, IDatabaseContextFactory contextFactory)
         {
             _logger = logger;
             _contextFactory = contextFactory;
@@ -28,7 +26,7 @@ namespace IW4MAdmin.Application.Meta
 
         public async Task<ResourceQueryHelperResult<UpdatedAliasResponse>> QueryResource(ClientPaginationRequest query)
         {
-            await using var ctx = _contextFactory.CreateContext(enableTracking: false);
+            using var ctx = _contextFactory.CreateContext(enableTracking: false);
             int linkId = ctx.Clients.First(_client => _client.ClientId == query.ClientId).AliasLinkId;
 
             var iqAliasUpdates = ctx.Aliases

Application/Migration/ConfigurationMigration.cs

@@ -1,8 +1,11 @@
 using SharedLibraryCore;
+using SharedLibraryCore.Interfaces;
 using System;
+using System.Collections.Generic;
 using System.IO;
 using System.Linq;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
+using System.Text;
+using System.Text.RegularExpressions;
 
 namespace IW4MAdmin.Application.Migration
 {
@@ -53,6 +56,7 @@ namespace IW4MAdmin.Application.Migration
 
                 if (!Directory.Exists(configDirectory))
                 {
+                    log?.WriteDebug($"Creating directory for configs {configDirectory}");
                     Directory.CreateDirectory(configDirectory);
                 }
 
@@ -62,6 +66,7 @@ namespace IW4MAdmin.Application.Migration
 
                 foreach (var configFile in configurationFiles)
                 {
+                    log?.WriteDebug($"Moving config file {configFile}");
                     string destinationPath = Path.Join("Configuration", configFile);
                     if (!File.Exists(destinationPath))
                     {
@@ -72,6 +77,7 @@ namespace IW4MAdmin.Application.Migration
                 if (!File.Exists(Path.Join("Database", "Database.db")) &&
                     File.Exists("Database.db"))
                 {
+                    log?.WriteDebug("Moving database file");
                     File.Move("Database.db", Path.Join("Database", "Database.db"));
                 }
             }

Application/Migration/DatabaseHousekeeping.cs

@@ -1,24 +0,0 @@
-using System;
-using System.Linq;
-using System.Threading;
-using System.Threading.Tasks;
-using IW4MAdmin.Plugins.Stats.Models;
-using SharedLibraryCore.Database;
-using SharedLibraryCore.Interfaces;
-
-namespace IW4MAdmin.Application.Migration
-{
-    public static class DatabaseHousekeeping
-    {
-        private static readonly DateTime CutoffDate = DateTime.UtcNow.AddMonths(-6);
-        
-        public static async Task RemoveOldRatings(IDatabaseContextFactory contextFactory, CancellationToken token)
-        {
-            await using var context = contextFactory.CreateContext();
-            var dbSet = context.Set<EFRating>();
-            var itemsToDelete = dbSet.Where(rating => rating.When <= CutoffDate);
-            dbSet.RemoveRange(itemsToDelete);
-            await context.SaveChangesAsync(token);
-        }
-    }
-}

Application/Misc/ClientNoticeMessageFormatter.cs

@@ -1,120 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using SharedLibraryCore;
-using SharedLibraryCore.Configuration;
-using SharedLibraryCore.Database.Models;
-using SharedLibraryCore.Interfaces;
-
-namespace IW4MAdmin.Application.Misc
-{
-    /// <summary>
-    /// implementation of IClientNoticeMessageFormatter
-    /// </summary>
-    public class ClientNoticeMessageFormatter : IClientNoticeMessageFormatter
-    {
-        private readonly ITranslationLookup _transLookup;
-        private readonly ApplicationConfiguration _appConfig;
-
-        public ClientNoticeMessageFormatter(ITranslationLookup transLookup, ApplicationConfiguration appConfig)
-        {
-            _transLookup = transLookup;
-            _appConfig = appConfig;
-        }
-
-        public string BuildFormattedMessage(IRConParserConfiguration config, EFPenalty currentPenalty, EFPenalty originalPenalty = null)
-        {
-            var isNewLineSeparator = config.NoticeLineSeparator == Environment.NewLine;
-            var penalty = originalPenalty ?? currentPenalty;
-            var builder = new StringBuilder();
-            // build the top level header
-            var header = _transLookup[$"SERVER_{penalty.Type.ToString().ToUpper()}_TEXT"];
-            builder.Append(header);
-            builder.Append(config.NoticeLineSeparator);
-            // build the reason
-            var reason = _transLookup["GAME_MESSAGE_PENALTY_REASON"].FormatExt(penalty.Offense);
-
-            if (isNewLineSeparator)
-            {
-                foreach (var splitReason in SplitOverMaxLength(reason, config.NoticeMaxCharactersPerLine))
-                {
-                    builder.Append(splitReason);
-                    builder.Append(config.NoticeLineSeparator);
-                }
-            }
-
-            else
-            {
-                builder.Append(reason);
-                builder.Append(config.NoticeLineSeparator);
-            }
-
-            if (penalty.Type == EFPenalty.PenaltyType.TempBan)
-            {
-                // build the time remaining if temporary
-                var timeRemainingValue = penalty.Expires.HasValue
-                    ? (penalty.Expires - DateTime.UtcNow).Value.HumanizeForCurrentCulture()
-                    : "--";
-                var timeRemaining = _transLookup["GAME_MESSAGE_PENALTY_TIME_REMAINING"].FormatExt(timeRemainingValue);
-
-                if (isNewLineSeparator)
-                {
-                    foreach (var splitReason in SplitOverMaxLength(timeRemaining, config.NoticeMaxCharactersPerLine))
-                    {
-                        builder.Append(splitReason);
-                        builder.Append(config.NoticeLineSeparator);
-                    }
-                }
-
-                else
-                {
-                    builder.Append(timeRemaining);
-                }
-            }
-
-            if (penalty.Type == EFPenalty.PenaltyType.Ban)
-            {
-                // provide a place to appeal the ban (should always be specified but including a placeholder just incase)
-                builder.Append(_transLookup["GAME_MESSAGE_PENALTY_APPEAL"].FormatExt(_appConfig.ContactUri ?? "--"));
-            }
-            
-            // final format looks something like:
-            /*
-             * You are permanently banned
-             * Reason - toxic behavior
-             * Visit example.com to appeal
-             */
-
-            return builder.ToString();
-        }
-
-        private static IEnumerable<string> SplitOverMaxLength(string source, int maxCharactersPerLine)
-        {
-            if (source.Length <= maxCharactersPerLine)
-            {
-                return new[] {source};
-            }
-
-            var segments = new List<string>();
-            var currentLocation = 0;
-            while (currentLocation < source.Length)
-            {
-                var nextLocation = currentLocation + maxCharactersPerLine;
-                // there's probably a more efficient way to do this but this is readable
-                segments.Add(string.Concat(
-                    source
-                    .Skip(currentLocation)
-                    .Take(Math.Min(maxCharactersPerLine, source.Length - currentLocation))));
-                currentLocation = nextLocation;
-            }
-
-            if (currentLocation < source.Length)
-            {
-                segments.Add(source.Substring(currentLocation, source.Length - currentLocation));
-            }
-
-            return segments;
-        }
-    }
-}

Application/Misc/EventProfiler.cs

@@ -0,0 +1,63 @@
+using SharedLibraryCore;
+using SharedLibraryCore.Interfaces;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+
+namespace IW4MAdmin.Application.Misc
+{
+    internal class EventPerformance
+    {
+        public long ExecutionTime { get; set; }
+        public GameEvent Event { get; set; }
+        public string EventInfo => $"{Event.Type}, {Event.FailReason}, {Event.IsBlocking}, {Event.Data}, {Event.Message}, {Event.Extra}";
+    }
+
+    public class DuplicateKeyComparer<TKey> : IComparer<TKey> where TKey : IComparable
+    {
+        public int Compare(TKey x, TKey y)
+        {
+            int result = x.CompareTo(y);
+
+            if (result == 0)
+                return 1;
+            else
+                return result;
+        }
+    }
+
+    internal class EventProfiler
+    {
+        public double AverageEventTime { get; private set; }
+        public double MaxEventTime => Events.Values.Last().ExecutionTime;
+        public double MinEventTime => Events.Values[0].ExecutionTime;
+        public int TotalEventCount => Events.Count;
+        public SortedList<long, EventPerformance> Events { get; private set; } = new SortedList<long, EventPerformance>(new DuplicateKeyComparer<long>());
+        private readonly ILogger _logger;
+
+        public EventProfiler(ILogger logger)
+        {
+            _logger = logger;
+        }
+
+        public void Profile(DateTime start, DateTime end, GameEvent gameEvent)
+        {
+            _logger.WriteDebug($"Starting profile of event {gameEvent.Id}");
+            long executionTime = (long)Math.Round((end - start).TotalMilliseconds);
+
+            var perf = new EventPerformance()
+            {
+                Event = gameEvent,
+                ExecutionTime = executionTime
+            };
+
+            lock (Events)
+            {
+                Events.Add(executionTime, perf);
+            }
+
+            AverageEventTime = (AverageEventTime * (TotalEventCount - 1) + executionTime) / TotalEventCount;
+            _logger.WriteDebug($"Finished profile of event {gameEvent.Id}");
+        }
+    }
+}

Application/Misc/Logger.cs

@@ -1,47 +1,132 @@
-using System;
+using IW4MAdmin.Application.IO;
-using Microsoft.Extensions.Logging;
+using SharedLibraryCore;
-using ILogger = SharedLibraryCore.Interfaces.ILogger;
+using SharedLibraryCore.Interfaces;
+using System;
+using System.Diagnostics;
+using System.IO;
+using System.Threading;
 
 namespace IW4MAdmin.Application
 {
-    [Obsolete]
     public class Logger : ILogger
     {
-        private readonly Microsoft.Extensions.Logging.ILogger _logger;
+        enum LogType
-
-        public Logger(ILogger<Logger> logger)
         {
-            _logger = logger;
+            Verbose,
+            Info,
+            Debug,
+            Warning,
+            Error,
+            Assert
+        }
+
+        readonly string FileName;
+        readonly ReaderWriterLockSlim WritingLock;
+        static readonly short MAX_LOG_FILES = 10;
+
+        public Logger(string fn)
+        {
+            FileName = Path.Join(Utilities.OperatingDirectory, "Log", $"{fn}.log");
+            WritingLock = new ReaderWriterLockSlim();
+            RotateLogs();
+        }
+
+        ~Logger()
+        {
+            WritingLock.Dispose();
+        }
+
+        /// <summary>
+        /// rotates logs when log is initialized
+        /// </summary>
+        private void RotateLogs()
+        {
+            string maxLog = FileName + MAX_LOG_FILES;
+
+            if (File.Exists(maxLog))
+            {
+                File.Delete(maxLog);
+            }
+
+            for (int i = MAX_LOG_FILES - 1; i >= 0; i--)
+            {
+                string logToMove = i == 0 ? FileName : FileName + i;
+                string movedLogName = FileName + (i + 1);
+
+                if (File.Exists(logToMove))
+                {
+                    File.Move(logToMove, movedLogName);
+                }
+            }
+        }
+
+        void Write(string msg, LogType type)
+        {
+            WritingLock.EnterWriteLock();
+
+            string stringType = type.ToString();
+            msg = msg.StripColors();
+
+            try
+            {
+                stringType = Utilities.CurrentLocalization.LocalizationIndex[$"GLOBAL_{type.ToString().ToUpper()}"];
+            }
+
+            catch (Exception) { }
+
+            string LogLine = $"[{DateTime.Now.ToString("MM.dd.yyy HH:mm:ss.fff")}] - {stringType}: {msg}";
+            try
+            {
+#if DEBUG
+                // lets keep it simple and dispose of everything quickly as logging wont be that much (relatively)
+                Console.WriteLine(msg);
+#else
+                if (type == LogType.Error || type == LogType.Verbose)
+                {
+                    Console.WriteLine(LogLine);
+                }
+                File.AppendAllText(FileName, $"{LogLine}{Environment.NewLine}");
+#endif
+            }
+
+            catch (Exception ex)
+            {
+                Console.WriteLine("Well.. It looks like your machine can't event write to the log file. That's something else...");
+                Console.WriteLine(ex.GetExceptionInfo());
+            }
+
+            WritingLock.ExitWriteLock();
         }
 
         public void WriteVerbose(string msg)
         {
-            _logger.LogInformation(msg);
+            Write(msg, LogType.Verbose);
         }
 
         public void WriteDebug(string msg)
         {
-            _logger.LogDebug(msg);
+            Write(msg, LogType.Debug);
         }
 
         public void WriteError(string msg)
         {
-            _logger.LogError(msg);
+            Write(msg, LogType.Error);
         }
 
         public void WriteInfo(string msg)
         {
-            WriteVerbose(msg);
+            Write(msg, LogType.Info);
         }
 
         public void WriteWarning(string msg)
         {
-            _logger.LogWarning(msg);
+            Write(msg, LogType.Warning);
         }
 
         public void WriteAssert(bool condition, string msg)
         {
-            throw new NotImplementedException();
+            if (!condition)
+                Write(msg, LogType.Assert);
         }
     }
 }

Application/Misc/MasterCommunication.cs

@@ -8,8 +8,6 @@ using System;
 using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Misc
 {
@@ -26,9 +24,10 @@ namespace IW4MAdmin.Application.Misc
         private readonly ApplicationConfiguration _appConfig;
         private readonly BuildNumber _fallbackVersion = BuildNumber.Parse("99.99.99.99");
         private readonly int _apiVersion = 1;
+
         private bool firstHeartBeat = true;
 
-        public MasterCommunication(ILogger<MasterCommunication> logger, ApplicationConfiguration appConfig, ITranslationLookup translationLookup, IMasterApi apiInstance, IManager manager)
+        public MasterCommunication(ILogger logger, ApplicationConfiguration appConfig, ITranslationLookup translationLookup, IMasterApi apiInstance, IManager manager)
         {
             _logger = logger;
             _transLookup = translationLookup;
@@ -56,7 +55,13 @@ namespace IW4MAdmin.Application.Misc
 
             catch (Exception e)
             {
-                _logger.LogWarning(e, "Unable to retrieve IW4MAdmin version information");
+                _logger.WriteWarning(_transLookup["MANAGER_VERSION_FAIL"]);
+                while (e.InnerException != null)
+                {
+                    e = e.InnerException;
+                }
+
+                _logger.WriteDebug(e.Message);
             }
 
             if (version.CurrentVersionStable == _fallbackVersion)
@@ -105,12 +110,12 @@ namespace IW4MAdmin.Application.Misc
 
                 catch (System.Net.Http.HttpRequestException e)
                 {
-                    _logger.LogWarning(e, "Could not send heartbeat");
+                    _logger.WriteWarning($"Could not send heartbeat - {e.Message}");
                 }
 
                 catch (AggregateException e)
                 {
-                    _logger.LogWarning(e, "Could not send heartbeat");
+                    _logger.WriteWarning($"Could not send heartbeat - {e.Message}");
                     var exceptions = e.InnerExceptions.Where(ex => ex.GetType() == typeof(ApiException));
 
                     foreach (var ex in exceptions)
@@ -124,7 +129,7 @@ namespace IW4MAdmin.Application.Misc
 
                 catch (ApiException e)
                 {
-                    _logger.LogWarning(e, "Could not send heartbeat");
+                    _logger.WriteWarning($"Could not send heartbeat - {e.Message}");
                     if (e.StatusCode == System.Net.HttpStatusCode.Unauthorized)
                     {
                         connected = false;
@@ -133,7 +138,7 @@ namespace IW4MAdmin.Application.Misc
 
                 catch (Exception e)
                 {
-                    _logger.LogWarning(e, "Could not send heartbeat");
+                    _logger.WriteWarning($"Could not send heartbeat - {e.Message}");
                 }
 
 
@@ -197,7 +202,7 @@ namespace IW4MAdmin.Application.Misc
 
             if (response.ResponseMessage.StatusCode != System.Net.HttpStatusCode.OK)
             {
-                _logger.LogWarning("Non success response code from master is {statusCode}, message is {message}", response.ResponseMessage.StatusCode, response.StringContent);
+                _logger.WriteWarning($"Response code from master is {response.ResponseMessage.StatusCode}, message is {response.StringContent}");
             }
         }
     }

Application/Misc/MetaService.cs

@@ -7,8 +7,6 @@ using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Misc
 {
@@ -22,7 +20,7 @@ namespace IW4MAdmin.Application.Misc
         private readonly IDatabaseContextFactory _contextFactory;
         private readonly ILogger _logger;
 
-        public MetaService(ILogger<MetaService> logger, IDatabaseContextFactory contextFactory)
+        public MetaService(ILogger logger, IDatabaseContextFactory contextFactory)
         {
             _logger = logger;
             _metaActions = new Dictionary<MetaType, List<dynamic>>();
@@ -37,7 +35,7 @@ namespace IW4MAdmin.Application.Misc
                 return;
             }
 
-            await using var ctx = _contextFactory.CreateContext();
+            using var ctx = _contextFactory.CreateContext();
 
             var existingMeta = await ctx.EFMeta
                 .Where(_meta => _meta.Key == metaKey)
@@ -66,7 +64,7 @@ namespace IW4MAdmin.Application.Misc
 
         public async Task<EFMeta> GetPersistentMeta(string metaKey, EFClient client)
         {
-            await using var ctx = _contextFactory.CreateContext(enableTracking: false);
+            using var ctx = _contextFactory.CreateContext(enableTracking: false);
 
             return await ctx.EFMeta
                 .Where(_meta => _meta.Key == metaKey)

Application/Misc/MiddlewareActionHandler.cs

@@ -1,9 +1,8 @@
-using SharedLibraryCore.Interfaces;
+using SharedLibraryCore;
+using SharedLibraryCore.Interfaces;
 using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Misc
 {
@@ -12,7 +11,7 @@ namespace IW4MAdmin.Application.Misc
         private readonly IDictionary<string, IList<object>> _actions;
         private readonly ILogger _logger;
 
-        public MiddlewareActionHandler(ILogger<MiddlewareActionHandler> logger)
+        public MiddlewareActionHandler(ILogger logger)
         {
             _actions = new Dictionary<string, IList<object>>();
             _logger = logger;
@@ -39,7 +38,8 @@ namespace IW4MAdmin.Application.Misc
                     }
                     catch (Exception e) 
                     {
-                        _logger.LogWarning(e, "Failed to invoke middleware action {name}", name);
+                        _logger.WriteWarning($"Failed to invoke middleware action {name}");
+                        _logger.WriteDebug(e.GetExceptionInfo());
                     }
                 }
 

Application/Misc/PluginImporter.cs

@@ -5,12 +5,9 @@ using System.Reflection;
 using SharedLibraryCore.Interfaces;
 using System.Linq;
 using SharedLibraryCore;
-using IW4MAdmin.Application.API.Master;
+using IW4MAdmin.Application.Misc;
-using Microsoft.Extensions.Logging;
-using SharedLibraryCore.Configuration;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
-namespace IW4MAdmin.Application.Misc
+namespace IW4MAdmin.Application.Helpers
 {
     /// <summary>
     /// implementation of IPluginImporter
@@ -18,19 +15,12 @@ namespace IW4MAdmin.Application.Misc
     /// </summary>
     public class PluginImporter : IPluginImporter
     {
-        private IEnumerable<PluginSubscriptionContent> _pluginSubscription;
         private static readonly string PLUGIN_DIR = "Plugins";
         private readonly ILogger _logger;
-        private readonly IRemoteAssemblyHandler _remoteAssemblyHandler;
-        private readonly IMasterApi _masterApi;
-        private readonly ApplicationConfiguration _appConfig;
 
-        public PluginImporter(ILogger<PluginImporter> logger, ApplicationConfiguration appConfig, IMasterApi masterApi, IRemoteAssemblyHandler remoteAssemblyHandler)
+        public PluginImporter(ILogger logger)
         {
             _logger = logger;
-            _masterApi = masterApi;
-            _remoteAssemblyHandler = remoteAssemblyHandler;
-            _appConfig = appConfig;
         }
 
         /// <summary>
@@ -43,16 +33,16 @@ namespace IW4MAdmin.Application.Misc
 
             if (Directory.Exists(pluginDir))
             {
-                var scriptPluginFiles = Directory.GetFiles(pluginDir, "*.js").AsEnumerable().Union(GetRemoteScripts());
+                string[] scriptPluginFiles = Directory.GetFiles(pluginDir, "*.js");
 
-                _logger.LogDebug("Discovered {count} potential script plugins", scriptPluginFiles.Count());
+                _logger.WriteInfo($"Discovered {scriptPluginFiles.Length} potential script plugins");
 
-                if (scriptPluginFiles.Count() > 0)
+                if (scriptPluginFiles.Length > 0)
                 {
                     foreach (string fileName in scriptPluginFiles)
                     {
-                        _logger.LogDebug("Discovered script plugin {fileName}", fileName);
+                        _logger.WriteInfo($"Discovered script plugin {fileName}");
-                        var plugin = new ScriptPlugin(_logger, fileName);
+                        var plugin = new ScriptPlugin(fileName);
                         yield return plugin;
                     }
                 }
@@ -72,70 +62,27 @@ namespace IW4MAdmin.Application.Misc
             if (Directory.Exists(pluginDir))
             {
                 var dllFileNames = Directory.GetFiles(pluginDir, "*.dll");
-                _logger.LogDebug("Discovered {count} potential plugin assemblies", dllFileNames.Length);
+                _logger.WriteInfo($"Discovered {dllFileNames.Length} potential plugin assemblies");
 
                 if (dllFileNames.Length > 0)
                 {
-                    // we only want to load the most recent assembly in case of duplicates
+                    var assemblies = dllFileNames.Select(_name => Assembly.LoadFrom(_name));
-                    var assemblies = dllFileNames.Select(_name => Assembly.LoadFrom(_name))
-                        .Union(GetRemoteAssemblies())
-                        .GroupBy(_assembly => _assembly.FullName).Select(_assembly => _assembly.OrderByDescending(_assembly => _assembly.GetName().Version).First());
 
                     pluginTypes = assemblies
                         .SelectMany(_asm => _asm.GetTypes())
                         .Where(_assemblyType => _assemblyType.GetInterface(nameof(IPlugin), false) != null);
 
-                    _logger.LogDebug("Discovered {count} plugin implementations", pluginTypes.Count());
+                    _logger.WriteInfo($"Discovered {pluginTypes.Count()} plugin implementations");
 
                     commandTypes = assemblies
                         .SelectMany(_asm => _asm.GetTypes())
                         .Where(_assemblyType => _assemblyType.IsClass && _assemblyType.BaseType == typeof(Command));
 
-                    _logger.LogDebug("Discovered {count} plugin commands", commandTypes.Count());
+                    _logger.WriteInfo($"Discovered {commandTypes.Count()} plugin commands");
                 }
             }
 
             return (pluginTypes, commandTypes);
         }
-
-        private IEnumerable<Assembly> GetRemoteAssemblies()
-        {
-            try
-            {
-                if (_pluginSubscription == null)
-                    _pluginSubscription = _masterApi.GetPluginSubscription(Guid.Parse(_appConfig.Id), _appConfig.SubscriptionId).Result;
-
-                return _remoteAssemblyHandler.DecryptAssemblies(_pluginSubscription.Where(sub => sub.Type == PluginType.Binary).Select(sub => sub.Content).ToArray());
-            }
-
-            catch (Exception ex)
-            {
-                _logger.LogWarning(ex, "Could not load remote assemblies");
-                return Enumerable.Empty<Assembly>();
-            }
-        }
-
-        private IEnumerable<string> GetRemoteScripts()
-        {
-            try
-            {
-                if (_pluginSubscription == null)
-                    _pluginSubscription = _masterApi.GetPluginSubscription(Guid.Parse(_appConfig.Id), _appConfig.SubscriptionId).Result;
-
-                return _remoteAssemblyHandler.DecryptScripts(_pluginSubscription.Where(sub => sub.Type == PluginType.Script).Select(sub => sub.Content).ToArray());
-            }
-
-            catch (Exception ex)
-            {
-                _logger.LogWarning(ex,"Could not load remote scripts");
-                return Enumerable.Empty<string>();
-            }
-        }
-    }
-
-    public enum PluginType
-    {
-        Binary,
-        Script
     }
 }

Application/Misc/RemoteAssemblyHandler.cs

@@ -1,76 +0,0 @@
-using SharedLibraryCore.Configuration;
-using SharedLibraryCore.Interfaces;
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Reflection;
-using System.Security.Cryptography;
-using System.Text;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
-
-namespace IW4MAdmin.Application.Misc
-{
-    public class RemoteAssemblyHandler : IRemoteAssemblyHandler
-    {
-        private const int keyLength = 32;
-        private const int tagLength = 16;
-        private const int nonceLength = 12;
-        private const int iterationCount = 10000;
-
-        private readonly ApplicationConfiguration _appconfig;
-        private readonly ILogger _logger;
-
-        public RemoteAssemblyHandler(ILogger<RemoteAssemblyHandler> logger, ApplicationConfiguration appconfig)
-        {
-            _appconfig = appconfig;
-            _logger = logger;
-        }
-
-        public IEnumerable<Assembly> DecryptAssemblies(string[] encryptedAssemblies)
-        {
-            return DecryptContent(encryptedAssemblies)
-                .Select(decryptedAssembly => Assembly.Load(decryptedAssembly));
-        }
-
-        public IEnumerable<string> DecryptScripts(string[] encryptedScripts)
-        {
-            return DecryptContent(encryptedScripts).Select(decryptedScript => Encoding.UTF8.GetString(decryptedScript));
-        }
-
-        private byte[][] DecryptContent(string[] content)
-        {
-            if (string.IsNullOrEmpty(_appconfig.Id) || string.IsNullOrWhiteSpace(_appconfig.SubscriptionId))
-            {
-                _logger.LogWarning($"{nameof(_appconfig.Id)} and {nameof(_appconfig.SubscriptionId)} must be provided to attempt loading remote assemblies/scripts");
-                return new byte[0][];
-            }
-
-            var assemblies = content.Select(piece =>
-            {
-                byte[] byteContent = Convert.FromBase64String(piece);
-                byte[] encryptedContent = byteContent.Take(byteContent.Length - (tagLength + nonceLength)).ToArray();
-                byte[] tag = byteContent.Skip(byteContent.Length - (tagLength + nonceLength)).Take(tagLength).ToArray();
-                byte[] nonce = byteContent.Skip(byteContent.Length - nonceLength).Take(nonceLength).ToArray();
-                byte[] decryptedContent = new byte[encryptedContent.Length];
-
-                var keyGen = new Rfc2898DeriveBytes(Encoding.UTF8.GetBytes(_appconfig.SubscriptionId), Encoding.UTF8.GetBytes(_appconfig.Id.ToString()), iterationCount, HashAlgorithmName.SHA512);
-                var encryption = new AesGcm(keyGen.GetBytes(keyLength));
-
-                try
-                {
-                    encryption.Decrypt(nonce, encryptedContent, tag, decryptedContent);
-                }
-
-                catch (CryptographicException ex)
-                {
-                    _logger.LogError(ex, "Could not decrypt remote plugin assemblies");
-                }
-
-                return decryptedContent;
-            });
-
-            return assemblies.ToArray();
-        }
-    }
-}

Application/Misc/ScriptCommand.cs

@@ -4,9 +4,7 @@ using SharedLibraryCore.Configuration;
 using SharedLibraryCore.Interfaces;
 using System;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
 using static SharedLibraryCore.Database.Models.EFClient;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Misc
 {
@@ -16,15 +14,13 @@ namespace IW4MAdmin.Application.Misc
     public class ScriptCommand : Command
     {
         private readonly Action<GameEvent> _executeAction;
-        private readonly ILogger _logger;
 
         public ScriptCommand(string name, string alias, string description, bool isTargetRequired, Permission permission,
-            CommandArgument[] args, Action<GameEvent> executeAction, CommandConfiguration config, ITranslationLookup layout, ILogger<ScriptCommand> logger)
+            CommandArgument[] args, Action<GameEvent> executeAction, CommandConfiguration config, ITranslationLookup layout)
             : base(config, layout)
         {
 
             _executeAction = executeAction;
-            _logger = logger;
             Name = name;
             Alias = alias;
             Description = description;
@@ -33,21 +29,14 @@ namespace IW4MAdmin.Application.Misc
             Arguments = args;
         }
 
-        public override async Task ExecuteAsync(GameEvent e)
+        public override Task ExecuteAsync(GameEvent E)
         {
             if (_executeAction == null)
             {
                 throw new InvalidOperationException($"No execute action defined for command \"{Name}\"");
             }
 
-            try
+            return Task.Run(() => _executeAction(E));
-            {
-                await Task.Run(() => _executeAction(e));
-            }
-            catch (Exception ex)
-            {
-                _logger.LogError(ex, "Failed to execute ScriptCommand action for command {command} {@event}", Name, e);
-            }
         }
     }
 }

Application/Misc/ScriptPlugin.cs

@@ -1,5 +1,4 @@
-using System;
+using Jint;
-using Jint;
 using Jint.Native;
 using Jint.Runtime;
 using Microsoft.CSharp.RuntimeBinder;
@@ -13,9 +12,6 @@ using System.Linq;
 using System.Text;
 using System.Threading;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using Serilog.Context;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.Misc
 {
@@ -43,11 +39,9 @@ namespace IW4MAdmin.Application.Misc
         private readonly SemaphoreSlim _onProcessing;
         private bool successfullyLoaded;
         private readonly List<string> _registeredCommandNames;
-        private readonly ILogger _logger;
 
-        public ScriptPlugin(ILogger logger, string filename, string workingDirectory = null)
+        public ScriptPlugin(string filename, string workingDirectory = null)
         {
-            _logger = logger;
             _fileName = filename;
             Watcher = new FileSystemWatcher()
             {
@@ -90,7 +84,7 @@ namespace IW4MAdmin.Application.Misc
 
                     foreach (string commandName in _registeredCommandNames)
                     {
-                        _logger.LogDebug("Removing plugin registered command {command}", commandName);
+                        manager.GetLogger(0).WriteDebug($"Removing plugin registered command \"{commandName}\"");
                         manager.RemoveCommandByName(commandName);
                     }
 
@@ -118,28 +112,7 @@ namespace IW4MAdmin.Application.Misc
                     })
                     .CatchClrExceptions());
 
-                try
+                _scriptEngine.Execute(script);
-                {
-                    _scriptEngine.Execute(script);
-                }
-                catch (JavaScriptException ex)
-                {
-                
-                    _logger.LogError(ex,
-                        "Encountered JavaScript runtime error while executing {methodName} for script plugin {plugin} at {@locationInfo}",
-                        nameof(Initialize), _fileName, ex.Location);
-                    throw new PluginException($"A JavaScript parsing error occured while initializing script plugin");
-                }
-
-                catch (Exception e)
-                {
-
-                        _logger.LogError(e,
-                            "Encountered unexpected error while running {methodName} for script plugin {plugin}",
-                            nameof(Initialize), _fileName);
-                    throw new PluginException($"An unexpected error occured while initialization script plugin");
-                }
-
                 _scriptEngine.SetValue("_localization", Utilities.CurrentLocalization);
                 _scriptEngine.SetValue("_serviceResolver", serviceResolver);
                 dynamic pluginObject = _scriptEngine.GetValue("plugin").ToObject();
@@ -156,7 +129,7 @@ namespace IW4MAdmin.Application.Misc
                     {
                         foreach (var command in GenerateScriptCommands(commands, scriptCommandFactory))
                         {
-                            _logger.LogDebug("Adding plugin registered command {commandName}", command.Name);
+                            manager.GetLogger(0).WriteDebug($"Adding plugin registered command \"{command.Name}\"");
                             manager.AddAdditionalCommand(command);
                             _registeredCommandNames.Add(command.Name);
                         }
@@ -194,20 +167,12 @@ namespace IW4MAdmin.Application.Misc
 
             catch (JavaScriptException ex)
             {
-                _logger.LogError(ex,
+                throw new PluginException($"An error occured while initializing script plugin: {ex.Error} (Line: {ex.Location.Start.Line}, Character: {ex.Location.Start.Column})") { PluginFile = _fileName };
-                    "Encountered JavaScript runtime error while executing {methodName} for script plugin {plugin} initialization {@locationInfo}",
-                    nameof(OnLoadAsync), _fileName, ex.Location);
-                
-                throw new PluginException("An error occured while initializing script plugin");
             }
 
-            catch (Exception ex)
+            catch
             {
-                _logger.LogError(ex,
+                throw;
-                    "Encountered unexpected error while running {methodName} for script plugin {plugin}",
-                    nameof(OnLoadAsync), _fileName);
-                
-                throw new PluginException("An unexpected error occured while initializing script plugin");
             }
 
             finally
@@ -233,28 +198,9 @@ namespace IW4MAdmin.Application.Misc
                     _scriptEngine.Execute("plugin.onEventAsync(_gameEvent, _server)").GetCompletionValue();
                 }
 
-                catch (JavaScriptException ex)
+                catch
                 {
-                    using (LogContext.PushProperty("Server", S.ToString()))
+                    throw;
-                    {
-                        _logger.LogError(ex,
-                            "Encountered JavaScript runtime error while executing {methodName} for script plugin {plugin} with event type {eventType} {@locationInfo}",
-                            nameof(OnEventAsync), _fileName, E.Type, ex.Location);
-                    }
-
-                    throw new PluginException($"An error occured while executing action for script plugin");
-                }
-
-                catch (Exception e)
-                {
-                    using (LogContext.PushProperty("Server", S.ToString()))
-                    {
-                        _logger.LogError(e,
-                            "Encountered unexpected error while running {methodName} for script plugin {plugin} with event type {eventType}",
-                            nameof(OnEventAsync), _fileName, E.Type);
-                    }
-
-                    throw new PluginException($"An error occured while executing action for script plugin");
                 }
 
                 finally
@@ -269,7 +215,7 @@ namespace IW4MAdmin.Application.Misc
 
         public Task OnLoadAsync(IManager manager)
         {
-            _logger.LogDebug("OnLoad executing for {name}", Name);
+            manager.GetLogger(0).WriteDebug($"OnLoad executing for {Name}");
             _scriptEngine.SetValue("_manager", manager);
             return Task.FromResult(_scriptEngine.Execute("plugin.onLoadAsync(_manager)").GetCompletionValue());
         }

Application/RCon/ConnectionState.cs

@@ -18,7 +18,7 @@ namespace IW4MAdmin.Application.RCon
         }
 
         public int ConnectionAttempts { get; set; }
-        private const int BufferSize = 16384;
+        const int BufferSize = 4096;
         public readonly byte[] ReceiveBuffer = new byte[BufferSize];
         public readonly SemaphoreSlim OnComplete = new SemaphoreSlim(1, 1);
         public readonly ManualResetEventSlim OnSentData = new ManualResetEventSlim(false);

Application/RCon/RConConnection.cs

@@ -10,11 +10,7 @@ using System.Net;
 using System.Net.Sockets;
 using System.Text;
 using System.Text.RegularExpressions;
-using System.Threading;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using Serilog.Context;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.RCon
 {
@@ -27,12 +23,11 @@ namespace IW4MAdmin.Application.RCon
         public IPEndPoint Endpoint { get; private set; }
         public string RConPassword { get; private set; }
 
-        private IRConParser parser;
         private IRConParserConfiguration config;
         private readonly ILogger _log;
         private readonly Encoding _gameEncoding;
 
-        public RConConnection(string ipAddress, int port, string password, ILogger<RConConnection> log, Encoding gameEncoding)
+        public RConConnection(string ipAddress, int port, string password, ILogger log, Encoding gameEncoding)
         {
             Endpoint = new IPEndPoint(IPAddress.Parse(ipAddress), port);
             _gameEncoding = gameEncoding;
@@ -40,10 +35,9 @@ namespace IW4MAdmin.Application.RCon
             _log = log;
         }
 
-        public void SetConfiguration(IRConParser parser)
+        public void SetConfiguration(IRConParserConfiguration config)
         {
-            this.parser = parser;
+            this.config = config;
-            config = parser.Configuration;
         }
 
         public async Task<string[]> SendQueryAsync(StaticHelpers.QueryType type, string parameters = "")
@@ -55,8 +49,9 @@ namespace IW4MAdmin.Application.RCon
 
             var connectionState = ActiveQueries[this.Endpoint];
 
-            _log.LogDebug("Waiting for semaphore to be released [{endpoint}]", Endpoint);
+#if DEBUG == true
-
+            _log.WriteDebug($"Waiting for semaphore to be released [{this.Endpoint}]");
+#endif
             // enter the semaphore so only one query is sent at a time per server.
             await connectionState.OnComplete.WaitAsync();
 
@@ -69,8 +64,10 @@ namespace IW4MAdmin.Application.RCon
 
             connectionState.LastQuery = DateTime.Now;
 
-            _log.LogDebug("Semaphore has been released [{endpoint}]", Endpoint);
+#if DEBUG == true
-            _log.LogDebug("Query {@queryInfo}", new { endpoint=Endpoint.ToString(), type, parameters });
+            _log.WriteDebug($"Semaphore has been released [{this.Endpoint}]");
+            _log.WriteDebug($"Query [{this.Endpoint},{type.ToString()},{parameters}]");
+#endif
 
             byte[] payload = null;
             bool waitForResponse = config.WaitForResponse;
@@ -115,34 +112,18 @@ namespace IW4MAdmin.Application.RCon
 
             // this happens when someone tries to send something that can't be converted into a 7 bit character set
             // e.g: emoji -> windows-1252
-            catch (OverflowException ex)
+            catch (OverflowException)
             {
                 connectionState.OnComplete.Release(1);
-                using (LogContext.PushProperty("Server", Endpoint.ToString()))
+                throw new NetworkException($"Invalid character encountered when converting encodings - {parameters}");
-                {
-                    _log.LogError(ex, "Could not convert RCon data payload to desired encoding {encoding} {params}",
-                        _gameEncoding.EncodingName, parameters);
-                }
-
-                throw new RConException($"Invalid character encountered when converting encodings");
             }
 
             byte[][] response = null;
 
-            retrySend:
+        retrySend:
-            if (connectionState.ConnectionAttempts > 1)
-            {
-                using (LogContext.PushProperty("Server", Endpoint.ToString()))
-                {
-                    _log.LogInformation(
-                        "Retrying RCon message ({connectionAttempts}/{allowedConnectionFailures} attempts) with parameters {payload}", 
-                        connectionState.ConnectionAttempts, 
-                        StaticHelpers.AllowedConnectionFails, parameters);
-                }
-            }
             using (var socket = new Socket(AddressFamily.InterNetwork, SocketType.Dgram, ProtocolType.Udp)
             {
-                DontFragment = false,
+                DontFragment = true,
                 Ttl = 100,
                 ExclusiveAddressUse = true,
             })
@@ -152,19 +133,16 @@ namespace IW4MAdmin.Application.RCon
                 connectionState.OnReceivedData.Reset();
                 connectionState.ConnectionAttempts++;
                 connectionState.BytesReadPerSegment.Clear();
-                bool exceptionCaught = false;
+#if DEBUG == true
-
+                _log.WriteDebug($"Sending {payload.Length} bytes to [{this.Endpoint}] ({connectionState.ConnectionAttempts}/{StaticHelpers.AllowedConnectionFails})");
-                _log.LogDebug("Sending {payloadLength} bytes to [{endpoint}] ({connectionAttempts}/{allowedConnectionFailures})",
+#endif
-                payload.Length, Endpoint, connectionState.ConnectionAttempts, StaticHelpers.AllowedConnectionFails);
-
                 try
                 {
-                    
+                    response = await SendPayloadAsync(payload, waitForResponse);
-                    response = await SendPayloadAsync(payload, waitForResponse, parser.OverrideTimeoutForCommand(parameters));
 
                     if ((response.Length == 0 || response[0].Length == 0) && waitForResponse)
                     {
-                        throw new RConException("Expected response but got 0 bytes back");
+                        throw new NetworkException("Expected response but got 0 bytes back");
                     }
 
                     connectionState.ConnectionAttempts = 0;
@@ -172,28 +150,18 @@ namespace IW4MAdmin.Application.RCon
 
                 catch
                 {
-                    // we want to retry with a delay
                     if (connectionState.ConnectionAttempts < StaticHelpers.AllowedConnectionFails)
                     {
-                        exceptionCaught = true;
+                        await Task.Delay(StaticHelpers.FloodProtectionInterval);
-                        await Task.Delay(StaticHelpers.SocketTimeout(connectionState.ConnectionAttempts));
                         goto retrySend;
                     }
 
-                    using (LogContext.PushProperty("Server", Endpoint.ToString()))
+                    throw new NetworkException(Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_COMMUNICATION"].FormatExt(Endpoint));
-                    {
-                        _log.LogWarning(
-                            "Made {connectionAttempts} attempts to send RCon data to server, but received no response",
-                            connectionState.ConnectionAttempts);
-                    }
-                    connectionState.ConnectionAttempts = 0;
-                    throw new NetworkException("Reached maximum retry attempts to send RCon data to server");
                 }
 
                 finally
                 {
-                    // we don't want to release if we're going to retry the query
+                    if (connectionState.OnComplete.CurrentCount == 0)
-                    if (connectionState.OnComplete.CurrentCount == 0 && !exceptionCaught)
                     {
                         connectionState.OnComplete.Release(1);
                     }
@@ -202,22 +170,23 @@ namespace IW4MAdmin.Application.RCon
 
             if (response.Length == 0)
             {
-                _log.LogDebug("Received empty response for RCon request {@query}", new { endpoint=Endpoint.ToString(), type, parameters });
+                _log.WriteWarning($"Received empty response for request [{type.ToString()}, {parameters}, {Endpoint.ToString()}]");
                 return new string[0];
             }
 
             string responseString = type == StaticHelpers.QueryType.COMMAND_STATUS ?
-               ReassembleSegmentedStatus(response) : RecombineMessages(response);
+               ReassembleSegmentedStatus(response) :
+               _gameEncoding.GetString(response[0]) + '\n';
 
             // note: not all games respond if the pasword is wrong or not set
             if (responseString.Contains("Invalid password") || responseString.Contains("rconpassword"))
             {
-                throw new RConException(Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_RCON_INVALID"]);
+                throw new NetworkException(Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_RCON_INVALID"]);
             }
 
             if (responseString.Contains("rcon_password"))
             {
-                throw new RConException(Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_RCON_NOTSET"]);
+                throw new NetworkException(Utilities.CurrentLocalization.LocalizationIndex["SERVER_ERROR_RCON_NOTSET"]);
             }
 
             if (responseString.Contains(config.ServerNotRunningResponse))
@@ -230,13 +199,7 @@ namespace IW4MAdmin.Application.RCon
 
             if (headerSplit.Length != 2)
             {
-                using (LogContext.PushProperty("Server", Endpoint.ToString()))
+                throw new NetworkException("Unexpected response header from server");
-                {
-                    _log.LogWarning("Invalid response header from server. Expected {expected}, but got {response}",
-                        config.CommandPrefixes.RConResponse, headerSplit.FirstOrDefault());
-                }
-
-                throw new RConException("Unexpected response header from server");
             }
 
             string[] splitResponse = headerSplit.Last().Split(new char[] { '\n' }, StringSplitOptions.RemoveEmptyEntries);
@@ -271,36 +234,7 @@ namespace IW4MAdmin.Application.RCon
             return string.Join("", splitStatusStrings);
         }
 
-        /// <summary>
+        private async Task<byte[][]> SendPayloadAsync(byte[] payload, bool waitForResponse)
-        /// Recombines multiple game messages into one
-        /// </summary>
-        /// <param name="payload"></param>
-        /// <returns></returns>
-        private string RecombineMessages(byte[][] payload)
-        {
-            if (payload.Length == 1)
-            {
-                return _gameEncoding.GetString(payload[0]).TrimEnd('\n') + '\n';
-            }
-
-            else
-            {
-                var builder = new StringBuilder();
-                for (int i = 0; i < payload.Length; i++)
-                {
-                    string message = _gameEncoding.GetString(payload[i]).TrimEnd('\n') + '\n';
-                    if (i > 0)
-                    {
-                        message = message.Replace(config.CommandPrefixes.RConResponse, "");
-                    }
-                    builder.Append(message);
-                }
-                builder.Append('\n');
-                return builder.ToString();
-            }
-        }
-
-        private async Task<byte[][]> SendPayloadAsync(byte[] payload, bool waitForResponse, TimeSpan overrideTimeout)
         {
             var connectionState = ActiveQueries[this.Endpoint];
             var rconSocket = (Socket)connectionState.SendEventArgs.UserToken;
@@ -324,17 +258,11 @@ namespace IW4MAdmin.Application.RCon
 
             if (sendDataPending)
             {
-                // the send has not been completed asynchronously
+                // the send has not been completed asyncronously
-                // this really shouldn't ever happen because it's UDP
+                if (!await Task.Run(() => connectionState.OnSentData.Wait(StaticHelpers.SocketTimeout)))
-                if (!await Task.Run(() => connectionState.OnSentData.Wait(StaticHelpers.SocketTimeout(1))))
                 {
-                    using(LogContext.PushProperty("Server", Endpoint.ToString()))
-                    {
-                        _log.LogWarning("Socket timed out while sending RCon data on attempt {attempt}",
-                            connectionState.ConnectionAttempts);
-                    }
                     rconSocket.Close();
-                    throw new NetworkException("Timed out sending RCon data", rconSocket);
+                    throw new NetworkException("Timed out sending data", rconSocket);
                 }
             }
 
@@ -350,37 +278,15 @@ namespace IW4MAdmin.Application.RCon
 
             if (receiveDataPending)
             {
-                _log.LogDebug("Waiting to asynchronously receive data on attempt #{connectionAttempts}", connectionState.ConnectionAttempts);
+                if (!await Task.Run(() => connectionState.OnReceivedData.Wait(10000)))
-                if (!await Task.Run(() => connectionState.OnReceivedData.Wait(
-                    new[]
-                    {
-                        StaticHelpers.SocketTimeout(connectionState.ConnectionAttempts), 
-                        overrideTimeout
-                    }.Max())))
                 {
-                    if (connectionState.ConnectionAttempts > 1) // this reduces some spam for unstable connections
-                    {
-                        using (LogContext.PushProperty("Server", Endpoint.ToString()))
-                        {
-                            _log.LogWarning(
-                                "Socket timed out while waiting for RCon response on attempt {attempt} with timeout delay of {timeout}",
-                                connectionState.ConnectionAttempts,
-                                StaticHelpers.SocketTimeout(connectionState.ConnectionAttempts));
-                        }
-                    }
-
                     rconSocket.Close();
-                    throw new NetworkException("Timed out receiving RCon response", rconSocket);
+                    throw new NetworkException("Timed out waiting for response", rconSocket);
                 }
             }
 
             rconSocket.Close();
 
-            return GetResponseData(connectionState);
-        }
-
-        private byte[][] GetResponseData(ConnectionState connectionState)
-        {
             var responseList = new List<byte[]>();
             int totalBytesRead = 0;
 
@@ -399,72 +305,58 @@ namespace IW4MAdmin.Application.RCon
 
         private void OnDataReceived(object sender, SocketAsyncEventArgs e)
         {
-            _log.LogDebug("Read {bytesTransferred} bytes from {endpoint}", e.BytesTransferred, e.RemoteEndPoint);
+#if DEBUG == true
+            _log.WriteDebug($"Read {e.BytesTransferred} bytes from {e.RemoteEndPoint.ToString()}");
+#endif
 
             // this occurs when we close the socket
             if (e.BytesTransferred == 0)
             {
-                _log.LogDebug("No bytes were transmitted so the connection was probably closed");
                 ActiveQueries[this.Endpoint].OnReceivedData.Set();
                 return;
             }
 
-            if (!(sender is Socket sock))
+            if (sender is Socket sock)
             {
-                return;
+                var state = ActiveQueries[this.Endpoint];
-            }
+                state.BytesReadPerSegment.Add(e.BytesTransferred);
 
-            var state = ActiveQueries[this.Endpoint];
+                try
-            state.BytesReadPerSegment.Add(e.BytesTransferred);
-
-            // I don't even want to know why this works for getting more data from Cod4x
-            // but I'm leaving it in here as long as it doesn't break anything.
-            // it's very stupid...
-            Thread.Sleep(150);
-
-            try
-            {
-                var totalBytesTransferred = e.BytesTransferred;
-                _log.LogDebug("{total} total bytes transferred with {available} bytes remaining", totalBytesTransferred, sock.Available);
-                // we still have available data so the payload was segmented
-                while (sock.Available > 0)
                 {
-                    _log.LogDebug("{available} more bytes to be read", sock.Available);
+                    // we still have available data so the payload was segmented
-
+                    if (sock.Available > 0)
-                    var bufferSpaceAvailable = sock.Available + totalBytesTransferred - state.ReceiveBuffer.Length;
-                    if (bufferSpaceAvailable >= 0 )
                     {
-                        _log.LogWarning("Not enough buffer space to store incoming data {bytesNeeded} additional bytes required", bufferSpaceAvailable);
+                        state.ReceiveEventArgs.SetBuffer(state.ReceiveBuffer, e.BytesTransferred, state.ReceiveBuffer.Length - e.BytesTransferred);
-                        continue;
+
+                        if (!sock.ReceiveAsync(state.ReceiveEventArgs))
+                        {
+#if DEBUG == true
+                            _log.WriteDebug($"Read {state.ReceiveEventArgs.BytesTransferred} synchronous bytes from {e.RemoteEndPoint.ToString()}");
+#endif
+                            // we need to increment this here because the callback isn't executed if there's no pending IO
+                            state.BytesReadPerSegment.Add(state.ReceiveEventArgs.BytesTransferred);
+                            ActiveQueries[this.Endpoint].OnReceivedData.Set();
+                        }
                     }
 
-                    state.ReceiveEventArgs.SetBuffer(state.ReceiveBuffer, totalBytesTransferred, sock.Available);
+                    else
-
-                    if (sock.ReceiveAsync(state.ReceiveEventArgs))
                     {
-                        _log.LogDebug("Remaining bytes are async");
+                        ActiveQueries[this.Endpoint].OnReceivedData.Set();
-                        continue;
                     }
-                        
-                    _log.LogDebug("Read {bytesTransferred} synchronous bytes from {endpoint}", state.ReceiveEventArgs.BytesTransferred, e.RemoteEndPoint);
-                    // we need to increment this here because the callback isn't executed if there's no pending IO
-                    state.BytesReadPerSegment.Add(state.ReceiveEventArgs.BytesTransferred);
-                    totalBytesTransferred += state.ReceiveEventArgs.BytesTransferred;
                 }
 
-                ActiveQueries[this.Endpoint].OnReceivedData.Set();
+                catch (ObjectDisposedException)
-            }
+                {
-
+                    ActiveQueries[this.Endpoint].OnReceivedData.Set();
-            catch (ObjectDisposedException)
+                }
-            {
-                _log.LogDebug("Socket was disposed while receiving data");
-                ActiveQueries[this.Endpoint].OnReceivedData.Set();
             }
         }
 
         private void OnDataSent(object sender, SocketAsyncEventArgs e)
         {
-            _log.LogDebug("Sent {byteCount} bytes to {endpoint}", e.Buffer?.Length, e.ConnectSocket?.RemoteEndPoint);
+#if DEBUG == true
+            _log.WriteDebug($"Sent {e.Buffer?.Length} bytes to {e.ConnectSocket?.RemoteEndPoint?.ToString()}");
+#endif
             ActiveQueries[this.Endpoint].OnSentData.Set();
         }
     }

Application/RconParsers/BaseRConParser.cs

@@ -8,19 +8,14 @@ using System.Collections.Generic;
 using System.Linq;
 using System.Text.RegularExpressions;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
 using static SharedLibraryCore.Server;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Application.RconParsers
 {
     public class BaseRConParser : IRConParser
     {
-        private readonly ILogger _logger;
+        public BaseRConParser(IParserRegexFactory parserRegexFactory)
-        
-        public BaseRConParser(ILogger<BaseRConParser> logger, IParserRegexFactory parserRegexFactory)
         {
-            _logger = logger;
             Configuration = new DynamicRConParserConfiguration(parserRegexFactory)
             {
                 CommandPrefixes = new CommandPrefix()
@@ -81,22 +76,7 @@ namespace IW4MAdmin.Application.RconParsers
 
         public async Task<Dvar<T>> GetDvarAsync<T>(IRConConnection connection, string dvarName, T fallbackValue = default)
         {
-            string[] lineSplit;
+            string[] lineSplit = await connection.SendQueryAsync(StaticHelpers.QueryType.GET_DVAR, dvarName);
-
-            try
-            {
-                lineSplit = await connection.SendQueryAsync(StaticHelpers.QueryType.GET_DVAR, dvarName);
-            }
-            catch
-            {
-                if (fallbackValue == null)
-                {
-                    throw;
-                }
-
-                lineSplit = new string[0];
-            }
-
             string response = string.Join('\n', lineSplit).TrimEnd('\0');
             var match = Regex.Match(response, Configuration.Dvar.Pattern);
 
@@ -138,7 +118,12 @@ namespace IW4MAdmin.Application.RconParsers
         public virtual async Task<(List<EFClient>, string, string)> GetStatusAsync(IRConConnection connection)
         {
             string[] response = await connection.SendQueryAsync(StaticHelpers.QueryType.COMMAND_STATUS);
-            _logger.LogDebug("Status Response {response}", string.Join(Environment.NewLine, response));
+#if DEBUG
+            foreach (var line in response)
+            {
+                Console.WriteLine(line);
+            }
+#endif
             return (ClientsFromStatus(response), MapFromStatus(response), GameTypeFromStatus(response));
         }
 
@@ -205,12 +190,6 @@ namespace IW4MAdmin.Application.RconParsers
 
                 if (match.Success)
                 {
-                    if (match.Values[Configuration.Status.GroupMapping[ParserRegex.GroupType.RConPing]] == "ZMBI")
-                    {
-                        _logger.LogDebug("Ignoring detected client {client} because they are zombie state", string.Join(",", match.Values));
-                        continue;
-                    }
-                    
                     int clientNumber = int.Parse(match.Values[Configuration.Status.GroupMapping[ParserRegex.GroupType.RConClientNumber]]);
                     int score = int.Parse(match.Values[Configuration.Status.GroupMapping[ParserRegex.GroupType.RConScore]]);
 
@@ -225,13 +204,12 @@ namespace IW4MAdmin.Application.RconParsers
                     long networkId;
                     string name = match.Values[Configuration.Status.GroupMapping[ParserRegex.GroupType.RConName]].TrimNewLine();
                     string networkIdString;
-                    var ip = match.Values[Configuration.Status.GroupMapping[ParserRegex.GroupType.RConIpAddress]].Split(':')[0].ConvertToIP();
 
                     try
                     {
                         networkIdString = match.Values[Configuration.Status.GroupMapping[ParserRegex.GroupType.RConNetworkId]];
 
-                        networkId = networkIdString.IsBotGuid() || (ip == null && ping == 999) ?
+                        networkId = networkIdString.IsBotGuid() ?
                             name.GenerateGuidFromString() :
                             networkIdString.ConvertGuidToLong(Configuration.GuidNumberStyle);
                     }
@@ -241,6 +219,8 @@ namespace IW4MAdmin.Application.RconParsers
                         continue;
                     }
 
+                    int? ip = match.Values[Configuration.Status.GroupMapping[ParserRegex.GroupType.RConIpAddress]].Split(':')[0].ConvertToIP();
+
                     var client = new EFClient()
                     {
                         CurrentAlias = new EFAlias()
@@ -283,16 +263,5 @@ namespace IW4MAdmin.Application.RconParsers
         public T GetDefaultDvarValue<T>(string dvarName) => Configuration.DefaultDvarValues.ContainsKey(dvarName) ?
             (T)Convert.ChangeType(Configuration.DefaultDvarValues[dvarName], typeof(T)) :
             default;
-
-        public TimeSpan OverrideTimeoutForCommand(string command)
-        {
-            if (command.Contains("map_rotate", StringComparison.InvariantCultureIgnoreCase) ||
-                command.StartsWith("map ", StringComparison.InvariantCultureIgnoreCase))
-            {
-                return TimeSpan.FromSeconds(30);
-            }
-
-            return TimeSpan.Zero;
-        }
     }
 }

Application/RconParsers/DynamicRConParser.cs

@@ -1,5 +1,4 @@
-using Microsoft.Extensions.Logging;
+using SharedLibraryCore.Interfaces;
-using SharedLibraryCore.Interfaces;
 
 namespace IW4MAdmin.Application.RconParsers
 {
@@ -9,7 +8,7 @@ namespace IW4MAdmin.Application.RconParsers
     /// </summary>
     sealed internal class DynamicRConParser : BaseRConParser
     {
-        public DynamicRConParser(ILogger<BaseRConParser> logger, IParserRegexFactory parserRegexFactory) : base(logger, parserRegexFactory)
+        public DynamicRConParser(IParserRegexFactory parserRegexFactory) : base(parserRegexFactory)
         {
         }
     }

Application/RconParsers/DynamicRConParserConfiguration.cs

@@ -1,5 +1,4 @@
-using System;
+using SharedLibraryCore.Interfaces;
-using SharedLibraryCore.Interfaces;
 using SharedLibraryCore.RCon;
 using System.Collections.Generic;
 using System.Globalization;
@@ -23,9 +22,6 @@ namespace IW4MAdmin.Application.RconParsers
         public NumberStyles GuidNumberStyle { get; set; } = NumberStyles.HexNumber;
         public IDictionary<string, string> OverrideDvarNameMapping { get; set; } = new Dictionary<string, string>();
         public IDictionary<string, string> DefaultDvarValues { get; set; } = new Dictionary<string, string>();
-        public int NoticeMaximumLines { get; set; } = 8;
-        public int NoticeMaxCharactersPerLine { get; set; } = 50;
-        public string NoticeLineSeparator { get; set; } = Environment.NewLine;
 
         public DynamicRConParserConfiguration(IParserRegexFactory parserRegexFactory)
         {

IW4MAdmin.sln

@@ -8,9 +8,11 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Solution Items", "Solution
 	ProjectSection(SolutionItems) = preProject
 		GameFiles\IW4x\userraw\scripts\_commands.gsc = GameFiles\IW4x\userraw\scripts\_commands.gsc
 		GameFiles\IW4x\userraw\scripts\_customcallbacks.gsc = GameFiles\IW4x\userraw\scripts\_customcallbacks.gsc
-		DeploymentFiles\deployment-pipeline.yml = DeploymentFiles\deployment-pipeline.yml
+		PostPublish.ps1 = PostPublish.ps1
-		DeploymentFiles\PostPublish.ps1 = DeploymentFiles\PostPublish.ps1
+		pre-release-pipeline.yml = pre-release-pipeline.yml
 		README.md = README.md
+		RunPublishPre.cmd = RunPublishPre.cmd
+		RunPublishRelease.cmd = RunPublishRelease.cmd
 		version.txt = version.txt
 	EndProjectSection
 EndProject
@@ -35,7 +37,6 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "ScriptPlugins", "ScriptPlug
 		Plugins\ScriptPlugins\ActionOnReport.js = Plugins\ScriptPlugins\ActionOnReport.js
 		Plugins\ScriptPlugins\ParserCoD4x.js = Plugins\ScriptPlugins\ParserCoD4x.js
 		Plugins\ScriptPlugins\ParserIW4x.js = Plugins\ScriptPlugins\ParserIW4x.js
-		Plugins\ScriptPlugins\ParserIW6x.js = Plugins\ScriptPlugins\ParserIW6x.js
 		Plugins\ScriptPlugins\ParserPIW5.js = Plugins\ScriptPlugins\ParserPIW5.js
 		Plugins\ScriptPlugins\ParserPT6.js = Plugins\ScriptPlugins\ParserPT6.js
 		Plugins\ScriptPlugins\ParserRektT5M.js = Plugins\ScriptPlugins\ParserRektT5M.js
@@ -44,7 +45,6 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "ScriptPlugins", "ScriptPlug
 		Plugins\ScriptPlugins\SampleScriptPluginCommand.js = Plugins\ScriptPlugins\SampleScriptPluginCommand.js
 		Plugins\ScriptPlugins\SharedGUIDKick.js = Plugins\ScriptPlugins\SharedGUIDKick.js
 		Plugins\ScriptPlugins\VPNDetection.js = Plugins\ScriptPlugins\VPNDetection.js
-		Plugins\ScriptPlugins\ParserT4.js = Plugins\ScriptPlugins\ParserT4.js
 	EndProjectSection
 EndProject
 Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Web", "Web", "{A848FCF1-8527-4AA8-A1AA-50D29695C678}"

Plugins/AutomessageFeed/AutomessageFeed.csproj

@@ -10,7 +10,7 @@
 
   <ItemGroup>
     <PackageReference Include="Microsoft.SyndicationFeed.ReaderWriter" Version="1.0.2" />
-    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2020.12.20.1" PrivateAssets="All" />
+    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2.4.9" PrivateAssets="All" />
   </ItemGroup>
 
   <Target Name="PostBuild" AfterTargets="PostBuildEvent">

Plugins/IW4ScriptCommands/IW4ScriptCommands.csproj

@@ -10,7 +10,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2020.12.20.1" PrivateAssets="All" />
+    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2.4.9" PrivateAssets="All" />
   </ItemGroup>
 
   <Target Name="PostBuild" AfterTargets="PostBuildEvent">

Plugins/LiveRadar/LiveRadar.csproj

@@ -16,7 +16,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2020.12.20.1" PrivateAssets="All" />
+    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2.4.9" PrivateAssets="All" />
   </ItemGroup>
 
   <ItemGroup>

Plugins/LiveRadar/Plugin.cs

@@ -1,13 +1,11 @@
 using LiveRadar.Configuration;
 using SharedLibraryCore;
+using SharedLibraryCore.Configuration;
+using SharedLibraryCore.Interfaces;
 using System;
 using System.Collections.Generic;
-using System.Globalization;
 using System.Linq;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using SharedLibraryCore.Interfaces;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace LiveRadar
 {
@@ -23,13 +21,11 @@ namespace LiveRadar
         private readonly Dictionary<string, long> _botGuidLookups;
         private bool addedPage;
         private readonly object lockObject = new object();
-        private readonly ILogger _logger;
 
-        public Plugin(ILogger<Plugin> logger, IConfigurationHandlerFactory configurationHandlerFactory)
+        public Plugin(IConfigurationHandlerFactory configurationHandlerFactory)
         {
             _configurationHandler = configurationHandlerFactory.GetConfigurationHandler<LiveRadarConfiguration>("LiveRadarConfiguration");
             _botGuidLookups = new Dictionary<string, long>();
-            _logger = logger;
         }
 
         public Task OnEventAsync(GameEvent E, Server S)
@@ -69,9 +65,7 @@ namespace LiveRadar
 
                     lock (lockObject)
                     {
-                        generatedBotGuid = _botGuidLookups.ContainsKey(botKey) 
+                        generatedBotGuid = _botGuidLookups.ContainsKey(botKey) ? _botGuidLookups[botKey] : 0;
-                            ? _botGuidLookups[botKey] 
-                            : (E.Extra.ToString() ?? "0").ConvertGuidToLong(NumberStyles.HexNumber);
                     }
 
                     var radarUpdate = RadarEvent.Parse(E.Data, generatedBotGuid);
@@ -86,7 +80,8 @@ namespace LiveRadar
 
                 catch (Exception e)
                 {
-                    _logger.LogError(e, "Could not parse live radar output: {data}", e.Data);
+                    S.Logger.WriteWarning($"Could not parse live radar output: {e.Data}");
+                    S.Logger.WriteDebug(e.GetExceptionInfo());
                 }
             }
 

Plugins/Login/Login.csproj

@@ -23,7 +23,7 @@
   </Target>
 
   <ItemGroup>
-    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2020.12.20.1" PrivateAssets="All" />
+    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2.4.9" PrivateAssets="All" />
   </ItemGroup>
 
 </Project>

Plugins/ProfanityDeterment/ProfanityDeterment.csproj

@@ -16,7 +16,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2020.12.20.1" PrivateAssets="All" />
+    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2.4.9" PrivateAssets="All" />
   </ItemGroup>
 
   <Target Name="PostBuild" AfterTargets="PostBuildEvent">

Plugins/ScriptPlugins/ParserCoD4x.js

@@ -15,22 +15,18 @@ var plugin = {
         eventParser = manager.GenerateDynamicEventParser(this.name);
 
         rconParser.Configuration.StatusHeader.Pattern = 'num +score +ping +playerid +steamid +name +lastmsg +address +qport +rate *';
-        rconParser.Configuration.Status.Pattern = '^ *([0-9]+) +-?([0-9]+) +((?:[A-Z]+|[0-9]+)) +((?:[a-z]|[0-9]{16,32})|0) +([[0-9]+|0]) +(.{0,32}) +([0-9]+) +(\\d+\\.\\d+\\.\\d+.\\d+\\:-*\\d{1,5}|0+.0+:-*\\d{1,5}|loopback|bot) +(-*[0-9]+) +([0-9]+) *$';
+        rconParser.Configuration.Status.Pattern = '^ *([0-9]+) +-?([0-9]+) +((?:[A-Z]+|[0-9]+)) +((?:[a-z]|[0-9]{16,32})|bot[0-9]+) ([0-9]+) +(.{0,32}) +([0-9]+) +(\\d+\\.\\d+\\.\\d+.\\d+\\:-*\\d{1,5}|0+.0+:-*\\d{1,5}|loopback) +(-*[0-9]+) +([0-9]+) *$';
         rconParser.Configuration.Status.AddMapping(104, 6); // RConName
         rconParser.Configuration.Status.AddMapping(105, 8); // RConIPAddress
-        rconParser.Configuration.CommandPrefixes.RConResponse = '\xff\xff\xff\xffprint\n';
 
         rconParser.Configuration.Dvar.Pattern = '^"(.+)" is: "(.+)?" default: "(.+)?" info: "(.+)?"$';
         rconParser.Configuration.Dvar.AddMapping(109, 2); // DVAR latched value
         rconParser.Configuration.Dvar.AddMapping(110, 4); // dvar info
-        rconParser.Configuration.GuidNumberStyle = 7; // Integer
+        rconParser.Version = 'CoD4 X - win_mingw-x86 build 963 Mar 12 2019';
-        rconParser.Configuration.NoticeLineSeparator = '. '; // CoD4x does not support \n in the client notice
-        rconParser.Version = 'CoD4 X - win_mingw-x86 build 1056 Dec 12 2020';
         rconParser.GameName = 1; // IW3
 
         eventParser.Configuration.GameDirectory = 'main';
-        eventParser.Configuration.GuidNumberStyle = 7; // Integer
+        eventParser.Version = 'CoD4 X - win_mingw-x86 build 963 Mar 12 2019';
-        eventParser.Version = 'CoD4 X - win_mingw-x86 build 1056 Dec 12 2020';
         eventParser.GameName = 1; // IW3
         eventParser.URLProtocolFormat = 'cod4://{{ip}}:{{port}}';
     },

Plugins/ScriptPlugins/ParserIW6x.js

@@ -1,45 +0,0 @@
-var rconParser;
-var eventParser;
-
-var plugin = {
-    author: 'Xerxes, RaidMax',
-    version: 0.2,
-    name: 'IW6x Parser',
-    isParser: true,
-
-    onEventAsync: function (gameEvent, server) {
-    },
-
-    onLoadAsync: function (manager) {
-        rconParser = manager.GenerateDynamicRConParser(this.name);
-        eventParser = manager.GenerateDynamicEventParser(this.name);
-
-        rconParser.Configuration.CommandPrefixes.Tell = 'tell {0} {1}';
-        rconParser.Configuration.CommandPrefixes.Say = 'say {0}';
-        rconParser.Configuration.CommandPrefixes.Kick = 'clientkick {0} "{1}"';
-        rconParser.Configuration.CommandPrefixes.Ban = 'clientkick {0} "{1}"';
-        rconParser.Configuration.CommandPrefixes.TempBan = 'clientkick {0} "{1}"';
-        rconParser.Configuration.CommandPrefixes.RConResponse = '\xff\xff\xff\xffprint\n';
-        rconParser.Configuration.Dvar.Pattern = '^ *\\"(.+)\\" is: \\"(.+)?\\" default: \\"(.+)?\\"\\n(?:latched: \\"(.+)?\\"\\n)? *(.+)$';
-        rconParser.Configuration.Status.Pattern = '^ *([0-9]+) +-?([0-9]+) +(Yes|No) +((?:[A-Z]+|[0-9]+)) +((?:[a-z]|[0-9]){8,32}|(?:[a-z]|[0-9]){8,32}|bot[0-9]+|(?:[0-9]+)) *(.{0,32}) +(\\d+\\.\\d+\\.\\d+.\\d+\\:-*\\d{1,5}|0+.0+:-*\\d{1,5}|loopback|unknown|bot) +(-*[0-9]+) *$';
-        rconParser.Configuration.StatusHeader.Pattern = 'num +score +bot +ping +guid +name +address +qport *';
-        rconParser.Configuration.WaitForResponse = false;
-        rconParser.Configuration.Status.AddMapping(102, 4);
-        rconParser.Configuration.Status.AddMapping(103, 5);
-        rconParser.Configuration.Status.AddMapping(104, 6);
-        rconParser.Configuration.DefaultDvarValues.Add('g_log', 'games_mp.log'); // todo: remove this once proper log support is implemented
-        rconParser.Configuration.DefaultDvarValues.Add('g_logsync', '1'); // todo: remove this once proper log support is implemented
-
-        rconParser.Version = 'IW6 MP 3.15 build 2 Sat Sep 14 2013 03:58:30PM win64';
-        rconParser.GameName = 4; // IW6
-        eventParser.Version = 'IW6 MP 3.15 build 2 Sat Sep 14 2013 03:58:30PM win64';
-        eventParser.GameName = 4; // IW6
-        eventParser.Configuration.GameDirectory = 'iw6x';
-    },
-
-    onUnloadAsync: function () {
-    },
-
-    onTickAsync: function (server) {
-    }
-};

Plugins/ScriptPlugins/ParserPIW5.js

@@ -3,7 +3,7 @@ var eventParser;
 
 var plugin = {
     author: 'RaidMax',
-    version: 0.6,
+    version: 0.5,
     name: 'Plutonium IW5 Parser',
     isParser: true,
 
@@ -26,7 +26,6 @@ var plugin = {
         rconParser.Configuration.Dvar.AddMapping(107, 2);
         rconParser.Configuration.WaitForResponse = true;
         rconParser.Configuration.CanGenerateLogPath = true;
-        rconParser.Configuration.NoticeLineSeparator = '. ';
 
         rconParser.Configuration.StatusHeader.Pattern = 'num +score +bot +ping +guid +name +lastmsg +address +qport +rate *';
         rconParser.Configuration.Status.Pattern = '^ *([0-9]+) +-?([0-9]+) +(?:[0-1]{1}) +([0-9]{1,4}|[A-Z]{4}) +([a-f|A-F|0-9]{16}) +(.+?) +(?:[0-9]+) +(\\d+\\.\\d+\\.\\d+\\.\\d+\\:-?\\d{1,5}|0+\\.0+:-?\\d{1,5}|loopback) +(?:-?[0-9]+) +(?:[0-9]+) *$';

Plugins/ScriptPlugins/ParserPT6.js

@@ -3,7 +3,7 @@ var eventParser;
 
 var plugin = {
     author: 'RaidMax, Xerxes',
-    version: 0.9,
+    version: 0.8,
     name: 'Plutonium T6 Parser',
     isParser: true,
 
@@ -26,7 +26,6 @@ var plugin = {
         rconParser.Configuration.Dvar.AddMapping(106, 1);
         rconParser.Configuration.Dvar.AddMapping(107, 2);
         rconParser.Configuration.WaitForResponse = false;
-        rconParser.Configuration.NoticeLineSeparator = '. ';
 
         rconParser.Configuration.StatusHeader.Pattern = 'num +score +bot +ping +guid +name +lastmsg +address +qport +rate *';
         rconParser.Configuration.Status.Pattern = '^ *([0-9]+) +([0-9]+) +(?:[0-1]{1}) +([0-9]+) +([A-F0-9]+|0) +(.+?) +(?:[0-9]+) +(\\d+\\.\\d+\\.\\d+\\.\\d+\\:-?\\d{1,5}|0+\\.0+:-?\\d{1,5}|loopback) +(?:-?[0-9]+) +(?:[0-9]+) *$';

Plugins/ScriptPlugins/ParserRektT5M.js

@@ -3,7 +3,7 @@ var eventParser;
 
 var plugin = {
     author: 'RaidMax',
-    version: 0.3,
+    version: 0.2,
     name: 'RektT5m Parser',
     isParser: true,
 
@@ -16,7 +16,7 @@ var plugin = {
 
         eventParser.Configuration.GameDirectory = 'data';
 
-        rconParser.Configuration.CommandPrefixes.RConResponse = '\xff\xff\xff\xff\x01print\n';
+        rconParser.Configuration.CommandPrefixes.RConResponse = '\xff\xff\xff\xff\x01print';
         rconParser.Configuration.CommandPrefixes.Tell = 'tell {0} {1}';
         rconParser.Configuration.CommandPrefixes.RConGetInfo = undefined;
 

Plugins/ScriptPlugins/ParserT4.js

@@ -1,30 +0,0 @@
-var rconParser;
-var eventParser;
-
-var plugin = {
-    author: 'RaidMax',
-    version: 0.1,
-    name: 'Call of Duty 5: World at War Parser',
-    isParser: true,
-
-    onEventAsync: function (gameEvent, server) {
-    },
-
-    onLoadAsync: function (manager) {
-        rconParser = manager.GenerateDynamicRConParser(this.name);
-        eventParser = manager.GenerateDynamicEventParser(this.name);
-        rconParser.Configuration.CommandPrefixes.RConResponse = '\xff\xff\xff\xffprint\n';
-        rconParser.Configuration.GuidNumberStyle = 7; // Integer
-        rconParser.Version = 'Call of Duty Multiplayer COD_WaW MP build 1.7.1263 CL(350073) JADAMS2 Thu Oct 29 15:43:55 2009 win-x86';
-        
-        eventParser.Configuration.GuidNumberStyle = 7; // Integer
-        eventParser.GameName = 5; // T4
-        eventParser.Version = 'Call of Duty Multiplayer COD_WaW MP build 1.7.1263 CL(350073) JADAMS2 Thu Oct 29 15:43:55 2009 win-x86';
-    },
-
-    onUnloadAsync: function () {
-    },
-
-    onTickAsync: function (server) {
-    }
-};

Plugins/ScriptPlugins/ParserTeknoMW3.js

@@ -3,7 +3,7 @@ var eventParser;
 
 var plugin = {
     author: 'RaidMax',
-    version: 0.8,
+    version: 0.7,
     name: 'Tekno MW3 Parser',
     isParser: true,
 
@@ -27,7 +27,6 @@ var plugin = {
         rconParser.Configuration.CommandPrefixes.TempBan = 'tempbanclient {0} "{1}"';
         rconParser.Configuration.Dvar.AddMapping(107, 1); // RCon DvarValue
         rconParser.Configuration.Dvar.Pattern = '^(.*)$';
-        rconParser.Configuration.NoticeLineSeparator = '. ';
 
         rconParser.Configuration.DefaultDvarValues.Add('sv_running', '1');
         rconParser.Configuration.OverrideDvarNameMapping.Add('_website', 'sv_clanWebsite');

Plugins/Stats/Cheat/Detection.cs

@@ -1,13 +1,13 @@
 using IW4MAdmin.Plugins.Stats.Models;
+using SharedLibraryCore;
 using SharedLibraryCore.Database.Models;
 using SharedLibraryCore.Helpers;
 using SharedLibraryCore.Interfaces;
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Text;
 using System.Text.RegularExpressions;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Plugins.Stats.Cheat
 {
@@ -223,6 +223,11 @@ namespace IW4MAdmin.Plugins.Stats.Cheat
                 if (weightedSessionAverage > Thresholds.MaxOffset(totalSessionHits) &&
                     totalSessionHits >= (Thresholds.MediumSampleMinKills * 2))
                 {
+                    Log.WriteDebug("*** Reached Max Session Average for Angle Difference ***");
+                    Log.WriteDebug($"Session Average = {weightedSessionAverage}");
+                    Log.WriteDebug($"HitCount = {HitCount}");
+                    Log.WriteDebug($"ID = {hit.AttackerId}");
+
                     results.Add(new DetectionPenaltyResult()
                     {
                         ClientPenalty = EFPenalty.PenaltyType.Ban,
@@ -233,13 +238,17 @@ namespace IW4MAdmin.Plugins.Stats.Cheat
                     });
                 }
 
-                Log.LogDebug("PredictVsReal={realAgainstPredict}", realAgainstPredict);
+#if DEBUG
+                Log.WriteDebug($"PredictVsReal={realAgainstPredict}");
+#endif
             }
             #endregion
 
             #region STRAIN
             double currentStrain = Strain.GetStrain(hit.Distance / 0.0254, hit.ViewAngles, Math.Max(50, LastOffset == 0 ? 50 : (hit.TimeOffset - LastOffset)));
-            Log.LogDebug("Current Strain: {currentStrain}", currentStrain);
+#if DEBUG == true
+            Log.WriteDebug($"Current Strain: {currentStrain}");
+#endif
             LastOffset = hit.TimeOffset;
 
             if (currentStrain > ClientStats.MaxStrain)
@@ -322,18 +331,18 @@ namespace IW4MAdmin.Plugins.Stats.Cheat
             if (!shouldIgnoreDetection)
             {
                 validButtonHitCount++;
+            }
 
-                double lastDiff = hit.TimeOffset - hit.TimeSinceLastAttack;
+            double lastDiff = hit.TimeOffset - hit.TimeSinceLastAttack;
-                if (validButtonHitCount > 0 && lastDiff <= 0)
+            if (validButtonHitCount > 0 && lastDiff <= 0)
+            {
+                results.Add(new DetectionPenaltyResult()
                 {
-                    results.Add(new DetectionPenaltyResult()
+                    ClientPenalty = EFPenalty.PenaltyType.Ban,
-                    {
+                    Value = lastDiff,
-                        ClientPenalty = EFPenalty.PenaltyType.Ban,
+                    HitCount = HitCount,
-                        Value = lastDiff,
+                    Type = DetectionType.Button
-                        HitCount = HitCount,
+                });
-                        Type = DetectionType.Button
-                    });
-                }
             }
             #endregion
 

Plugins/Stats/Cheat/Strain.cs

@@ -27,6 +27,12 @@ namespace IW4MAdmin.Plugins.Stats.Cheat
             double[] distance = Utilities.AngleStuff(newAngle, LastAngle);
             LastDistance = distance[0] + distance[1];
 
+#if DEBUG == true
+            Console.WriteLine($"Angle Between = {LastDistance}");
+            Console.WriteLine($"Distance From Target = {killDistance}");
+            Console.WriteLine($"Time Offset = {deltaTime}");
+            Console.WriteLine($"Decay Factor = {decayFactor} ");
+#endif
             // this happens on first kill
             if ((distance[0] == 0 && distance[1] == 0) ||
                 deltaTime == 0 ||

Plugins/Stats/Commands/MostKillsCommand.cs

@@ -2,6 +2,7 @@
 using System;
 using System.Linq;
 using System.Threading.Tasks;
+
 using SharedLibraryCore;
 using IW4MAdmin.Plugins.Stats.Models;
 using System.Collections.Generic;
@@ -18,8 +19,7 @@ namespace IW4MAdmin.Plugins.Stats.Commands
         private readonly IDatabaseContextFactory _contextFactory;
         private readonly CommandConfiguration _config;
 
-        public MostKillsCommand(CommandConfiguration config, ITranslationLookup translationLookup,
+        public MostKillsCommand(CommandConfiguration config, ITranslationLookup translationLookup, IDatabaseContextFactory contextFactory) : base(config, translationLookup)
-            IDatabaseContextFactory contextFactory) : base(config, translationLookup)
         {
             Name = "mostkills";
             Description = translationLookup["PLUGINS_STATS_COMMANDS_MOSTKILLS_DESC"];
@@ -32,8 +32,7 @@ namespace IW4MAdmin.Plugins.Stats.Commands
 
         public override async Task ExecuteAsync(GameEvent E)
         {
-            var mostKills = await GetMostKills(StatManager.GetIdForServer(E.Owner), Plugin.Config.Configuration(),
+            var mostKills = await GetMostKills(StatManager.GetIdForServer(E.Owner), Plugin.Config.Configuration(), _contextFactory, _translationLookup);
-                _contextFactory, _translationLookup);
             if (!E.Message.IsBroadcastCommand(_config.BroadcastCommandPrefix))
             {
                 foreach (var stat in mostKills)
@@ -51,33 +50,33 @@ namespace IW4MAdmin.Plugins.Stats.Commands
             }
         }
 
-        public static async Task<IEnumerable<string>> GetMostKills(long? serverId, StatsConfiguration config,
+        public static async Task<IEnumerable<string>> GetMostKills(long? serverId, StatsConfiguration config, IDatabaseContextFactory contextFactory, ITranslationLookup translationLookup)
-            IDatabaseContextFactory contextFactory, ITranslationLookup translationLookup)
         {
-            await using var ctx = contextFactory.CreateContext(enableTracking: false);
+            using (var ctx = contextFactory.CreateContext(enableTracking: false))
-            var dayInPast = DateTime.UtcNow.AddDays(-config.MostKillsMaxInactivityDays);
+            {
+                var dayInPast = DateTime.UtcNow.AddDays(-config.MostKillsMaxInactivityDays);
 
-            var iqStats = (from stats in ctx.Set<EFClientStatistics>()
+                var iqStats = (from stats in ctx.Set<EFClientStatistics>()
-                    join client in ctx.Clients
+                               join client in ctx.Clients
-                        on stats.ClientId equals client.ClientId
+                               on stats.ClientId equals client.ClientId
-                    join alias in ctx.Aliases
+                               join alias in ctx.Aliases
-                        on client.CurrentAliasId equals alias.AliasId
+                               on client.CurrentAliasId equals alias.AliasId
-                    where stats.ServerId == serverId
+                               where stats.ServerId == serverId
-                    where client.Level != EFClient.Permission.Banned
+                               where client.Level != EFClient.Permission.Banned
-                    where client.LastConnection >= dayInPast
+                               where client.LastConnection >= dayInPast
-                    orderby stats.Kills descending
+                               orderby stats.Kills descending
-                    select new
+                               select new
-                    {
+                               {
-                        alias.Name,
+                                   alias.Name,
-                        stats.Kills
+                                   stats.Kills
-                    })
+                               })
-                .Take(config.MostKillsClientLimit);
+                               .Take(config.MostKillsClientLimit);
 
-            var iqList = await iqStats.ToListAsync();
+                var iqList = await iqStats.ToListAsync();
 
-            return iqList.Select((stats, index) => translationLookup["PLUGINS_STATS_COMMANDS_MOSTKILLS_FORMAT"]
+                return iqList.Select((stats, index) => translationLookup["PLUGINS_STATS_COMMANDS_MOSTKILLS_FORMAT"].FormatExt(index + 1, stats.Name, stats.Kills))
-                    .FormatExt(index + 1, stats.Name, stats.Kills))
+                    .Prepend(Utilities.CurrentLocalization.LocalizationIndex["PLUGINS_STATS_COMMANDS_MOSTKILLS_HEADER"]);
-                .Prepend(Utilities.CurrentLocalization.LocalizationIndex["PLUGINS_STATS_COMMANDS_MOSTKILLS_HEADER"]);
+            }
         }
     }
 }

Plugins/Stats/Commands/MostPlayedCommand.cs

@@ -16,7 +16,7 @@ namespace IW4MAdmin.Plugins.Stats.Commands
 {
     class MostPlayedCommand : Command
     {
-        public static async Task<List<string>> GetMostPlayed(Server s, ITranslationLookup translationLookup, IDatabaseContextFactory contextFactory)
+        public static async Task<List<string>> GetMostPlayed(Server s, ITranslationLookup translationLookup)
         {
             long serverId = StatManager.GetIdForServer(s);
 
@@ -25,39 +25,42 @@ namespace IW4MAdmin.Plugins.Stats.Commands
                 $"^5--{translationLookup["PLUGINS_STATS_COMMANDS_MOSTPLAYED_TEXT"]}--"
             };
 
-            await using var context = contextFactory.CreateContext(false);
+            using (var db = new DatabaseContext(true))
-            var thirtyDaysAgo = DateTime.UtcNow.AddMonths(-1);
+            {
+                db.ChangeTracker.AutoDetectChangesEnabled = false;
+                db.ChangeTracker.QueryTrackingBehavior = QueryTrackingBehavior.NoTracking;
 
-            var iqStats = (from stats in context.Set<EFClientStatistics>()
+                var thirtyDaysAgo = DateTime.UtcNow.AddMonths(-1);
-                           join client in context.Clients
-                           on stats.ClientId equals client.ClientId
-                           join alias in context.Aliases
-                           on client.CurrentAliasId equals alias.AliasId
-                           where stats.ServerId == serverId
-                           where client.Level != EFClient.Permission.Banned
-                           where client.LastConnection >= thirtyDaysAgo
-                           orderby stats.TimePlayed descending
-                           select new
-                           {
-                               alias.Name,
-                               client.TotalConnectionTime,
-                               stats.Kills
-                           })
-                            .Take(5);
 
-            var iqList = await iqStats.ToListAsync();
+                var iqStats = (from stats in db.Set<EFClientStatistics>()
+                               join client in db.Clients
+                               on stats.ClientId equals client.ClientId
+                               join alias in db.Aliases
+                               on client.CurrentAliasId equals alias.AliasId
+                               where stats.ServerId == serverId
+                               where client.Level != EFClient.Permission.Banned
+                               where client.LastConnection >= thirtyDaysAgo
+                               orderby stats.TimePlayed descending
+                               select new
+                               {
+                                   alias.Name,
+                                   client.TotalConnectionTime,
+                                   stats.Kills
+                               })
+                                .Take(5);
 
-            mostPlayed.AddRange(iqList.Select(stats => translationLookup["COMMANDS_MOST_PLAYED_FORMAT"].FormatExt(stats.Name, stats.Kills, (DateTime.UtcNow - DateTime.UtcNow.AddSeconds(-stats.TotalConnectionTime)).HumanizeForCurrentCulture())));
+                var iqList = await iqStats.ToListAsync();
+
+                mostPlayed.AddRange(iqList.Select(stats => translationLookup["COMMANDS_MOST_PLAYED_FORMAT"].FormatExt(stats.Name, stats.Kills, (DateTime.UtcNow - DateTime.UtcNow.AddSeconds(-stats.TotalConnectionTime)).HumanizeForCurrentCulture())));
+            }
 
 
             return mostPlayed;
         }
 
         private readonly CommandConfiguration _config;
-        private readonly IDatabaseContextFactory _contextFactory;
 
-        public MostPlayedCommand(CommandConfiguration config, ITranslationLookup translationLookup,
+        public MostPlayedCommand(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup) 
-            IDatabaseContextFactory contextFactory) : base(config, translationLookup) 
         {
             Name = "mostplayed";
             Description = translationLookup["PLUGINS_STATS_COMMANDS_MOSTPLAYED_DESC"];
@@ -66,12 +69,11 @@ namespace IW4MAdmin.Plugins.Stats.Commands
             RequiresTarget = false;
 
             _config = config;
-            _contextFactory = contextFactory;
         }
 
         public override async Task ExecuteAsync(GameEvent E)
         {
-            var topStats = await GetMostPlayed(E.Owner, _translationLookup, _contextFactory);
+            var topStats = await GetMostPlayed(E.Owner, _translationLookup);
             if (!E.Message.IsBroadcastCommand(_config.BroadcastCommandPrefix))
             {
                 foreach (var stat in topStats)

Plugins/Stats/Commands/ResetStats.cs

@@ -12,36 +12,31 @@ namespace IW4MAdmin.Plugins.Stats.Commands
 {
     public class ResetStats : Command
     {
-        private readonly IDatabaseContextFactory _contextFactory;
+        public ResetStats(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
-        
-        public ResetStats(CommandConfiguration config, ITranslationLookup translationLookup, 
-            IDatabaseContextFactory contextFactory) : base(config, translationLookup)
         {
             Name = "resetstats";
             Description = translationLookup["PLUGINS_STATS_COMMANDS_RESET_DESC"];
             Alias = "rs";
             Permission = EFClient.Permission.User;
             RequiresTarget = false;
-            AllowImpersonation = true;
+            //AllowImpersonation = true;
-
-            _contextFactory = contextFactory;
         }
 
-        public override async Task ExecuteAsync(GameEvent gameEvent)
+        public override async Task ExecuteAsync(GameEvent E)
         {
-            if (gameEvent.Origin.ClientNumber >= 0)
+            if (E.Origin.ClientNumber >= 0)
             {
-                var serverId = Helpers.StatManager.GetIdForServer(gameEvent.Owner);
 
-                await using var context = _contextFactory.CreateContext();
+                long serverId = Helpers.StatManager.GetIdForServer(E.Owner);
-                var clientStats = await context.Set<EFClientStatistics>()
-                    .Where(s => s.ClientId == gameEvent.Origin.ClientId)
-                    .Where(s => s.ServerId == serverId)
-                    .FirstOrDefaultAsync();
 
-                // want to prevent resetting stats before they've gotten any kills
+                EFClientStatistics clientStats;
-                if (clientStats != null)
+                using (var ctx = new DatabaseContext(disableTracking: true))
                 {
+                    clientStats = await ctx.Set<EFClientStatistics>()
+                        .Where(s => s.ClientId == E.Origin.ClientId)
+                        .Where(s => s.ServerId == serverId)
+                        .FirstAsync();
+
                     clientStats.Deaths = 0;
                     clientStats.Kills = 0;
                     clientStats.SPM = 0.0;
@@ -49,18 +44,19 @@ namespace IW4MAdmin.Plugins.Stats.Commands
                     clientStats.TimePlayed = 0;
                     // todo: make this more dynamic
                     clientStats.EloRating = 200.0;
-                    await context.SaveChangesAsync();
+
+                    // reset the cached version
+                    Plugin.Manager.ResetStats(E.Origin);
+
+                    // fixme: this doesn't work properly when another context exists
+                    await ctx.SaveChangesAsync();
                 }
-
+                E.Origin.Tell(_translationLookup["PLUGINS_STATS_COMMANDS_RESET_SUCCESS"]);
-                // reset the cached version
-                Plugin.Manager.ResetStats(gameEvent.Origin);
-
-                gameEvent.Origin.Tell(_translationLookup["PLUGINS_STATS_COMMANDS_RESET_SUCCESS"]);
             }
 
             else
             {
-                gameEvent.Origin.Tell(_translationLookup["PLUGINS_STATS_COMMANDS_RESET_FAIL"]);
+                E.Origin.Tell(_translationLookup["PLUGINS_STATS_COMMANDS_RESET_FAIL"]);
             }
         }
     }

Plugins/Stats/Commands/TopStats.cs

@@ -16,7 +16,7 @@ namespace IW4MAdmin.Plugins.Stats.Commands
 {
     class TopStats : Command
     {
-        public static async Task<List<string>> GetTopStats(Server s, ITranslationLookup translationLookup, IDatabaseContextFactory contextFactory)
+        public static async Task<List<string>> GetTopStats(Server s, ITranslationLookup translationLookup)
         {
             long serverId = StatManager.GetIdForServer(s); 
             var topStatsText = new List<string>()
@@ -24,34 +24,36 @@ namespace IW4MAdmin.Plugins.Stats.Commands
                 $"^5--{translationLookup["PLUGINS_STATS_COMMANDS_TOP_TEXT"]}--"
             };
 
-            await using var context = contextFactory.CreateContext(false);
+            using (var db = new DatabaseContext(true))
-            var fifteenDaysAgo = DateTime.UtcNow.AddDays(-15);
+            {
-            int minPlayTime = Plugin.Config.Configuration().TopPlayersMinPlayTime;
+                var fifteenDaysAgo = DateTime.UtcNow.AddDays(-15);
+                int minPlayTime = Plugin.Config.Configuration().TopPlayersMinPlayTime;
 
-            var iqStats = (from stats in context.Set<EFClientStatistics>()
+                var iqStats = (from stats in db.Set<EFClientStatistics>()
-                           join client in context.Clients
+                               join client in db.Clients
-                           on stats.ClientId equals client.ClientId
+                               on stats.ClientId equals client.ClientId
-                           join alias in context.Aliases
+                               join alias in db.Aliases
-                           on client.CurrentAliasId equals alias.AliasId
+                               on client.CurrentAliasId equals alias.AliasId
-                           where stats.ServerId == serverId
+                               where stats.ServerId == serverId
-                           where stats.TimePlayed >= minPlayTime
+                               where stats.TimePlayed >= minPlayTime
-                           where client.Level != EFClient.Permission.Banned
+                               where client.Level != EFClient.Permission.Banned
-                           where client.LastConnection >= fifteenDaysAgo
+                               where client.LastConnection >= fifteenDaysAgo
-                           orderby (stats.EloRating + stats.Skill) / 2.0d descending
+                               orderby (stats.EloRating + stats.Skill) / 2.0d descending
-                           select new
+                               select new
-                           {
+                               {
-                               stats.KDR,
+                                   stats.KDR,
-                               stats.Performance,
+                                   stats.Performance,
-                               alias.Name
+                                   alias.Name
-                           })
+                               })
-                          .Take(5);
+                              .Take(5);
 
-            var statsList = (await iqStats.ToListAsync())
+                var statsList = (await iqStats.ToListAsync())
-                .Select(stats => $"^3{stats.Name}^7 - ^5{stats.KDR} ^7{translationLookup["PLUGINS_STATS_TEXT_KDR"]} | ^5{stats.Performance} ^7{translationLookup["PLUGINS_STATS_COMMANDS_PERFORMANCE"]}");
+                    .Select(stats => $"^3{stats.Name}^7 - ^5{stats.KDR} ^7{translationLookup["PLUGINS_STATS_TEXT_KDR"]} | ^5{stats.Performance} ^7{translationLookup["PLUGINS_STATS_COMMANDS_PERFORMANCE"]}");
 
-            topStatsText.AddRange(statsList);
+                topStatsText.AddRange(statsList);
+            }
 
-                // no one qualified
+            // no one qualified
             if (topStatsText.Count == 1)
             {
                 topStatsText = new List<string>()
@@ -64,10 +66,8 @@ namespace IW4MAdmin.Plugins.Stats.Commands
         }
 
         private readonly CommandConfiguration _config;
-        private readonly IDatabaseContextFactory _contextFactory;
 
-        public TopStats(CommandConfiguration config, ITranslationLookup translationLookup, 
+        public TopStats(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup) 
-            IDatabaseContextFactory contextFactory) : base(config, translationLookup) 
         {
             Name = "topstats";
             Description = translationLookup["PLUGINS_STATS_COMMANDS_TOP_DESC"];
@@ -76,12 +76,11 @@ namespace IW4MAdmin.Plugins.Stats.Commands
             RequiresTarget = false;
 
             _config = config;
-            _contextFactory = contextFactory;
         }
 
         public override async Task ExecuteAsync(GameEvent E)
         {
-            var topStats = await GetTopStats(E.Owner, _translationLookup, _contextFactory);
+            var topStats = await GetTopStats(E.Owner, _translationLookup);
             if (!E.Message.IsBroadcastCommand(_config.BroadcastCommandPrefix))
             {
                 foreach (var stat in topStats)

Plugins/Stats/Commands/ViewStats.cs

@@ -14,11 +14,9 @@ namespace IW4MAdmin.Plugins.Stats.Commands
 {
     public class ViewStatsCommand : Command
     {
-        private readonly IDatabaseContextFactory _contextFactory;
+        public ViewStatsCommand(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
-
-        public ViewStatsCommand(CommandConfiguration config, ITranslationLookup translationLookup,
-            IDatabaseContextFactory contextFactory) : base(config, translationLookup)
         {
+
             Name = "stats";
             Description = translationLookup["PLUGINS_STATS_COMMANDS_VIEW_DESC"];
             Alias = "xlrstats";
@@ -33,13 +31,15 @@ namespace IW4MAdmin.Plugins.Stats.Commands
                 }
             };
 
-            _contextFactory = contextFactory;
+            _config = config;
         }
 
+        private readonly CommandConfiguration _config;
+
         public override async Task ExecuteAsync(GameEvent E)
         {
             string statLine;
-            EFClientStatistics pStats = null;
+            EFClientStatistics pStats;
 
             if (E.Data.Length > 0 && E.Target == null)
             {
@@ -51,67 +51,52 @@ namespace IW4MAdmin.Plugins.Stats.Commands
                 }
             }
 
-            var serverId = StatManager.GetIdForServer(E.Owner);
+            long serverId = StatManager.GetIdForServer(E.Owner);
+
 
-            // getting stats for a particular client
             if (E.Target != null)
             {
-                var performanceRanking = await Plugin.Manager.GetClientOverallRanking(E.Target.ClientId);
+                int performanceRanking = await Plugin.Manager.GetClientOverallRanking(E.Target.ClientId);
-                var performanceRankingString = performanceRanking == 0
+                string performanceRankingString = performanceRanking == 0 ? _translationLookup["WEBFRONT_STATS_INDEX_UNRANKED"] : $"{_translationLookup["WEBFRONT_STATS_INDEX_RANKED"]} #{performanceRanking}";
-                    ? _translationLookup["WEBFRONT_STATS_INDEX_UNRANKED"]
-                    : $"{_translationLookup["WEBFRONT_STATS_INDEX_RANKED"]} #{performanceRanking}";
 
-                // target is currently connected so we want their cached stats if they exist
+                if (E.Owner.GetClientsAsList().Any(_client => _client.Equals(E.Target)))
-                if (E.Owner.GetClientsAsList().Any(client => client.Equals(E.Target)))
                 {
                     pStats = E.Target.GetAdditionalProperty<EFClientStatistics>(StatManager.CLIENT_STATS_KEY);
                 }
 
-                // target is not connected so we want to look up via database
+                else
-                if (pStats == null)
                 {
-                    await using var context = _contextFactory.CreateContext(false);
+                    using (var ctx = new DatabaseContext(true))
-                    pStats = (await context.Set<EFClientStatistics>()
+                    {
-                        .FirstOrDefaultAsync(c => c.ServerId == serverId && c.ClientId == E.Target.ClientId));
+                        pStats = (await ctx.Set<EFClientStatistics>().FirstAsync(c => c.ServerId == serverId && c.ClientId == E.Target.ClientId));
+                    }
                 }
-
+                statLine = $"^5{pStats.Kills} ^7{_translationLookup["PLUGINS_STATS_TEXT_KILLS"]} | ^5{pStats.Deaths} ^7{_translationLookup["PLUGINS_STATS_TEXT_DEATHS"]} | ^5{pStats.KDR} ^7KDR | ^5{pStats.Performance} ^7{_translationLookup["PLUGINS_STATS_COMMANDS_PERFORMANCE"].ToUpper()} | {performanceRankingString}";
-                // if it's still null then they've not gotten a kill or death yet
-                statLine = pStats == null
-                    ? _translationLookup["PLUGINS_STATS_COMMANDS_NOTAVAILABLE"]
-                    : $"^5{pStats.Kills} ^7{_translationLookup["PLUGINS_STATS_TEXT_KILLS"]} | ^5{pStats.Deaths} ^7{_translationLookup["PLUGINS_STATS_TEXT_DEATHS"]} | ^5{pStats.KDR} ^7KDR | ^5{pStats.Performance} ^7{_translationLookup["PLUGINS_STATS_COMMANDS_PERFORMANCE"].ToUpper()} | {performanceRankingString}";
             }
 
-            // getting self stats
             else
             {
-                var performanceRanking = await Plugin.Manager.GetClientOverallRanking(E.Origin.ClientId);
+                int performanceRanking = await Plugin.Manager.GetClientOverallRanking(E.Origin.ClientId);
-                var performanceRankingString = performanceRanking == 0
+                string performanceRankingString = performanceRanking == 0 ? _translationLookup["WEBFRONT_STATS_INDEX_UNRANKED"] : $"{_translationLookup["WEBFRONT_STATS_INDEX_RANKED"]} #{performanceRanking}";
-                    ? _translationLookup["WEBFRONT_STATS_INDEX_UNRANKED"]
-                    : $"{_translationLookup["WEBFRONT_STATS_INDEX_RANKED"]} #{performanceRanking}";
 
-                // check if current client is connected to the server
+                if (E.Owner.GetClientsAsList().Any(_client => _client.Equals(E.Origin)))
-                if (E.Owner.GetClientsAsList().Any(client => client.Equals(E.Origin)))
                 {
                     pStats = E.Origin.GetAdditionalProperty<EFClientStatistics>(StatManager.CLIENT_STATS_KEY);
                 }
 
-                // happens if the user has not gotten a kill/death since connecting
+                else
-                if (pStats == null)
                 {
-                    await using var context = _contextFactory.CreateContext(false);
+                    using (var ctx = new DatabaseContext(true))
-                    pStats = (await context.Set<EFClientStatistics>()
+                    {
-                        .FirstOrDefaultAsync(c => c.ServerId == serverId && c.ClientId == E.Origin.ClientId));
+                        pStats = (await ctx.Set<EFClientStatistics>().FirstAsync(c => c.ServerId == serverId && c.ClientId == E.Origin.ClientId));
+                    }
                 }
-
+                statLine = $"^5{pStats.Kills} ^7{_translationLookup["PLUGINS_STATS_TEXT_KILLS"]} | ^5{pStats.Deaths} ^7{_translationLookup["PLUGINS_STATS_TEXT_DEATHS"]} | ^5{pStats.KDR} ^7KDR | ^5{pStats.Performance} ^7{_translationLookup["PLUGINS_STATS_COMMANDS_PERFORMANCE"].ToUpper()} | {performanceRankingString}";
-                // if it's still null then they've not gotten a kill or death yet
-                statLine = pStats == null
-                    ? _translationLookup["PLUGINS_STATS_COMMANDS_NOTAVAILABLE"]
-                    : $"^5{pStats.Kills} ^7{_translationLookup["PLUGINS_STATS_TEXT_KILLS"]} | ^5{pStats.Deaths} ^7{_translationLookup["PLUGINS_STATS_TEXT_DEATHS"]} | ^5{pStats.KDR} ^7KDR | ^5{pStats.Performance} ^7{_translationLookup["PLUGINS_STATS_COMMANDS_PERFORMANCE"].ToUpper()} | {performanceRankingString}";
             }
 
             if (E.Message.IsBroadcastCommand(_config.BroadcastCommandPrefix))
             {
-                var name = E.Target == null ? E.Origin.Name : E.Target.Name;
+                string name = E.Target == null ? E.Origin.Name : E.Target.Name;
                 E.Owner.Broadcast(_translationLookup["PLUGINS_STATS_COMMANDS_VIEW_SUCCESS"].FormatExt(name));
                 E.Owner.Broadcast(statLine);
             }

Plugins/Stats/Config/AnticheatConfiguration.cs

@@ -15,7 +15,7 @@ namespace Stats.Config
                 Game.IW4, new Dictionary<DetectionType, string[]>
                 {
                     { DetectionType.Chest, new[] { "m21.+" } },
-                    { DetectionType.Recoil, new[] { "ranger.*_mp", "model1887.*_mp", ".+shotgun.*_mp", "turret_minigun_mp" } },
+                    { DetectionType.Recoil, new[] { "ranger.*_mp", "model1887.*_mp", ".+shotgun.*_mp" } },
                     { DetectionType.Button, new[] { ".*akimbo.*" } }
                 }
             }

Plugins/Stats/Helpers/MigrationHelper.cs

@@ -1,7 +0,0 @@
-namespace Stats.Helpers
-{
-    public class MigrationHelper
-    {
-        
-    }
-}

Plugins/Stats/Helpers/StatsResourceQueryHelper.cs

@@ -26,7 +26,7 @@ namespace Stats.Helpers
         public async Task<ResourceQueryHelperResult<StatsInfoResult>> QueryResource(StatsInfoRequest query)
         {
             var result = new ResourceQueryHelperResult<StatsInfoResult>();
-            await using var context = _contextFactory.CreateContext(enableTracking: false);
+            using var context = _contextFactory.CreateContext(enableTracking: false);
 
             // we need to get the ratings separately because there's not explicit FK
             var ratings = await context.Set<EFClientRatingHistory>()

Plugins/Stats/Models/EFClientMessage.cs

@@ -17,6 +17,5 @@ namespace IW4MAdmin.Plugins.Stats.Models
         public virtual EFClient Client { get; set; }
         public string Message { get; set; }
         public DateTime TimeSent { get; set; }
-        public bool SentIngame { get; set; }
     }
 }

Plugins/Stats/Models/ModelConfiguration.cs

@@ -23,13 +23,8 @@ namespace Stats.Models
             builder.Entity<EFRating>()
                 .HasIndex(p => new { p.Performance, p.Ranking, p.When });
 
-            builder.Entity<EFRating>()
+            builder.Entity<EFClientMessage>()
-                .HasIndex(p => new { p.When, p.ServerId, p.Performance, p.ActivityAmount });
+                .HasIndex(p => p.TimeSent);
-
-            builder.Entity<EFClientMessage>(message =>
-            {
-                message.HasIndex(p => p.TimeSent);
-            });
 
             // force pluralization
             builder.Entity<EFClientKill>().ToTable("EFClientKills");

Plugins/Stats/Plugin.cs

@@ -14,8 +14,6 @@ using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using SharedLibraryCore.Commands;
 
 namespace IW4MAdmin.Plugins.Stats
 {
@@ -30,24 +28,23 @@ namespace IW4MAdmin.Plugins.Stats
         public static StatManager Manager { get; private set; }
         public static IManager ServerManager;
         public static IConfigurationHandler<StatsConfiguration> Config { get; private set; }
-
+#if DEBUG
+        int scriptDamageCount;
+        int scriptKillCount;
+#endif
         private readonly IDatabaseContextFactory _databaseContextFactory;
         private readonly ITranslationLookup _translationLookup;
         private readonly IMetaService _metaService;
         private readonly IResourceQueryHelper<ChatSearchQuery, MessageResponse> _chatQueryHelper;
-        private readonly ILogger<StatManager> _managerLogger;
-        private readonly ILogger<Plugin> _logger;
 
-        public Plugin(ILogger<Plugin> logger, IConfigurationHandlerFactory configurationHandlerFactory, IDatabaseContextFactory databaseContextFactory,
+        public Plugin(IConfigurationHandlerFactory configurationHandlerFactory, IDatabaseContextFactory databaseContextFactory,
-            ITranslationLookup translationLookup, IMetaService metaService, IResourceQueryHelper<ChatSearchQuery, MessageResponse> chatQueryHelper, ILogger<StatManager> managerLogger)
+            ITranslationLookup translationLookup, IMetaService metaService, IResourceQueryHelper<ChatSearchQuery, MessageResponse> chatQueryHelper)
         {
             Config = configurationHandlerFactory.GetConfigurationHandler<StatsConfiguration>("StatsPluginSettings");
             _databaseContextFactory = databaseContextFactory;
             _translationLookup = translationLookup;
             _metaService = metaService;
             _chatQueryHelper = chatQueryHelper;
-            _managerLogger = managerLogger;
-            _logger = logger;
         }
 
         public async Task OnEventAsync(GameEvent E, Server S)
@@ -57,6 +54,11 @@ namespace IW4MAdmin.Plugins.Stats
                 case GameEvent.EventType.Start:
                     Manager.AddServer(S);
                     break;
+                case GameEvent.EventType.Stop:
+                    break;
+                case GameEvent.EventType.PreConnect:
+                    await Manager.AddPlayer(E.Origin);
+                    break;
                 case GameEvent.EventType.Disconnect:
                     await Manager.RemovePlayer(E.Origin);
                     break;
@@ -64,7 +66,7 @@ namespace IW4MAdmin.Plugins.Stats
                     if (!string.IsNullOrEmpty(E.Data) &&
                         E.Origin.ClientId > 1)
                     {
-                        await Manager.AddMessageAsync(E.Origin.ClientId, StatManager.GetIdForServer(S), true, E.Data);
+                        await Manager.AddMessageAsync(E.Origin.ClientId, StatManager.GetIdForServer(S), E.Data);
                     }
                     break;
                 case GameEvent.EventType.MapChange:
@@ -75,13 +77,21 @@ namespace IW4MAdmin.Plugins.Stats
                 case GameEvent.EventType.MapEnd:
                     await Manager.Sync(S);
                     break;
-                case GameEvent.EventType.Command:
+                case GameEvent.EventType.JoinTeam:
-                    var shouldPersist = !string.IsNullOrEmpty(E.Data) &&
+                    break;
-                                        E.Extra is SayCommand;
+                case GameEvent.EventType.Broadcast:
-                    if (shouldPersist)
+                    break;
-                    {
+                case GameEvent.EventType.Tell:
-                        await Manager.AddMessageAsync(E.Origin.ClientId, StatManager.GetIdForServer(S), false, E.Data);
+                    break;
-                    }
+                case GameEvent.EventType.Kick:
+                    break;
+                case GameEvent.EventType.Ban:
+                    break;
+                case GameEvent.EventType.Unknown:
+                    break;
+                case GameEvent.EventType.Report:
+                    break;
+                case GameEvent.EventType.Flag:
                     break;
                 case GameEvent.EventType.ScriptKill:
                     string[] killInfo = (E.Data != null) ? E.Data.Split(';') : new string[0];
@@ -93,14 +103,22 @@ namespace IW4MAdmin.Plugins.Stats
                             E.Origin = E.Target;
                         }
 
-                        await EnsureClientsAdded(E.Origin, E.Target);
+#if DEBUG
+                        scriptKillCount++;
+                        S.Logger.WriteInfo($"Start ScriptKill {scriptKillCount}");
+#endif
+
                         await Manager.AddScriptHit(false, E.Time, E.Origin, E.Target, StatManager.GetIdForServer(S), S.CurrentMap.Name, killInfo[7], killInfo[8],
                             killInfo[5], killInfo[6], killInfo[3], killInfo[4], killInfo[9], killInfo[10], killInfo[11], killInfo[12], killInfo[13], killInfo[14], killInfo[15], killInfo[16], killInfo[17]);
+
+#if DEBUG
+                        S.Logger.WriteInfo($"End ScriptKill {scriptKillCount}");
+#endif
                     }
 
                     else
                     {
-                        _logger.LogDebug("Skipping script kill as it is ignored or data in customcallbacks is outdated/missing");
+                        S.Logger.WriteDebug("Skipping script kill as it is ignored or data in customcallbacks is outdated/missing");
                     }
                     break;
                 case GameEvent.EventType.Kill:
@@ -112,7 +130,6 @@ namespace IW4MAdmin.Plugins.Stats
                             E.Origin = E.Target;
                         }
 
-                        await EnsureClientsAdded(E.Origin, E.Target);
                         await Manager.AddStandardKill(E.Origin, E.Target);
                     }
                     break;
@@ -138,14 +155,22 @@ namespace IW4MAdmin.Plugins.Stats
                             E.Origin = E.Target;
                         }
 
-                        await EnsureClientsAdded(E.Origin, E.Target);
+#if DEBUG
+                        scriptDamageCount++;
+                        S.Logger.WriteInfo($"Start ScriptDamage {scriptDamageCount}");
+#endif
+
                         await Manager.AddScriptHit(true, E.Time, E.Origin, E.Target, StatManager.GetIdForServer(S), S.CurrentMap.Name, killInfo[7], killInfo[8],
                             killInfo[5], killInfo[6], killInfo[3], killInfo[4], killInfo[9], killInfo[10], killInfo[11], killInfo[12], killInfo[13], killInfo[14], killInfo[15], killInfo[16], killInfo[17]);
+
+#if DEBUG
+                        S.Logger.WriteInfo($"End ScriptDamage {scriptDamageCount}");
+#endif
                     }
 
                     else
                     {
-                        _logger.LogDebug("Skipping script damage as it is ignored or data in customcallbacks is outdated/missing");
+                        S.Logger.WriteDebug("Skipping script damage as it is ignored or data in customcallbacks is outdated/missing");
                     }
                     break;
             }
@@ -173,9 +198,11 @@ namespace IW4MAdmin.Plugins.Stats
             {
                 IList<EFClientStatistics> clientStats;
                 int messageCount = 0;
-                await using var ctx = _databaseContextFactory.CreateContext(enableTracking: false);
+                using (var ctx = _databaseContextFactory.CreateContext(enableTracking: false))
-                clientStats = await ctx.Set<EFClientStatistics>().Where(c => c.ClientId == request.ClientId).ToListAsync();
+                {
-                messageCount = await ctx.Set<EFClientMessage>().CountAsync(_message => _message.ClientId == request.ClientId);
+                    clientStats = await ctx.Set<EFClientStatistics>().Where(c => c.ClientId == request.ClientId).ToListAsync();
+                    messageCount = await ctx.Set<EFClientMessage>().CountAsync(_message => _message.ClientId == request.ClientId);
+                }
 
                 int kills = clientStats.Sum(c => c.Kills);
                 int deaths = clientStats.Sum(c => c.Deaths);
@@ -250,11 +277,13 @@ namespace IW4MAdmin.Plugins.Stats
             {
                 IList<EFClientStatistics> clientStats;
 
-                await using var ctx = _databaseContextFactory.CreateContext(enableTracking: false);
+                using (var ctx = _databaseContextFactory.CreateContext(enableTracking: false))
-                clientStats = await ctx.Set<EFClientStatistics>()
+                {
-                    .Include(c => c.HitLocations)
+                    clientStats = await ctx.Set<EFClientStatistics>()
-                    .Where(c => c.ClientId == request.ClientId)
+                        .Include(c => c.HitLocations)
-                    .ToListAsync();
+                        .Where(c => c.ClientId == request.ClientId)
+                        .ToListAsync();
+                }
 
                 double headRatio = 0;
                 double chestRatio = 0;
@@ -387,28 +416,32 @@ namespace IW4MAdmin.Plugins.Stats
 
             async Task<string> totalKills(Server server)
             {
-                await using var context = _databaseContextFactory.CreateContext(false);
+                using (var ctx = new DatabaseContext(disableTracking: true))
-                long kills = await context.Set<EFServerStatistics>().Where(s => s.Active).SumAsync(s => s.TotalKills);
+                {
-                return kills.ToString("#,##0", new System.Globalization.CultureInfo(Utilities.CurrentLocalization.LocalizationName));
+                    long kills = await ctx.Set<EFServerStatistics>().Where(s => s.Active).SumAsync(s => s.TotalKills);
+                    return kills.ToString("#,##0", new System.Globalization.CultureInfo(Utilities.CurrentLocalization.LocalizationName));
+                }
             }
 
             async Task<string> totalPlayTime(Server server)
             {
-                await using var context = _databaseContextFactory.CreateContext(false);
+                using (var ctx = new DatabaseContext(disableTracking: true))
-                long playTime = await context.Set<EFServerStatistics>().Where(s => s.Active).SumAsync(s => s.TotalPlayTime);
+                {
-                return (playTime / 3600.0).ToString("#,##0", new System.Globalization.CultureInfo(Utilities.CurrentLocalization.LocalizationName));
+                    long playTime = await ctx.Set<EFServerStatistics>().Where(s => s.Active).SumAsync(s => s.TotalPlayTime);
+                    return (playTime / 3600.0).ToString("#,##0", new System.Globalization.CultureInfo(Utilities.CurrentLocalization.LocalizationName));
+                }
             }
 
             async Task<string> topStats(Server s)
             {
                 // todo: this needs to needs to be updated when we DI the lookup
-                return string.Join(Environment.NewLine, await Commands.TopStats.GetTopStats(s, Utilities.CurrentLocalization.LocalizationIndex, _databaseContextFactory));
+                return string.Join(Environment.NewLine, await Commands.TopStats.GetTopStats(s, Utilities.CurrentLocalization.LocalizationIndex));
             }
 
             async Task<string> mostPlayed(Server s)
             {
                 // todo: this needs to needs to be updated when we DI the lookup
-                return string.Join(Environment.NewLine, await Commands.MostPlayedCommand.GetMostPlayed(s, Utilities.CurrentLocalization.LocalizationIndex, _databaseContextFactory));
+                return string.Join(Environment.NewLine, await Commands.MostPlayedCommand.GetMostPlayed(s, Utilities.CurrentLocalization.LocalizationIndex));
             }
 
             async Task<string> mostKills(Server gameServer)
@@ -424,7 +457,7 @@ namespace IW4MAdmin.Plugins.Stats
             manager.GetMessageTokens().Add(new MessageToken("MOSTKILLS", mostKills));
 
             ServerManager = manager;
-            Manager = new StatManager(_managerLogger, manager, _databaseContextFactory, Config);
+            Manager = new StatManager(manager, _databaseContextFactory, Config);
         }
 
         public Task OnTickAsync(Server S)
@@ -465,21 +498,5 @@ namespace IW4MAdmin.Plugins.Stats
         /// <param name="s"></param>
         /// <returns></returns>
         private bool ShouldOverrideAnticheatSetting(Server s) => Config.Configuration().AnticheatConfiguration.Enable && s.GameName == Server.Game.IW5;
-
-        /// <summary>
-        /// Makes sure both clients are added
-        /// </summary>
-        /// <param name="origin"></param>
-        /// <param name="target"></param>
-        /// <returns></returns>
-        private async Task EnsureClientsAdded(EFClient origin, EFClient target)
-        {
-            await Manager.AddPlayer(origin);
-
-            if (!origin.Equals(target))
-            {
-                await Manager.AddPlayer(target);
-            }
-        }
     }
 }

Plugins/Stats/Stats.csproj

@@ -13,15 +13,13 @@
     <Copyright>2018</Copyright>
     <Configurations>Debug;Release;Prerelease</Configurations>
     <LangVersion>8.0</LangVersion>
-    <IsPackable>false</IsPackable>
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2020.12.20.1" PrivateAssets="All" />
+    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2.4.9" PrivateAssets="All" />
   </ItemGroup>
 
   <Target Name="PostBuild" AfterTargets="PostBuildEvent">
     <Exec Command="dotnet publish $(ProjectPath) -c $(ConfigurationName) -o $(ProjectDir)..\..\Build\Plugins --no-build --no-restore --no-dependencies" />
   </Target>
-
 </Project>

Plugins/Web/StatsWeb/API/StatsController.cs

@@ -1,12 +1,11 @@
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Mvc;
+using SharedLibraryCore;
 using SharedLibraryCore.Dtos;
 using SharedLibraryCore.Interfaces;
 using Stats.Dtos;
 using System;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace StatsWeb.API
 {
@@ -17,7 +16,7 @@ namespace StatsWeb.API
         private readonly ILogger _logger;
         private readonly IResourceQueryHelper<StatsInfoRequest, StatsInfoResult> _statsQueryHelper;
 
-        public StatsController(ILogger<StatsController> logger, IResourceQueryHelper<StatsInfoRequest, StatsInfoResult> statsQueryHelper)
+        public StatsController(ILogger logger, IResourceQueryHelper<StatsInfoRequest, StatsInfoResult> statsQueryHelper)
         {
             _statsQueryHelper = statsQueryHelper;
             _logger = logger;
@@ -57,7 +56,8 @@ namespace StatsWeb.API
 
             catch (Exception e)
             {
-                _logger.LogWarning(e, "Could not get client stats for client id {clientId}", clientId);
+                _logger.WriteWarning($"Could not get client stats for client id {clientId}");
+                _logger.WriteDebug(e.GetExceptionInfo());
 
                 return StatusCode(StatusCodes.Status500InternalServerError, new ErrorResponse
                 {

Plugins/Web/StatsWeb/ChatResourceQueryHelper.cs

@@ -10,8 +10,6 @@ using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace StatsWeb
 {
@@ -25,7 +23,7 @@ namespace StatsWeb
         private readonly ApplicationConfiguration _appConfig;
         private List<EFServer> serverCache;
 
-        public ChatResourceQueryHelper(ILogger<ChatResourceQueryHelper> logger, IDatabaseContextFactory contextFactory, ApplicationConfiguration appConfig)
+        public ChatResourceQueryHelper(ILogger logger, IDatabaseContextFactory contextFactory, ApplicationConfiguration appConfig)
         {
             _contextFactory = contextFactory;
             _logger = logger;
@@ -41,7 +39,7 @@ namespace StatsWeb
             }
 
             var result = new ResourceQueryHelperResult<MessageResponse>();
-            await using var context = _contextFactory.CreateContext(enableTracking: false);
+            using var context = _contextFactory.CreateContext(enableTracking: false);
 
             if (serverCache == null)
             {
@@ -81,8 +79,7 @@ namespace StatsWeb
                     When = _message.TimeSent,
                     Message = _message.Message,
                     ServerName = query.IsProfileMeta ? "" : _message.Server.HostName,
-                    GameName = _message.Server.GameName == null ? Server.Game.IW4 : _message.Server.GameName.Value,
+                    GameName = _message.Server.GameName == null ? Server.Game.IW4 : _message.Server.GameName.Value
-                    SentIngame = _message.SentIngame
                 });
 
             if (query.Direction == SharedLibraryCore.Dtos.SortDirection.Descending)

Plugins/Web/StatsWeb/Controllers/StatsController.cs

@@ -10,11 +10,8 @@ using SharedLibraryCore.Interfaces;
 using Stats.Dtos;
 using StatsWeb.Extensions;
 using System;
-using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace IW4MAdmin.Plugins.Web.StatsWeb.Controllers
 {
@@ -24,17 +21,14 @@ namespace IW4MAdmin.Plugins.Web.StatsWeb.Controllers
         private readonly IManager _manager;
         private readonly IResourceQueryHelper<ChatSearchQuery, MessageResponse> _chatResourceQueryHelper;
         private readonly ITranslationLookup _translationLookup;
-        private readonly IDatabaseContextFactory _contextFactory;
 
-        public StatsController(ILogger<StatsController> logger, IManager manager, IResourceQueryHelper<ChatSearchQuery, 
+        public StatsController(ILogger logger, IManager manager, IResourceQueryHelper<ChatSearchQuery, MessageResponse> resourceQueryHelper,
-                MessageResponse> resourceQueryHelper, ITranslationLookup translationLookup, 
+            ITranslationLookup translationLookup) : base(manager)
-            IDatabaseContextFactory contextFactory) : base(manager)
         {
             _logger = logger;
             _manager = manager;
             _chatResourceQueryHelper = resourceQueryHelper;
             _translationLookup = translationLookup;
-            _contextFactory = contextFactory;
         }
 
         [HttpGet]
@@ -92,7 +86,7 @@ namespace IW4MAdmin.Plugins.Web.StatsWeb.Controllers
                 SentAfter = whenLower
             });
 
-            return View("_MessageContext", messages.Results.ToList());
+            return View("_MessageContext", messages.Results);
         }
 
         [HttpGet("Message/Find")]
@@ -114,13 +108,15 @@ namespace IW4MAdmin.Plugins.Web.StatsWeb.Controllers
 
             catch (ArgumentException e)
             {
-                _logger.LogWarning(e, "Could not parse chat message search query {query}", query);
+                _logger.WriteWarning($"Could not parse chat message search query - {query}");
+                _logger.WriteDebug(e.GetExceptionInfo());
                 ViewBag.Error = e;
             }
 
             catch (FormatException e)
             {
-                _logger.LogWarning(e, "Could not parse chat message search query filter format {query}", query);
+                _logger.WriteWarning($"Could not parse chat message search query filter format - {query}");
+                _logger.WriteDebug(e.GetExceptionInfo());
                 ViewBag.Error = e;
             }
 
@@ -140,13 +136,15 @@ namespace IW4MAdmin.Plugins.Web.StatsWeb.Controllers
 
             catch (ArgumentException e)
             {
-                _logger.LogWarning(e, "Could not parse chat message search query {query}", query);
+                _logger.WriteWarning($"Could not parse chat message search query - {query}");
+                _logger.WriteDebug(e.GetExceptionInfo());
                 throw;
             }
 
             catch (FormatException e)
             {
-                _logger.LogWarning(e, "Could not parse chat message search query filter format {query}", query);
+                _logger.WriteWarning($"Could not parse chat message search query filter format - {query}");
+                _logger.WriteDebug(e.GetExceptionInfo());
                 throw;
             }
 
@@ -158,48 +156,49 @@ namespace IW4MAdmin.Plugins.Web.StatsWeb.Controllers
         [Authorize]
         public async Task<IActionResult> GetAutomatedPenaltyInfoAsync(int penaltyId)
         {
-            await using var context = _contextFactory.CreateContext(false);
+            using (var ctx = new SharedLibraryCore.Database.DatabaseContext(true))
-            
-            var penalty = await context.Penalties
-                .Select(_penalty => new { _penalty.OffenderId, _penalty.PenaltyId, _penalty.When, _penalty.AutomatedOffense })
-                .FirstOrDefaultAsync(_penalty => _penalty.PenaltyId == penaltyId);
-
-            if (penalty == null)
             {
-                return NotFound();
+                var penalty = await ctx.Penalties
-            }
+                    .Select(_penalty => new { _penalty.OffenderId, _penalty.PenaltyId, _penalty.When, _penalty.AutomatedOffense })
+                    .FirstOrDefaultAsync(_penalty => _penalty.PenaltyId == penaltyId);
 
-            // todo: this can be optimized
+                if (penalty == null)
-            var iqSnapshotInfo = context.Set<Stats.Models.EFACSnapshot>()
-                .Where(s => s.ClientId == penalty.OffenderId)
-                .Include(s => s.LastStrainAngle)
-                .Include(s => s.HitOrigin)
-                .Include(s => s.HitDestination)
-                .Include(s => s.CurrentViewAngle)
-                .Include(s => s.PredictedViewAngles)
-                .ThenInclude(_angles => _angles.Vector)
-                .OrderBy(s => s.When)
-                .ThenBy(s => s.Hits);
-
-            var penaltyInfo = await iqSnapshotInfo.ToListAsync();
-
-            if (penaltyInfo.Count > 0)
-            {
-                return View("_PenaltyInfo", penaltyInfo);
-            }
-
-            // we want to show anything related to the automated offense 
-            else
-            {
-                return View("_MessageContext", new List<MessageResponse>
                 {
-                    new MessageResponse()
+                    return NotFound();
+                }
+
+                // todo: this can be optimized
+                var iqSnapshotInfo = ctx.Set<Stats.Models.EFACSnapshot>()
+                    .Where(s => s.ClientId == penalty.OffenderId)
+                    .Include(s => s.LastStrainAngle)
+                    .Include(s => s.HitOrigin)
+                    .Include(s => s.HitDestination)
+                    .Include(s => s.CurrentViewAngle)
+                    .Include(s => s.PredictedViewAngles)
+                    .ThenInclude(_angles => _angles.Vector)
+                    .OrderBy(s => s.When)
+                    .ThenBy(s => s.Hits);
+
+                var penaltyInfo = await iqSnapshotInfo.ToListAsync();
+
+                if (penaltyInfo.Count > 0)
+                {
+                    return View("_PenaltyInfo", penaltyInfo);
+                }
+
+                // we want to show anything related to the automated offense 
+                else
+                {
+                    return View("_MessageContext", new[]
                     {
-                        ClientId = penalty.OffenderId,
+                        new ChatInfo()
-                        Message = penalty.AutomatedOffense,
+                        {
-                        When = penalty.When
+                            ClientId = penalty.OffenderId,
-                    }
+                            Message = penalty.AutomatedOffense,
-                });
+                            Time = penalty.When
+                        }
+                    });
+                }
             }
         }
     }

Plugins/Web/StatsWeb/StatsWeb.csproj

@@ -14,7 +14,7 @@
     <RunPostBuildEvent>Always</RunPostBuildEvent>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2020.12.20.1" PrivateAssets="All" />
+    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2.4.9" PrivateAssets="All" />
   </ItemGroup>
 
   <ItemGroup>

Plugins/Web/StatsWeb/Views/Stats/Components/TopPlayers/_List.cshtml

@@ -6,37 +6,21 @@
     string rankIcon(double elo)
     {
         if (elo >= getDeviation(-0.75) && elo < getDeviation(1.25))
-        {
             return "0_no-place/menu_div_no_place.png";
-        }
         if (elo >= getDeviation(0.125) && elo < getDeviation(0.625))
-        {
             return "1_iron/menu_div_iron_sub03.png";
-        }
         if (elo >= getDeviation(0.625) && elo < getDeviation(1.0))
-        {
             return "2_bronze/menu_div_bronze_sub03.png";
-        }
         if (elo >= getDeviation(1.0) && elo < getDeviation(1.25))
-        {
             return "3_silver/menu_div_silver_sub03.png";
-        }
         if (elo >= getDeviation(1.25) && elo < getDeviation(1.5))
-        {
             return "4_gold/menu_div_gold_sub03.png";
-        }
         if (elo >= getDeviation(1.5) && elo < getDeviation(1.75))
-        {
             return "5_platinum/menu_div_platinum_sub03.png";
-        }
         if (elo >= getDeviation(1.75) && elo < getDeviation(2.0))
-        {
             return "6_semipro/menu_div_semipro_sub03.png";
-        }
         if (elo >= getDeviation(2.0))
-        {
             return "7_pro/menu_div_pro_sub03.png";
-        }
 
         return "0_no-place/menu_div_no_place.png";
     }

Plugins/Web/StatsWeb/Views/Stats/_MessageContext.cshtml

@@ -1,5 +1,5 @@
 @using SharedLibraryCore.Dtos.Meta.Responses
-@model IList<MessageResponse>
+@model IEnumerable<MessageResponse>
 @{
     Layout = null;
 }

Plugins/Welcome/Plugin.cs

@@ -65,12 +65,10 @@ namespace IW4MAdmin.Plugins.Welcome
         public string Name => "Welcome Plugin";
 
         private readonly IConfigurationHandler<WelcomeConfiguration> _configHandler;
-        private readonly IDatabaseContextFactory _contextFactory;
 
-        public Plugin(IConfigurationHandlerFactory configurationHandlerFactory, IDatabaseContextFactory contextFactory)
+        public Plugin(IConfigurationHandlerFactory configurationHandlerFactory)
         {
             _configHandler = configurationHandlerFactory.GetConfigurationHandler<WelcomeConfiguration>("WelcomePluginSettings");
-            _contextFactory = contextFactory;
         }
 
         public async Task OnLoadAsync(IManager manager)
@@ -100,9 +98,9 @@ namespace IW4MAdmin.Plugins.Welcome
                 {
                     string penaltyReason;
 
-                    await using var context = _contextFactory.CreateContext(false);
+                    using (var ctx = new DatabaseContext(disableTracking: true))
                     {
-                        penaltyReason = await context.Penalties
+                        penaltyReason = await ctx.Penalties
                             .Where(p => p.OffenderId == newPlayer.ClientId && p.Type == EFPenalty.PenaltyType.Flag)
                             .OrderByDescending(p => p.When)
                             .Select(p => p.AutomatedOffense ?? p.Offense)

Plugins/Welcome/Welcome.csproj

@@ -12,11 +12,11 @@
     <Description>Welcome plugin for IW4MAdmin welcomes clients to the server</Description>
     <Copyright>2018</Copyright>
     <Configurations>Debug;Release;Prerelease</Configurations>
-    <LangVersion>Latest</LangVersion>
+    <LangVersion>7.1</LangVersion>
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2020.12.20.1" PrivateAssets="All" />
+    <PackageReference Include="RaidMax.IW4MAdmin.SharedLibraryCore" Version="2.4.9" PrivateAssets="All" />
   </ItemGroup>
 
   <Target Name="PostBuild" AfterTargets="PostBuildEvent">

README.md

@@ -3,7 +3,7 @@
 [![ko-fi](https://www.ko-fi.com/img/githubbutton_sm.svg)](https://ko-fi.com/J3J821KUJ)
 
 ## About
-**IW4MAdmin** is an administration tool for [IW4x](https://iw4x.org/), [IW6x](https://xlabs.dev), [Pluto T6](https://forum.plutonium.pw/category/6/plutonium-t6), [Pluto IW5](https://forum.plutonium.pw/category/14/plutonium-iw5), [CoD4x](https://cod4x.me/), [TeknoMW3](https://github.com/Musta1337/TeknoMW3), and most Call of Duty® dedicated servers. It allows complete control of your server; from changing maps, to banning players, **IW4MAdmin** monitors and records activity on your server(s). With plugin support, extending its functionality is a breeze.
+**IW4MAdmin** is an administration tool for [IW4x](https://iw4x.org/), [Pluto T6](https://forum.plutonium.pw/category/6/plutonium-t6), [Pluto IW5](https://forum.plutonium.pw/category/14/plutonium-iw5), [CoD4x](https://cod4x.me/),  [TeknoMW3](https://github.com/Musta1337/TeknoMW3), and most Call of Duty® dedicated servers. It allows complete control of your server; from changing maps, to banning players, **IW4MAdmin** monitors and records activity on your server(s). With plugin support, extending its functionality is a breeze.
 ### Download
 Latest binary builds are always available at:
 - [GitHub](https://github.com/RaidMax/IW4M-Admin/releases)

RunPublishPre.cmd

@@ -0,0 +1,7 @@
+dotnet publish WebfrontCore/WebfrontCore.csproj -c Prerelease -f netcoreapp3.0 --force -o X:\IW4MAdmin\Publish\WindowsPrerelease /p:PublishProfile=Prerelease
+dotnet publish Application/Application.csproj -c Prerelease -f netcoreapp3.0 --force -o X:\IW4MAdmin\Publish\WindowsPrerelease /p:PublishProfile=Prerelease
+dotnet publish GameLogServer/GameLogServer.pyproj -c Release -f netcoreapp3.0 --force -o X:\IW4MAdmin\Publish\WindowsPrerelease\GameLogServer
+call "C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\Tools\VsDevCmd.bat"
+msbuild  GameLogServer/GameLogServer.pyproj  /p:PublishProfile=PreRelease /p:DeployOnBuild=true /p:PublishProfileRootFolder=X:\IW4MAdmin\GameLogServer\
+cd "X:\IW4MAdmin\DEPLOY\"
+PowerShell ".\upload_prerelease.ps1"

RunPublishRelease.cmd

@@ -0,0 +1,7 @@
+dotnet publish WebfrontCore/WebfrontCore.csproj -c Release -f netcoreapp3.0 --force -o X:\IW4MAdmin\Publish\Windows /p:PublishProfile=sTABLE
+dotnet publish Application/Application.csproj -c Release -f netcoreapp3.0 --force -o X:\IW4MAdmin\Publish\Windows /p:PublishProfile=Stable
+dotnet publish GameLogServer/GameLogServer.pyproj -c Release -f netcoreapp3.0 --force -o X:\IW4MAdmin\Publish\WindowsPrerelease\GameLogServer
+call "C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\Tools\VsDevCmd.bat"
+msbuild  GameLogServer/GameLogServer.pyproj  /p:PublishProfile=Stable /p:DeployOnBuild=true /p:PublishProfileRootFolder=X:\IW4MAdmin\GameLogServer\
+cd "X:\IW4MAdmin\DEPLOY\"
+PowerShell ".\upload_release.ps1"

SharedLibraryCore/BaseController.cs

@@ -134,7 +134,6 @@ namespace SharedLibraryCore
             ViewBag.Localization = Utilities.CurrentLocalization.LocalizationIndex;
             ViewBag.CustomBranding = Manager.GetApplicationSettings().Configuration().WebfrontCustomBranding ?? "IW4MAdmin";
             ViewBag.EnableColorCodes = Manager.GetApplicationSettings().Configuration().EnableColorCodes;
-            ViewBag.EnablePrivilegedUserPrivacy = Manager.GetApplicationSettings().Configuration().EnablePrivilegedUserPrivacy;
 
             base.OnActionExecuting(context);
         }

SharedLibraryCore/Command.cs

@@ -6,7 +6,6 @@ using SharedLibraryCore.Configuration;
 using SharedLibraryCore.Database.Models;
 using SharedLibraryCore.Interfaces;
 using static SharedLibraryCore.Server;
-using ILogger = Microsoft.Extensions.Logging.ILogger;
 
 namespace SharedLibraryCore
 {

SharedLibraryCore/Commands/NativeCommands.cs

@@ -12,8 +12,6 @@ using System.Security.Cryptography;
 using System.Text;
 using System.Text.RegularExpressions;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using Serilog.Context;
 using static SharedLibraryCore.Database.Models.EFClient;
 
 namespace SharedLibraryCore.Commands
@@ -101,8 +99,7 @@ namespace SharedLibraryCore.Commands
     /// </summary>
     public class WarnCommand : Command
     {
-        private readonly ApplicationConfiguration _appConfig;
+        public WarnCommand(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
-        public WarnCommand(ApplicationConfiguration appConfig, CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
         {
             Name = "warn";
             Description = _translationLookup["COMMANDS_WARN_DESC"];
@@ -122,15 +119,13 @@ namespace SharedLibraryCore.Commands
                     Required = true
                 }
             };
-            _appConfig = appConfig;
         }
 
-        public override Task ExecuteAsync(GameEvent gameEvent)
+        public override Task ExecuteAsync(GameEvent E)
         {
-            var reason = gameEvent.Data.FindRuleForReason(_appConfig, gameEvent.Owner);
+            if (E.Target.Warn(E.Data, E.Origin).Failed)
-            if (gameEvent.Target.Warn(reason, gameEvent.Origin).Failed)
             {
-                gameEvent.Origin.Tell(_translationLookup["COMMANDS_WARN_FAIL"].FormatExt(gameEvent.Target.Name));
+                E.Origin.Tell(_translationLookup["COMMANDS_WARN_FAIL"].FormatExt(E.Target.Name));
             }
 
             return Task.CompletedTask;
@@ -175,9 +170,7 @@ namespace SharedLibraryCore.Commands
     /// </summary>
     public class KickCommand : Command
     {
-        private readonly ApplicationConfiguration _appConfig;
+        public KickCommand(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
-        
-        public KickCommand(ApplicationConfiguration appConfig, CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
         {
             Name = "kick";
             Description = _translationLookup["COMMANDS_KICK_DESC"];
@@ -197,22 +190,20 @@ namespace SharedLibraryCore.Commands
                     Required = true
                 }
             };
-            _appConfig = appConfig;
         }
 
-        public override async Task ExecuteAsync(GameEvent gameEvent)
+        public override async Task ExecuteAsync(GameEvent E)
         {
-            var reason = gameEvent.Data.FindRuleForReason(_appConfig, gameEvent.Owner);
+            switch ((await E.Target.Kick(E.Data, E.Origin).WaitAsync(Utilities.DefaultCommandTimeout, E.Owner.Manager.CancellationToken)).FailReason)
-            switch ((await gameEvent.Target.Kick(reason, gameEvent.Origin).WaitAsync(Utilities.DefaultCommandTimeout, gameEvent.Owner.Manager.CancellationToken)).FailReason)
             {
                 case GameEvent.EventFailReason.None:
-                    gameEvent.Origin.Tell(_translationLookup["COMMANDS_KICK_SUCCESS"].FormatExt(gameEvent.Target.Name));
+                    E.Origin.Tell(_translationLookup["COMMANDS_KICK_SUCCESS"].FormatExt(E.Target.Name));
                     break;
                 case GameEvent.EventFailReason.Exception:
-                    gameEvent.Origin.Tell(_translationLookup["SERVER_ERROR_COMMAND_INGAME"]);
+                    E.Origin.Tell(_translationLookup["SERVER_ERROR_COMMAND_INGAME"]);
                     break;
                 default:
-                    gameEvent.Origin.Tell(_translationLookup["COMMANDS_KICK_FAIL"].FormatExt(gameEvent.Target.Name));
+                    E.Origin.Tell(_translationLookup["COMMANDS_KICK_FAIL"].FormatExt(E.Target.Name));
                     break;
             }
         }
@@ -248,50 +239,12 @@ namespace SharedLibraryCore.Commands
         }
     }
 
-    /// <summary>
-    /// Prints out a message to all clients on all servers
-    /// </summary>
-    public class SayAllCommand : Command
-    {
-        public SayAllCommand(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
-        {
-            Name = "sayall";
-            Description = _translationLookup["COMMANDS_SAY_ALL_DESC"];
-            Alias = "sa";
-            Permission = Permission.Moderator;
-            RequiresTarget = false;
-            Arguments = new[]
-            {
-                new CommandArgument()
-                {
-                    Name = _translationLookup["COMMANDS_ARGS_MESSAGE"],
-                    Required = true
-                }
-            };
-        }
-
-        public override Task ExecuteAsync(GameEvent E)
-        {
-            string message = _translationLookup["COMMANDS_SAY_ALL_MESSAGE_FORMAT"].FormatExt(E.Origin.Name, E.Data);
-
-            foreach (var server in E.Owner.Manager.GetServers())
-            {
-                server.Broadcast(message, E.Origin);
-            }
-
-            E.Origin.Tell(_translationLookup["COMMANDS_SAY_SUCCESS"]);
-            return Task.CompletedTask;
-        }
-    }
-
     /// <summary>
     /// Temporarily bans a client
     /// </summary>
     public class TempBanCommand : Command
     {
-        private readonly ApplicationConfiguration _appConfig;
+        public TempBanCommand(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
-        
-        public TempBanCommand(ApplicationConfiguration appConfig, CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
         {
             Name = "tempban";
             Description = _translationLookup["COMMANDS_TEMPBAN_DESC"];
@@ -316,36 +269,35 @@ namespace SharedLibraryCore.Commands
                     Required = true
                 }
             };
-            _appConfig = appConfig;
         }
 
         private static readonly string TempBanRegex = @"([0-9]+\w+)\ (.+)";
 
-        public override async Task ExecuteAsync(GameEvent gameEvent)
+        public override async Task ExecuteAsync(GameEvent E)
         {
-            var match = Regex.Match(gameEvent.Data, TempBanRegex);
+            var match = Regex.Match(E.Data, TempBanRegex);
             if (match.Success)
             {
-                var tempbanReason = match.Groups[2].ToString().FindRuleForReason(_appConfig, gameEvent.Owner);
+                string tempbanReason = match.Groups[2].ToString();
                 var length = match.Groups[1].ToString().ParseTimespan();
 
-                if (length > gameEvent.Owner.Manager.GetApplicationSettings().Configuration().MaximumTempBanTime)
+                if (length > E.Owner.Manager.GetApplicationSettings().Configuration().MaximumTempBanTime)
                 {
-                    gameEvent.Origin.Tell(_translationLookup["COMMANDS_TEMPBAN_FAIL_TOOLONG"]);
+                    E.Origin.Tell(_translationLookup["COMMANDS_TEMPBAN_FAIL_TOOLONG"]);
                 }
 
                 else
                 {
-                    switch ((await gameEvent.Target.TempBan(tempbanReason, length, gameEvent.Origin).WaitAsync(Utilities.DefaultCommandTimeout, gameEvent.Owner.Manager.CancellationToken)).FailReason)
+                    switch ((await E.Target.TempBan(tempbanReason, length, E.Origin).WaitAsync(Utilities.DefaultCommandTimeout, E.Owner.Manager.CancellationToken)).FailReason)
                     {
                         case GameEvent.EventFailReason.None:
-                            gameEvent.Origin.Tell(_translationLookup["COMMANDS_TEMPBAN_SUCCESS"].FormatExt(gameEvent.Target, length.HumanizeForCurrentCulture()));
+                            E.Origin.Tell(_translationLookup["COMMANDS_TEMPBAN_SUCCESS"].FormatExt(E.Target, length.HumanizeForCurrentCulture()));
                             break;
                         case GameEvent.EventFailReason.Exception:
-                            gameEvent.Origin.Tell(_translationLookup["SERVER_ERROR_COMMAND_INGAME"]);
+                            E.Origin.Tell(_translationLookup["SERVER_ERROR_COMMAND_INGAME"]);
                             break;
                         default:
-                            gameEvent.Origin.Tell(_translationLookup["COMMANDS_TEMPBAN_FAIL"].FormatExt(gameEvent.Target.Name));
+                            E.Origin.Tell(_translationLookup["COMMANDS_TEMPBAN_FAIL"].FormatExt(E.Target.Name));
                             break;
                     }
                 }
@@ -358,8 +310,7 @@ namespace SharedLibraryCore.Commands
     /// </summary>
     public class BanCommand : Command
     {
-        private readonly ApplicationConfiguration _appConfig;
+        public BanCommand(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
-        public BanCommand(ApplicationConfiguration appConfig, CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
         {
             Name = "ban";
             Description = _translationLookup["COMMANDS_BAN_DESC"];
@@ -379,22 +330,20 @@ namespace SharedLibraryCore.Commands
                     Required = true
                 }
             };
-            _appConfig = appConfig;
         }
 
-        public override async Task ExecuteAsync(GameEvent gameEvent)
+        public override async Task ExecuteAsync(GameEvent E)
         {
-            var reason = gameEvent.Data.FindRuleForReason(_appConfig, gameEvent.Owner);
+            switch ((await E.Target.Ban(E.Data, E.Origin, false).WaitAsync(Utilities.DefaultCommandTimeout, E.Owner.Manager.CancellationToken)).FailReason)
-            switch ((await gameEvent.Target.Ban(reason, gameEvent.Origin, false).WaitAsync(Utilities.DefaultCommandTimeout, gameEvent.Owner.Manager.CancellationToken)).FailReason)
             {
                 case GameEvent.EventFailReason.None:
-                    gameEvent.Origin.Tell(_translationLookup["COMMANDS_BAN_SUCCESS"].FormatExt(gameEvent.Target.Name));
+                    E.Origin.Tell(_translationLookup["COMMANDS_BAN_SUCCESS"].FormatExt(E.Target.Name));
                     break;
                 case GameEvent.EventFailReason.Exception:
-                    gameEvent.Origin.Tell(_translationLookup["SERVER_ERROR_COMMAND_INGAME"]);
+                    E.Origin.Tell(_translationLookup["SERVER_ERROR_COMMAND_INGAME"]);
                     break;
                 default:
-                    gameEvent.Origin.Tell(_translationLookup["COMMANDS_BAN_FAIL"].FormatExt(gameEvent.Target.Name));
+                    E.Origin.Tell(_translationLookup["COMMANDS_BAN_FAIL"].FormatExt(E.Target.Name));
                     break;
             }
         }
@@ -647,11 +596,11 @@ namespace SharedLibraryCore.Commands
 
         public override async Task ExecuteAsync(GameEvent E)
         {
-            _ = !E.Origin.Masked ?
+            var _ = !E.Origin.Masked ?
                 E.Owner.Broadcast($"{_translationLookup["COMMANDS_MAPROTATE"]} [^5{E.Origin.Name}^7]", E.Origin) :
                 E.Owner.Broadcast(_translationLookup["COMMANDS_MAPROTATE"], E.Origin);
 
-            await Task.Delay(E.Owner.Manager.GetApplicationSettings().Configuration().MapChangeDelaySeconds * 1000);
+            await Task.Delay(5000);
             await E.Owner.ExecuteCommandAsync("map_rotate");
         }
     }
@@ -661,7 +610,7 @@ namespace SharedLibraryCore.Commands
     /// </summary>
     public class SetLevelCommand : Command
     {
-        public SetLevelCommand(CommandConfiguration config, ITranslationLookup translationLookup, ILogger<SetLevelCommand> logger) : base(config, translationLookup)
+        public SetLevelCommand(CommandConfiguration config, ITranslationLookup translationLookup, ILogger logger) : base(config, translationLookup)
         {
             Name = "setlevel";
             Description = _translationLookup["COMMANDS_SETLEVEL_DESC"];
@@ -726,12 +675,6 @@ namespace SharedLibraryCore.Commands
                 return;
             }
 
-            else if (gameEvent.Target.Level == Permission.Flagged)
-            {
-                gameEvent.Origin.Tell(_translationLookup["COMMANDS_SETLEVEL_FLAGGED"].FormatExt(gameEvent.Target.Name));
-                return;
-            }
-
             // stepped privilege is enabled, but the new level is too high
             else if (steppedPrivileges && !canPromoteSteppedPriv)
             {
@@ -748,26 +691,13 @@ namespace SharedLibraryCore.Commands
                     gameEvent.Owner.Manager.GetActiveClients()
                     .FirstOrDefault(c => c.ClientId == targetClient?.ClientId) ?? targetClient : targetClient;
 
-                logger.LogDebug("Beginning set level of client {origin} to {newPermission}", gameEvent.Origin.ToString(), newPerm);
+                logger.WriteInfo($"Beginning set level of client {gameEvent.Origin} to {newPerm}");
 
                 var result = await targetClient.SetLevel(newPerm, gameEvent.Origin).WaitAsync(Utilities.DefaultCommandTimeout, gameEvent.Owner.Manager.CancellationToken);
 
                 if (result.Failed)
                 {
-                    // user is the same level
+                    logger.WriteInfo($"Failed to set level of client {gameEvent.Origin}");
-                    if (result.FailReason == GameEvent.EventFailReason.Invalid)
-                    {
-                        gameEvent.Origin.Tell(_translationLookup["COMMANDS_SETLEVEL_INVALID"]
-                            .FormatExt(gameEvent.Target.Name, newPerm.ToString()));
-                        return;
-                    }
-                    
-                    using (LogContext.PushProperty("Server", gameEvent.Origin.CurrentServer?.ToString()))
-                    {
-                        logger.LogWarning("Failed to set level of client {origin} {reason}", 
-                            gameEvent.Origin.ToString(), 
-                            result.FailReason);
-                    }
                     gameEvent.Origin.Tell(_translationLookup["SERVER_ERROR_COMMAND_INGAME"]);
                     return;
                 }
@@ -830,9 +760,9 @@ namespace SharedLibraryCore.Commands
 
         public override Task ExecuteAsync(GameEvent E)
         {
-            var uptime = DateTime.Now - System.Diagnostics.Process.GetCurrentProcess().StartTime;
+            TimeSpan uptime = DateTime.Now - System.Diagnostics.Process.GetCurrentProcess().StartTime;
             var loc = _translationLookup;
-            E.Origin.Tell(loc["COMMANDS_UPTIME_TEXT"].FormatExt(uptime.HumanizeForCurrentCulture(4)));
+            E.Origin.Tell(loc["COMMANDS_UPTIME_TEXT"].FormatExt(uptime.Days, uptime.Hours, uptime.Minutes));
             return Task.CompletedTask;
         }
     }
@@ -898,18 +828,22 @@ namespace SharedLibraryCore.Commands
 
         public override async Task ExecuteAsync(GameEvent E)
         {
-            string newMap = E.Data.Trim();
+            string newMap = E.Data.Trim().ToLower();
-            int delay = E.Owner.Manager.GetApplicationSettings().Configuration().MapChangeDelaySeconds * 1000;
+            foreach (Map m in E.Owner.Maps)
+            {
+                if (m.Name.ToLower() == newMap || m.Alias.ToLower() == newMap)
+                {
+                    E.Owner.Broadcast(_translationLookup["COMMANDS_MAP_SUCCESS"].FormatExt(m.Alias));
+                    await Task.Delay((int)(Utilities.DefaultCommandTimeout.TotalMilliseconds / 2.0));
+                    await E.Owner.LoadMap(m.Name);
+                    return;
+                }
+            }
 
-            var foundMap = E.Owner.Maps.FirstOrDefault(_map => _map.Name.Equals(newMap, StringComparison.InvariantCultureIgnoreCase) ||
+            // todo: this can be moved into a single statement
-            _map.Alias.Equals(newMap, StringComparison.InvariantCultureIgnoreCase));
+            E.Owner.Broadcast(_translationLookup["COMMANDS_MAP_UKN"].FormatExt(newMap));
-
+            await Task.Delay(5000);
-            _ = foundMap == null ?
+            await E.Owner.LoadMap(newMap);
-                E.Owner.Broadcast(_translationLookup["COMMANDS_MAP_UKN"].FormatExt(newMap)) :
-                 E.Owner.Broadcast(_translationLookup["COMMANDS_MAP_SUCCESS"].FormatExt(foundMap.Alias));
-
-            await Task.Delay(delay);
-            await E.Owner.LoadMap(foundMap?.Name ?? newMap);
         }
     }
 
@@ -1450,10 +1384,7 @@ namespace SharedLibraryCore.Commands
     /// </summary>
     public class PruneAdminsCommand : Command
     {
-        private readonly IDatabaseContextFactory _contextFactory;
+        public PruneAdminsCommand(CommandConfiguration config, ITranslationLookup translationLookup) : base(config, translationLookup)
-        
-        public PruneAdminsCommand(CommandConfiguration config, ITranslationLookup translationLookup, 
-            IDatabaseContextFactory contextFactory) : base(config, translationLookup)
         {
             Name = "prune";
             Description = _translationLookup["COMMANDS_PRUNE_DESC"];
@@ -1495,15 +1426,16 @@ namespace SharedLibraryCore.Commands
             List<EFClient> inactiveUsers = null;
             // todo: make an event for this
             // update user roles
-            await using var context = _contextFactory.CreateContext();
+            using (var context = new DatabaseContext())
-            var lastActive = DateTime.UtcNow.AddDays(-inactiveDays);
+            {
-            inactiveUsers = await context.Clients
+                var lastActive = DateTime.UtcNow.AddDays(-inactiveDays);
-                .Where(c => c.Level > Permission.Flagged && c.Level <= Permission.Moderator)
+                inactiveUsers = await context.Clients
-                .Where(c => c.LastConnection < lastActive)
+                    .Where(c => c.Level > Permission.Flagged && c.Level <= Permission.Moderator)
-                .ToListAsync();
+                    .Where(c => c.LastConnection < lastActive)
-            inactiveUsers.ForEach(c => c.SetLevel(Permission.User, E.Origin));
+                    .ToListAsync();
-            await context.SaveChangesAsync();
+                inactiveUsers.ForEach(c => c.SetLevel(Permission.User, E.Origin));
-      
+                await context.SaveChangesAsync();
+            }
             E.Origin.Tell(_translationLookup["COMMANDS_PRUNE_SUCCESS"].FormatExt(inactiveUsers.Count));
         }
     }

SharedLibraryCore/Commands/RunAsCommand.cs

@@ -25,39 +25,40 @@ namespace SharedLibraryCore.Commands
             };
         }
 
-        public override async Task ExecuteAsync(GameEvent gameEvent)
+        public override async Task ExecuteAsync(GameEvent E)
         {
-            if (gameEvent.IsTargetingSelf())
+            if (E.IsTargetingSelf())
             {
-                gameEvent.Origin.Tell(_translationLookup["COMMANDS_RUN_AS_SELF"]);
+                E.Origin.Tell(_translationLookup["COMMANDS_RUN_AS_SELF"]);
                 return;
             }
 
-            if (!gameEvent.CanPerformActionOnTarget())
+            if (!E.CanPerformActionOnTarget())
             {
-                gameEvent.Origin.Tell(_translationLookup["COMMANDS_RUN_AS_FAIL_PERM"]);
+                E.Origin.Tell(_translationLookup["COMMANDS_RUN_AS_FAIL_PERM"]);
                 return;
             }
 
-            var cmd = $"{Utilities.CommandPrefix}{gameEvent.Data}";
+            string cmd = $"{Utilities.CommandPrefix}{E.Data}";
             var impersonatedCommandEvent = new GameEvent()
             {
                 Type = GameEvent.EventType.Command,
-                Origin = gameEvent.Target,
+                Origin = E.Target,
-                ImpersonationOrigin = gameEvent.Origin,
+                ImpersonationOrigin = E.Origin,
                 Message = cmd,
                 Data = cmd,
-                Owner = gameEvent.Owner,
+                Owner = E.Owner
-                CorrelationId = gameEvent.CorrelationId
             };
-            gameEvent.Owner.Manager.AddEvent(impersonatedCommandEvent);
+            E.Owner.Manager.AddEvent(impersonatedCommandEvent);
 
-            var result = await impersonatedCommandEvent.WaitAsync(Utilities.DefaultCommandTimeout, gameEvent.Owner.Manager.CancellationToken);
+            var result = await impersonatedCommandEvent.WaitAsync(Utilities.DefaultCommandTimeout, E.Owner.Manager.CancellationToken);
+            var response = E.Owner.CommandResult.Where(c => c.ClientId == E.Target.ClientId).ToList();
 
             // remove the added command response
-            foreach (var output in result.Output)
+            for (int i = 0; i < response.Count; i++)
             {
-                gameEvent.Origin.Tell(_translationLookup["COMMANDS_RUN_AS_SUCCESS"].FormatExt(output));
+                E.Origin.Tell(_translationLookup["COMMANDS_RUN_AS_SUCCESS"].FormatExt(response[i].Response));
+                E.Owner.CommandResult.Remove(response[i]);
             }
         }
     }

SharedLibraryCore/Configuration/ApplicationConfiguration.cs

@@ -1,7 +1,6 @@
 using SharedLibraryCore.Configuration.Attributes;
 using SharedLibraryCore.Interfaces;
 using System;
-using System.Collections.Generic;
 using System.ComponentModel.DataAnnotations;
 
 namespace SharedLibraryCore.Configuration
@@ -9,79 +8,61 @@ namespace SharedLibraryCore.Configuration
     public class ApplicationConfiguration : IBaseConfiguration
     {
         [LocalizedDisplayName("SETUP_ENABLE_WEBFRONT")]
-        [ConfigurationLinked("WebfrontBindUrl", "ManualWebfrontUrl", "WebfrontPrimaryColor", "WebfrontSecondaryColor",
+        [ConfigurationLinked("WebfrontBindUrl", "ManualWebfrontUrl", "WebfrontPrimaryColor", "WebfrontSecondaryColor", "WebfrontCustomBranding")]
-            "WebfrontCustomBranding")]
         public bool EnableWebFront { get; set; }
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_BIND_URL")]
         public string WebfrontBindUrl { get; set; }
-
         [ConfigurationOptional]
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_MANUAL_URL")]
         public string ManualWebfrontUrl { get; set; }
-
         [ConfigurationOptional]
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_PRIMARY_COLOR")]
         public string WebfrontPrimaryColor { get; set; }
-
         [ConfigurationOptional]
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_SECONDARY_COLOR")]
         public string WebfrontSecondaryColor { get; set; }
-
         [ConfigurationOptional]
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_CUSTOM_BRANDING")]
         public string WebfrontCustomBranding { get; set; }
 
         [LocalizedDisplayName("SETUP_ENABLE_MULTIOWN")]
         public bool EnableMultipleOwners { get; set; }
-
         [LocalizedDisplayName("SETUP_ENABLE_STEPPEDPRIV")]
         public bool EnableSteppedHierarchy { get; set; }
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_USE_LOCAL_TRANSLATIONS")]
         public bool UseLocalTranslations { get; set; }
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_IGNORE_BOTS")]
         public bool IgnoreBots { get; set; }
 
         [ConfigurationLinked("CustomSayName")]
         [LocalizedDisplayName("SETUP_ENABLE_CUSTOMSAY")]
         public bool EnableCustomSayName { get; set; }
-
         [LocalizedDisplayName("SETUP_SAY_NAME")]
         public string CustomSayName { get; set; }
 
         [LocalizedDisplayName("SETUP_DISPLAY_SOCIAL")]
         [ConfigurationLinked("SocialLinkAddress", "SocialLinkTitle")]
         public bool EnableSocialLink { get; set; }
-
         [LocalizedDisplayName("SETUP_SOCIAL_LINK")]
         public string SocialLinkAddress { get; set; }
-
         [LocalizedDisplayName("SETUP_SOCIAL_TITLE")]
         public string SocialLinkTitle { get; set; }
 
-        [LocalizedDisplayName("SETUP_CONTACT_URI")]
-        public string ContactUri { get; set; }
-
         [LocalizedDisplayName("SETUP_USE_CUSTOMENCODING")]
         [ConfigurationLinked("CustomParserEncoding")]
         public bool EnableCustomParserEncoding { get; set; }
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_ENCODING")]
         public string CustomParserEncoding { get; set; }
 
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_ENABLE_WHITELIST")]
         [ConfigurationLinked("WebfrontConnectionWhitelist")]
         public bool EnableWebfrontConnectionWhitelist { get; set; }
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_WHITELIST_LIST")]
         public string[] WebfrontConnectionWhitelist { get; set; } = new string[0];
 
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_CUSTOM_LOCALE")]
         [ConfigurationLinked("CustomLocale")]
         public bool EnableCustomLocale { get; set; }
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_CUSTOM_LOCALE")]
         public string CustomLocale { get; set; }
 
@@ -93,65 +74,38 @@ namespace SharedLibraryCore.Configuration
 
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_DB_PROVIDER")]
         public string DatabaseProvider { get; set; } = "sqlite";
-
         [ConfigurationOptional]
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_CONNECTION_STRING")]
         public string ConnectionString { get; set; }
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_RCON_POLLRATE")]
         public int RConPollRate { get; set; } = 5000;
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_MAX_TB")]
         public TimeSpan MaximumTempBanTime { get; set; } = new TimeSpan(24 * 30, 0, 0);
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_ENABLE_COLOR_CODES")]
         public bool EnableColorCodes { get; set; }
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_AUTOMESSAGE_PERIOD")]
         public int AutoMessagePeriod { get; set; }
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_AUTOMESSAGES")]
         public string[] AutoMessages { get; set; } = new string[0];
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_GLOBAL_RULES")]
         public string[] GlobalRules { get; set; } = new string[0];
-
         [LocalizedDisplayName("WEBFRONT_CONFIGURATION_DISALLOWED_NAMES")]
         public string[] DisallowedClientNames { get; set; } = new string[0];
-
-        [LocalizedDisplayName("WEBFRONT_CONFIGURATION_MAP_CHANGE_DELAY")]
-        public int MapChangeDelaySeconds { get; set; } = 5;
-
-        [LocalizedDisplayName("WEBFRONT_CONFIGURATION_BAN_DURATIONS")]
-        public TimeSpan[] BanDurations { get; set; } = {
-            TimeSpan.FromHours(1),
-            TimeSpan.FromHours(6),
-            TimeSpan.FromDays(1),
-            TimeSpan.FromDays(2),
-            TimeSpan.FromDays(7),
-            TimeSpan.FromDays(30)
-        };
-
-        [LocalizedDisplayName("WEBFRONT_CONFIGURATION_PRESET_BAN_REASONS")]
-        public Dictionary<string, string> PresetPenaltyReasons { get; set; } = new Dictionary<string, string>
-            {{"afk", "Away from keyboard"}, {"ci", "Connection interrupted. Reconnect"}};
-        [LocalizedDisplayName(("WEBFRONT_CONFIGURATION_ENABLE_PRIVILEGED_USER_PRIVACY"))]
-        public bool EnablePrivilegedUserPrivacy { get; set; }
         [UIHint("ServerConfiguration")]
         public ServerConfiguration[] Servers { get; set; }
 
-        [ConfigurationIgnore] public string Id { get; set; }
-        [ConfigurationIgnore] public string SubscriptionId { get; set; }
-        [ConfigurationIgnore] public MapConfiguration[] Maps { get; set; }
-        [ConfigurationIgnore] public QuickMessageConfiguration[] QuickMessages { get; set; }
-
         [ConfigurationIgnore]
-        public string WebfrontUrl => string.IsNullOrEmpty(ManualWebfrontUrl)
+        public string Id { get; set; }
-            ? WebfrontBindUrl?.Replace("0.0.0.0", "127.0.0.1")
+        [ConfigurationIgnore]
-            : ManualWebfrontUrl;
+        public MapConfiguration[] Maps { get; set; }
-
+        [ConfigurationIgnore]
-        [ConfigurationIgnore] public bool IgnoreServerConnectionLost { get; set; }
+        public QuickMessageConfiguration[] QuickMessages { get; set; }
-        [ConfigurationIgnore] public Uri MasterUrl { get; set; } = new Uri("http://api.raidmax.org:5000");
+        [ConfigurationIgnore]
+        public string WebfrontUrl => string.IsNullOrEmpty(ManualWebfrontUrl) ? WebfrontBindUrl?.Replace("0.0.0.0", "127.0.0.1") : ManualWebfrontUrl;
+        [ConfigurationIgnore]
+        public bool IgnoreServerConnectionLost { get; set; }
+        [ConfigurationIgnore]
+        public Uri MasterUrl { get; set; } = new Uri("http://api.raidmax.org:5000");
 
         public IBaseConfiguration Generate()
         {

SharedLibraryCore/Database/ContextSeed.cs

@@ -2,30 +2,31 @@
 using SharedLibraryCore.Database.Models;
 using System;
 using System.Linq;
-using System.Threading;
 using System.Threading.Tasks;
-using SharedLibraryCore.Interfaces;
 using static SharedLibraryCore.Database.Models.EFClient;
 
 namespace SharedLibraryCore.Database
 {
-    public static class ContextSeed
+    public class ContextSeed
     {
-        public static async Task Seed(IDatabaseContextFactory contextFactory, CancellationToken token)
+        private DatabaseContext context;
-        {
-            await using var context = contextFactory.CreateContext();
-            var strategy = context.Database.CreateExecutionStrategy();
-            await strategy.ExecuteAsync(async () =>
-            {
-                await context.Database.MigrateAsync(token);
-            });
 
-            if (!await context.AliasLinks.AnyAsync(token))
+        public ContextSeed(DatabaseContext ctx)
+        {
+            context = ctx;
+        }
+
+        public async Task Seed()
+        {
+            context.Database.Migrate();
+
+            if (context.AliasLinks.Count() == 0)
             {
                 var link = new EFAliasLink();
 
                 context.Clients.Add(new EFClient()
                 {
+                    ClientId = 1,
                     Active = false,
                     Connections = 0,
                     FirstConnection = DateTime.UtcNow,
@@ -43,7 +44,7 @@ namespace SharedLibraryCore.Database
                     },
                 });
 
-                await context.SaveChangesAsync(token);
+                await context.SaveChangesAsync();
             }
         }
     }

SharedLibraryCore/Database/DatabaseContext.cs

@@ -15,7 +15,7 @@ using System.Threading.Tasks;
 
 namespace SharedLibraryCore.Database
 {
-    public abstract class DatabaseContext : DbContext
+    public class DatabaseContext : DbContext
     {
         public DbSet<EFClient> Clients { get; set; }
         public DbSet<EFAlias> Aliases { get; set; }
@@ -24,6 +24,88 @@ namespace SharedLibraryCore.Database
         public DbSet<EFMeta> EFMeta { get; set; }
         public DbSet<EFChangeHistory> EFChangeHistory { get; set; }
 
+        static string _ConnectionString;
+        static string _provider;
+        private static readonly ILoggerFactory _loggerFactory = LoggerFactory.Create(builder =>
+        {
+            builder.AddConsole()
+            .AddDebug()
+            .AddFilter((category, level) => true);
+        });
+
+        public DatabaseContext(DbContextOptions<DatabaseContext> opt) : base(opt)
+        {
+        }
+
+        public DatabaseContext()
+        {
+        }
+
+        public override void Dispose()
+        {
+        }
+
+        public DatabaseContext(bool disableTracking) : this()
+        {
+            if (disableTracking)
+            {
+                this.ChangeTracker.AutoDetectChangesEnabled = false;
+                this.ChangeTracker.LazyLoadingEnabled = false;
+                this.ChangeTracker.QueryTrackingBehavior = QueryTrackingBehavior.NoTracking;
+            }
+
+            else
+            {
+                this.ChangeTracker.AutoDetectChangesEnabled = true;
+                this.ChangeTracker.LazyLoadingEnabled = true;
+                this.ChangeTracker.QueryTrackingBehavior = QueryTrackingBehavior.TrackAll;
+            }
+        }
+
+        public DatabaseContext(string connStr, string provider) : this()
+        {
+            _ConnectionString = connStr;
+            _provider = provider;
+        }
+
+        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
+        {
+            //optionsBuilder.UseLoggerFactory(_loggerFactory)
+            //  .EnableSensitiveDataLogging();
+
+            if (string.IsNullOrEmpty(_ConnectionString))
+            {
+                string currentPath = Utilities.OperatingDirectory;
+                // allows the application to find the database file
+                currentPath = !RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ?
+                    $"{Path.DirectorySeparatorChar}{currentPath}" :
+                    currentPath;
+
+                var connectionStringBuilder = new SqliteConnectionStringBuilder { DataSource = Path.Join(currentPath, "Database", "Database.db") };
+                var connectionString = connectionStringBuilder.ToString();
+                var connection = new SqliteConnection(connectionString);
+
+                if (!optionsBuilder.IsConfigured)
+                {
+                    optionsBuilder.UseSqlite(connection);
+                }
+            }
+
+            else
+            {
+                switch (_provider)
+                {
+                    default:
+                    case "mysql":
+                        optionsBuilder.UseMySql(_ConnectionString, _options => _options.EnableRetryOnFailure());
+                        break;
+                    case "postgresql":
+                        optionsBuilder.UseNpgsql(_ConnectionString, _options => _options.EnableRetryOnFailure());
+                        break;
+                }
+            }
+        }
+
         private void SetAuditColumns()
         {
             return;
@@ -47,24 +129,6 @@ namespace SharedLibraryCore.Database
             }
         }
 
-        public DatabaseContext()
-        {
-            if (!Utilities.IsMigration)
-            {
-                throw new InvalidOperationException();
-            }
-        }
-
-        public DatabaseContext(DbContextOptions<DatabaseContext> options) : base(options)
-        {
-            
-        }
-
-        protected DatabaseContext(DbContextOptions options) : base(options)
-        {
-            
-        }
-
         public override Task<int> SaveChangesAsync(bool acceptAllChangesOnSuccess, CancellationToken cancellationToken = default)
         {
             SetAuditColumns();
@@ -141,18 +205,7 @@ namespace SharedLibraryCore.Database
                 pluginDir = Path.Join(Utilities.OperatingDirectory, "..", "..", "..", "..", "BUILD", "Plugins");
             }
 
-            IEnumerable<string> directoryFiles = Enumerable.Empty<string>();
+            IEnumerable<string> directoryFiles = Directory.GetFiles(pluginDir).Where(f => f.EndsWith(".dll"));
-
-            try
-            {
-                directoryFiles = Directory.GetFiles(pluginDir).Where(f => f.EndsWith(".dll"));
-            }
-
-            catch (DirectoryNotFoundException)
-            {
-                // this is just an ugly thing for unit testing
-                directoryFiles = Directory.GetFiles(@"X:\IW4MAdmin\Tests\ApplicationTests\bin\Debug\netcoreapp3.1").Where(f => f.EndsWith("dll"));
-            }
 
             foreach (string dllPath in directoryFiles)
             {

SharedLibraryCore/Database/MigrationContext/MySqlDatabaseContext.cs

@@ -1,31 +0,0 @@
-using System;
-using Microsoft.EntityFrameworkCore;
-
-namespace SharedLibraryCore.Database.MigrationContext
-{
-    public class MySqlDatabaseContext : DatabaseContext
-    {
-        public MySqlDatabaseContext()
-        {
-            if (!Utilities.IsMigration)
-            {
-                throw new InvalidOperationException();
-            }
-        }
-
-        public MySqlDatabaseContext(DbContextOptions options) : base(options)
-        {
-            
-        }
-
-        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
-        {
-            if (Utilities.IsMigration)
-            {
-                optionsBuilder.UseMySql("Server=127.0.0.1;Database=IW4MAdmin_Migration;Uid=root;Pwd=password;")
-                    .EnableDetailedErrors(true)
-                    .EnableSensitiveDataLogging(true);
-            }
-        }
-    }
-}

SharedLibraryCore/Database/MigrationContext/PostgresqlDatabaseContext.cs

@@ -1,33 +0,0 @@
-using System;
-using Microsoft.EntityFrameworkCore;
-
-namespace SharedLibraryCore.Database.MigrationContext
-{
-    public class PostgresqlDatabaseContext : DatabaseContext
-    {
-        public PostgresqlDatabaseContext()
-        {
-            if (!Utilities.IsMigration)
-            {
-                throw new InvalidOperationException();
-            }
-        }
-        
-        public PostgresqlDatabaseContext(DbContextOptions options) : base(options)
-        {
-            
-        }
-        
-        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
-        {
-            if (Utilities.IsMigration)
-            {
-                optionsBuilder.UseNpgsql(
-                        "Host=127.0.0.1;Database=IW4MAdmin_Migration;Username=postgres;Password=password;")
-                    .EnableDetailedErrors(true)
-                    .EnableSensitiveDataLogging(true);
-            }
-        }
-        
-    }
-}

SharedLibraryCore/Database/MigrationContext/SqliteDatabaseContext.cs

@@ -1,31 +0,0 @@
-using System;
-using Microsoft.EntityFrameworkCore;
-
-namespace SharedLibraryCore.Database.MigrationContext
-{
-    public class SqliteDatabaseContext : DatabaseContext
-    {
-        public SqliteDatabaseContext()
-        {
-            if (!Utilities.IsMigration)
-            {
-                throw new InvalidOperationException();
-            }
-        }
-        
-        public SqliteDatabaseContext(DbContextOptions options) : base(options)
-        {
-            
-        }
-        
-        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
-        {
-            if (Utilities.IsMigration)
-            {
-                optionsBuilder.UseSqlite("Data Source=IW4MAdmin_Migration.db")
-                    .EnableDetailedErrors(true)
-                    .EnableSensitiveDataLogging(true);
-            }
-        }
-    }
-}

SharedLibraryCore/Dtos/Meta/Responses/MessageResponse.cs

@@ -28,11 +28,6 @@ namespace SharedLibraryCore.Dtos.Meta.Responses
         /// </summary>
         public bool IsQuickMessage { get; set; }
 
-        /// <summary>
-        /// indicates if the message was sent ingame
-        /// </summary>
-        public bool SentIngame { get; set; }
-
         public string HiddenMessage => string.Concat(Enumerable.Repeat('●', Message.Length));
     }
 }

SharedLibraryCore/Events/GameEvent.cs

@@ -1,11 +1,8 @@
 using SharedLibraryCore.Database.Models;
 using SharedLibraryCore.Events;
 using System;
-using System.Collections.Generic;
 using System.Threading;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using Serilog.Context;
 
 namespace SharedLibraryCore
 {
@@ -236,7 +233,7 @@ namespace SharedLibraryCore
         /// <summary>
         /// Specifies the game time offset as printed in the log
         /// </summary>
-        public long? GameTime { get; set; }
+        public int? GameTime { get; set; }
         public EFClient Origin;
         public EFClient Target;
         public EFClient ImpersonationOrigin { get; set; }
@@ -248,8 +245,6 @@ namespace SharedLibraryCore
         public long Id { get; private set; }
         public EventFailReason FailReason { get; set; }
         public bool Failed => FailReason != EventFailReason.None;
-        public Guid CorrelationId { get; set; } = Guid.NewGuid();
-        public List<string> Output { get; set; } = new List<string>();
 
         /// <summary>
         /// Indicates if the event should block until it is complete
@@ -259,6 +254,9 @@ namespace SharedLibraryCore
         public void Complete()
         {
             _eventFinishedWaiter.Set();
+#if DEBUG
+            Owner?.Logger.WriteDebug($"Completed internal for event {Id}");
+#endif
         }
 
         /// <summary>
@@ -268,7 +266,11 @@ namespace SharedLibraryCore
         public async Task<GameEvent> WaitAsync(TimeSpan timeSpan, CancellationToken token)
         {
             bool processed = false;
-            Utilities.DefaultLogger.LogDebug("Begin wait for event {Id}", Id);
+
+#if DEBUG
+            Owner?.Logger.WriteDebug($"Begin wait for event {Id}");
+#endif
+
             try
             {
                 processed = await Task.Run(() => _eventFinishedWaiter.WaitOne(timeSpan), token);
@@ -281,12 +283,14 @@ namespace SharedLibraryCore
 
             if (!processed)
             {
-                using(LogContext.PushProperty("Server", Owner?.ToString()))
+                Owner?.Logger.WriteError("Waiting for event to complete timed out");
-                {
+                Owner?.Logger.WriteDebug($"{Id}, {Type}, {Data}, {Extra}, {FailReason.ToString()}, {Message}, {Origin}, {Target}");
-                    Utilities.DefaultLogger.LogError("Waiting for event to complete timed out {@eventData}", new { Event = this, Message, Origin = Origin?.ToString(), Target = Target?.ToString()});
+#if DEBUG
-                }
+                //throw new Exception();
+#endif
             }
 
+
             // this lets us know if the the action timed out
             FailReason = FailReason == EventFailReason.None && !processed ? EventFailReason.Timeout : FailReason;
             return this;

SharedLibraryCore/Exceptions/RConException.cs

@@ -1,11 +0,0 @@
-using System;
-
-namespace SharedLibraryCore.Exceptions
-{
-    public class RConException : Exception
-    {
-        public RConException(string message) : base(message)
-        {
-        }
-    }
-}

SharedLibraryCore/Interfaces/IClientNoticeMessageFormatter.cs

@@ -1,16 +0,0 @@
-using SharedLibraryCore.Database.Models;
-
-namespace SharedLibraryCore.Interfaces
-{
-    public interface IClientNoticeMessageFormatter
-    {
-        /// <summary>
-        /// builds a game formatted notice message
-        /// </summary>
-        /// <param name="currentPenalty">current penalty the message is for</param>
-        /// <param name="originalPenalty">previous penalty the current penalty relates to</param>
-        /// <param name="config">RCon parser config</param>
-        /// <returns></returns>
-        string BuildFormattedMessage(IRConParserConfiguration config, EFPenalty currentPenalty, EFPenalty originalPenalty = null);
-    }
-}

SharedLibraryCore/Interfaces/IGameServer.cs

@@ -1,18 +0,0 @@
-using System.Threading.Tasks;
-using SharedLibraryCore.Database.Models;
-
-namespace SharedLibraryCore.Interfaces
-{
-    public interface IGameServer
-    {
-        /// <summary>
-        /// kicks target on behalf of origin for given reason
-        /// </summary>
-        /// <param name="reason">reason client is being kicked</param>
-        /// <param name="target">client to kick</param>
-        /// <param name="origin">source of kick action</param>
-        /// <param name="previousPenalty">previous penalty the kick is occuring for (if applicable)</param>
-        /// <returns></returns>
-        public Task Kick(string reason, EFClient target, EFClient origin, EFPenalty previousPenalty = null);
-    }
-}

SharedLibraryCore/Interfaces/ILogger.cs

@@ -1,8 +1,11 @@
 using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
 
 namespace SharedLibraryCore.Interfaces
 {
-    [Obsolete]
     public interface ILogger
     {
         void WriteVerbose(string msg);

SharedLibraryCore/Interfaces/IManager.cs

@@ -6,8 +6,6 @@ using SharedLibraryCore.Database.Models;
 using System.Threading;
 using System.Collections;
 using System;
-using System.Collections.Concurrent;
-using Microsoft.Extensions.Logging;
 
 namespace SharedLibraryCore.Interfaces
 {
@@ -17,7 +15,6 @@ namespace SharedLibraryCore.Interfaces
         Task Start();
         void Stop();
         void Restart();
-        [Obsolete]
         ILogger GetLogger(long serverId);
         IList<Server> GetServers();
         IList<IManagerCommand> GetCommands();
@@ -25,6 +22,7 @@ namespace SharedLibraryCore.Interfaces
         IList<EFClient> GetActiveClients();
         IConfigurationHandler<ApplicationConfiguration> GetApplicationSettings();
         ClientService GetClientService();
+        AliasService GetAliasService();
         PenaltyService GetPenaltyService();
         /// <summary>
         /// enumerates the registered plugin instances
@@ -87,6 +85,5 @@ namespace SharedLibraryCore.Interfaces
         /// event executed when event has finished executing 
         /// </summary>
         event EventHandler<GameEvent> OnGameEventExecuted;
-        ConcurrentDictionary<long, GameEvent> ProcessingEvents { get; }
     }
 }