Fix review

atidev · Nov 14, 2023 · 046efc5 · 046efc5
1 parent f959cf0
commit 046efc5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/ATI.Services.Consul/ConsulAdapter.cs b/ATI.Services.Consul/ConsulAdapter.cs
@@ -17,16 +17,17 @@ internal class ConsulAdapter: IDisposable
     private readonly MetricsFactory _metricsFactory = MetricsFactory.CreateHttpClientMetricsFactory(nameof(ConsulAdapter), "consul");
 
     /// <summary>
-    /// Возвращает список живых сервисов
+    /// Возвращает список живых инстансов сервиса
     /// </summary>
     /// <returns></returns>
-    public async Task<OperationResult<List<ServiceEntry>>> GetPassingServiceInstancesAsync(string serviceName,
-                                                                          string environment,
-                                                                          bool passingOnly = true)
+    public async Task<OperationResult<List<ServiceEntry>>> GetPassingServiceInstancesAsync(
+        string serviceName,
+        string environment,
+        bool passingOnly = true)
     {
         try
         {
-            using (_metricsFactory.CreateMetricsTimer(nameof(GetPassingServiceInstancesAsync)))
+            using (_metricsFactory.CreateMetricsTimer("/health/service/:service"))
             {
                 var fromConsul = await _consulClient.Health.Service(serviceName, environment, passingOnly);
                 if (fromConsul.StatusCode == HttpStatusCode.OK)

diff --git a/ATI.Services.Consul/ConsulServiceAddressCache.cs b/ATI.Services.Consul/ConsulServiceAddressCache.cs
@@ -60,5 +60,6 @@ private async Task ReloadCache()
     public void Dispose()
     {
         _updateCacheTimer.Dispose();
+        _consulAdapter.Dispose();
     }
 }