some fixes for server, perhaps

MareSynchronosServer/MareSynchronosServer/Startup.cs

@@ -18,6 +18,8 @@ using MareSynchronosShared.Utils;
 using MareSynchronosServer.Identity;
 using MareSynchronosShared.Services;
 using Prometheus;
+using Microsoft.Extensions.Options;
+using Grpc.Net.ClientFactory;
 
 namespace MareSynchronosServer;
 
@@ -207,7 +209,7 @@ public class Startup
                 };
             });
 
-            services.AddGrpcClient<ConfigurationService.ConfigurationServiceClient>(c =>
+            services.AddGrpcClient<ConfigurationService.ConfigurationServiceClient>("MainServer", c =>
             {
                 c.Address = new Uri(mareConfig.GetValue<string>(nameof(ServerConfiguration.MainServerGrpcAddress)));
             }).ConfigureChannel(c =>
@@ -221,8 +223,16 @@ public class Startup
 
             services.AddSingleton<IClientIdentificationService, GrpcClientIdentificationService>();
             services.AddHostedService(p => p.GetService<IClientIdentificationService>());
-            services.AddSingleton<IConfigurationService<ServerConfiguration>, MareConfigurationServiceClient<ServerConfiguration>>();
+            services.AddSingleton<IConfigurationService<ServerConfiguration>>(c => new MareConfigurationServiceClient<ServerConfiguration>(
-            services.AddSingleton<IConfigurationService<MareConfigurationAuthBase>, MareConfigurationServiceClient<MareConfigurationAuthBase>>();
+                c.GetService<ILogger<MareConfigurationServiceClient<ServerConfiguration>>>(),
+                c.GetService<IOptions<ServerConfiguration>>(),
+                c.GetService<GrpcClientFactory>(),
+                "MainServer"));
+            services.AddSingleton<IConfigurationService<MareConfigurationAuthBase>>(c => new MareConfigurationServiceClient<MareConfigurationAuthBase>(
+                c.GetService<ILogger<MareConfigurationServiceClient<MareConfigurationAuthBase>>>(),
+                c.GetService<IOptions<MareConfigurationAuthBase>>(),
+                c.GetService<GrpcClientFactory>(),
+                "MainServer"));
         }
         else
         {

MareSynchronosServer/MareSynchronosServices/Startup.cs

@@ -7,6 +7,9 @@ using MareSynchronosShared.Utils;
 using Grpc.Net.Client.Configuration;
 using MareSynchronosShared.Protos;
 using MareSynchronosShared.Services;
+using Grpc.Net.ClientFactory;
+using Microsoft.Extensions.Options;
+using Microsoft.Extensions.DependencyInjection;
 
 namespace MareSynchronosServices;
 
@@ -53,7 +56,7 @@ public class Startup
             };
         });
 
