From 511ba665cb03bf87b16b40a04764dea68486e662 Mon Sep 17 00:00:00 2001
From: Jared <translucency@outlook.com>
Date: Wed, 3 Feb 2021 22:41:06 -0800
Subject: [PATCH] Cosmetics.

---
 Server/Services/DbLogger.cs         | 18 +++++++++---------
 Server/Services/DbLoggerProvider.cs | 11 ++++++-----
 2 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/Server/Services/DbLogger.cs b/Server/Services/DbLogger.cs
index bd94cf7a..ceb5de3a 100644
--- a/Server/Services/DbLogger.cs
+++ b/Server/Services/DbLogger.cs
@@ -11,17 +11,17 @@ namespace Remotely.Server.Services
 {
     public class DbLogger : ILogger
     {
-        private readonly string categoryName;
-        private readonly IWebHostEnvironment hostEnvironment;
-        private readonly IServiceProvider serviceProvider;
+        private readonly string _categoryName;
+        private readonly IWebHostEnvironment _hostEnvironment;
+        private readonly IServiceProvider _serviceProvider;
 
         protected static ConcurrentStack<string> ScopeStack { get; } = new ConcurrentStack<string>();
 
         public DbLogger(string categoryName, IWebHostEnvironment hostEnvironment, IServiceProvider serviceProvider)
         {
-            this.categoryName = categoryName;
-            this.hostEnvironment = hostEnvironment;
-            this.serviceProvider = serviceProvider;
+            _categoryName = categoryName;
+            _hostEnvironment = hostEnvironment;
+            _serviceProvider = serviceProvider;
         }
 
         public IDisposable BeginScope<TState>(TState state)
@@ -38,7 +38,7 @@ namespace Remotely.Server.Services
                     break;
                 case LogLevel.Debug:
                 case LogLevel.Information:
-                    if (hostEnvironment.IsDevelopment())
+                    if (_hostEnvironment.IsDevelopment())
                     {
                         return true;
                     }
@@ -57,9 +57,9 @@ namespace Remotely.Server.Services
 
         public void Log<TState>(LogLevel logLevel, EventId eventId, TState state, Exception exception, Func<TState, Exception, string> formatter)
         {
-            using var scope = serviceProvider.CreateScope();
+            using var scope = _serviceProvider.CreateScope();
             var dataService = scope.ServiceProvider.GetRequiredService<IDataService>();
-            dataService.WriteLog(logLevel, categoryName, eventId, state.ToString(), exception, ScopeStack.ToList());
+            dataService.WriteLog(logLevel, _categoryName, eventId, state.ToString(), exception, ScopeStack.ToList());
         }
 
 
diff --git a/Server/Services/DbLoggerProvider.cs b/Server/Services/DbLoggerProvider.cs
index fca92c05..739fa324 100644
--- a/Server/Services/DbLoggerProvider.cs
+++ b/Server/Services/DbLoggerProvider.cs
@@ -6,23 +6,24 @@ namespace Remotely.Server.Services
 {
     public class DbLoggerProvider : ILoggerProvider
     {
-        private readonly IWebHostEnvironment hostEnvironment;
-        private readonly IServiceProvider serviceProvider;
+        private readonly IWebHostEnvironment _hostEnvironment;
+        private readonly IServiceProvider _serviceProvider;
 
         public DbLoggerProvider(IWebHostEnvironment hostEnvironment, IServiceProvider serviceProvider)
         {
-            this.hostEnvironment = hostEnvironment;
-            this.serviceProvider = serviceProvider;
+            _hostEnvironment = hostEnvironment;
+            _serviceProvider = serviceProvider;
         }
 
 
         public ILogger CreateLogger(string categoryName)
         {
-            return new DbLogger(categoryName, hostEnvironment, serviceProvider);
+            return new DbLogger(categoryName, _hostEnvironment, _serviceProvider);
         }
 
         public void Dispose()
         {
+            GC.SuppressFinalize(this);
         }
     }
 }