Make IMessenger scoped (which essentially becomes singleton per user).

immense · Jun 22, 2023 · 3bb5098 · 3bb5098
1 parent f5f11ca
commit 3bb5098
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/Server/Program.cs b/Server/Program.cs
@@ -219,7 +219,7 @@
 services.AddSingleton<IOtpProvider, OtpProvider>();
 services.AddSingleton<IEmbeddedServerDataSearcher, EmbeddedServerDataSearcher>();
 services.AddSingleton<ILogsManager, LogsManager>();
-services.AddSingleton(WeakReferenceMessenger.Default);
+services.AddScoped<IMessenger>((services) => new WeakReferenceMessenger());
 
 services.AddRemoteControlServer(config =>
 {

diff --git a/Server/Services/ScriptScheduler.cs b/Server/Services/ScriptScheduler.cs
@@ -61,9 +61,7 @@ public async Task DispatchScriptRuns()
         {
             using var scope = _serviceProvider.CreateScope();
             var scriptScheduleDispatcher = scope.ServiceProvider.GetRequiredService<IScriptScheduleDispatcher>();
-            var dataService = scope.ServiceProvider.GetRequiredService<IDataService>();
             var logger = scope.ServiceProvider.GetRequiredService<ILogger<ScriptScheduler>>();
-            var circuitConnection = scope.ServiceProvider.GetRequiredService<ICircuitConnection>();
 
             try
             {