-        services.AddGrpcClient<ConfigurationService.ConfigurationServiceClient>(c =>
+        services.AddGrpcClient<ConfigurationService.ConfigurationServiceClient>("MainServer", c =>
         {
             c.Address = new Uri(mareConfig.GetValue<string>(nameof(ServicesConfiguration.MainServerGrpcAddress)));
         }).ConfigureChannel(c =>
@@ -72,8 +75,16 @@ public class Startup
         services.AddSingleton<DiscordBotServices>();
         services.AddHostedService<DiscordBot>();
         services.AddSingleton<IConfigurationService<ServicesConfiguration>, MareConfigurationServiceServer<ServicesConfiguration>>();
-        services.AddSingleton<IConfigurationService<ServerConfiguration>, MareConfigurationServiceClient<ServerConfiguration>>();
+        services.AddSingleton<IConfigurationService<ServerConfiguration>>(c => new MareConfigurationServiceClient<ServerConfiguration>(
-        services.AddSingleton<IConfigurationService<MareConfigurationAuthBase>, MareConfigurationServiceClient<MareConfigurationAuthBase>>();
+                c.GetService<ILogger<MareConfigurationServiceClient<ServerConfiguration>>>(),
+                c.GetService<IOptions<ServerConfiguration>>(),
+                c.GetService<GrpcClientFactory>(),
+                "MainServer"));
+        services.AddSingleton<IConfigurationService<MareConfigurationAuthBase>>(c => new MareConfigurationServiceClient<MareConfigurationAuthBase>(
+            c.GetService<ILogger<MareConfigurationServiceClient<MareConfigurationAuthBase>>>(),
+            c.GetService<IOptions<MareConfigurationAuthBase>>(),
+            c.GetService<GrpcClientFactory>(),
+            "MainServer"));
     }
 
     public void Configure(IApplicationBuilder app, IWebHostEnvironment env)

MareSynchronosServer/MareSynchronosShared/Services/MareConfigurationServiceClient.cs

@@ -18,20 +18,16 @@ public class MareConfigurationServiceClient<T> : IConfigurationService<T> where
     private readonly T _config;
     private readonly ConcurrentDictionary<string, RemoteCachedEntry> _cachedRemoteProperties = new(StringComparer.Ordinal);
     private readonly ILogger<MareConfigurationServiceClient<T>> _logger;
-    private readonly ConfigurationServiceClient _configurationServiceClient;
+    private readonly GrpcClientFactory _grpcClientFactory;
-
+    private readonly string _grpcClientName;
-    public MareConfigurationServiceClient(ILogger<MareConfigurationServiceClient<T>> logger, IOptions<T> config, ConfigurationServiceClient configurationServiceClient)
+    private static SemaphoreSlim _readLock = new(1);
-    {
-        _config = config.Value;
-        _logger = logger;
-        _configurationServiceClient = configurationServiceClient;
-    }
 
     public MareConfigurationServiceClient(ILogger<MareConfigurationServiceClient<T>> logger, IOptions<T> config, GrpcClientFactory grpcClientFactory, string grpcClientName)
     {
         _config = config.Value;
         _logger = logger;
-        _configurationServiceClient = grpcClientFactory.CreateClient<ConfigurationServiceClient>(grpcClientName);
+        _grpcClientFactory = grpcClientFactory;
+        _grpcClientName = grpcClientName;
     }
 
     public bool IsMain => false;
@@ -44,14 +40,13 @@ public class MareConfigurationServiceClient<T> : IConfigurationService<T> where
         bool isRemote = prop.GetCustomAttributes(typeof(RemoteConfigurationAttribute), inherit: true).Any();
         if (isRemote)
         {
-            bool isCurrent = false;
+            _readLock.Wait();
-            if (_cachedRemoteProperties.TryGetValue(key, out var existingEntry) && existingEntry.Inserted > DateTime.Now - TimeSpan.FromMinutes(30))
+            if (_cachedRemoteProperties.TryGetValue(key, out var existingEntry) && existingEntry.Inserted > DateTime.Now - TimeSpan.FromMinutes(60))
             {
-                isCurrent = true;
+                _readLock.Release();
+                return (T1)_cachedRemoteProperties[key].Value;
             }
 
-            if (!isCurrent)
-            {
             try
             {
                 var result = GetValueFromGrpc(key, defaultValue, prop.PropertyType);
@@ -72,6 +67,9 @@ public class MareConfigurationServiceClient<T> : IConfigurationService<T> where
                     return defaultValue;
                 }
             }
+            finally
+            {
+                _readLock.Release();
             }
         }
 
@@ -85,12 +83,14 @@ public class MareConfigurationServiceClient<T> : IConfigurationService<T> where
         try
         {
             _logger.LogInformation("Getting {key} from Grpc", key);
-            var response = _configurationServiceClient.GetConfigurationEntry(new KeyMessage { Key = key, Default = Convert.ToString(defaultValue, CultureInfo.InvariantCulture) });
+            var configClient = _grpcClientFactory.CreateClient<ConfigurationServiceClient>(_grpcClientName);
+            var response = configClient.GetConfigurationEntry(new KeyMessage { Key = key, Default = Convert.ToString(defaultValue, CultureInfo.InvariantCulture) });
             _logger.LogInformation("Grpc Response for {key} = {value}", key, response.Value);
             return new RemoteCachedEntry(JsonSerializer.Deserialize(response.Value, t), DateTime.Now);
         }
-        catch
+        catch (Exception ex)
         {
+            _logger.LogWarning(ex, "Failure Getting Cached Entry");
             return null;
         }
     }
@@ -103,23 +103,38 @@ public class MareConfigurationServiceClient<T> : IConfigurationService<T> where
         bool isRemote = prop.GetCustomAttributes(typeof(RemoteConfigurationAttribute), inherit: true).Any();
         if (isRemote)
         {
-            bool isCurrent = false;
+            _readLock.Wait();
-            if (_cachedRemoteProperties.TryGetValue(key, out var existingEntry) && existingEntry.Inserted > DateTime.Now - TimeSpan.FromMinutes(30))
+            if (_cachedRemoteProperties.TryGetValue(key, out var existingEntry) && existingEntry.Inserted > DateTime.Now - TimeSpan.FromMinutes(60))
             {
-                isCurrent = true;
+                _readLock.Release();
+                return (T1)_cachedRemoteProperties[key].Value;
             }
 
-            if (!isCurrent)
+            try
             {
                 var result = GetValueFromGrpc(key, null, prop.PropertyType);
                 if (result == null) throw new KeyNotFoundException(key);
                 _cachedRemoteProperties[key] = result;
-            }
-
-            if (!_cachedRemoteProperties.ContainsKey(key)) throw new KeyNotFoundException(key);
-
                 return (T1)_cachedRemoteProperties[key].Value;
             }
+            catch (Exception ex)
+            {
+                if (existingEntry != null)
+                {
+                    _logger.LogWarning(ex, "Could not get value for {key} from Grpc, returning existing", key);
+                    return (T1)existingEntry.Value;
+                }
+                else
+                {
+                    _logger.LogWarning(ex, "Could not get value for {key} from Grpc, throwing exception", key);
+                    throw new KeyNotFoundException(key);
+                }
+            }
+            finally
+            {
+                _readLock.Release();
+            }
+        }
 
         var value = prop.GetValue(_config);
         return (T1)value;