Adding logging to database query exceptions,

thanks to @Recodify

Author: d4nt

Web/Controllers/ApiController.cs

@@ -7,19 +7,16 @@
 using System.Globalization;
 using System.Linq;
 using System.Net;
-using System.Text;
-using System.Collections.Generic;
-using System.IO;
 using System.Text.RegularExpressions;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.Extensions.Caching.Memory;
+using Microsoft.Extensions.Logging;
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.EntityFrameworkCore;
 using Hangfire;
 using Microsoft.Extensions.Configuration;
-using MimeKit;
 using Microsoft.AspNetCore.Identity;
 
 namespace QueryTree.Controllers
@@ -45,7 +42,8 @@ public ApiController(UserManager<ApplicationUser> userManager,
             IPasswordManager passwordManager, 
             IConfiguration config, 
             IScheduledEmailManager scheduledEmailManager,
-            IWebHostEnvironment env)
+            IWebHostEnvironment env,
+            ILoggerFactory loggerFactory)
         {
             _userManager = userManager;
             db = dbContext;
@@ -54,7 +52,7 @@ public ApiController(UserManager<ApplicationUser> userManager,
             _config = config;
             _scheduledEmailManager = scheduledEmailManager;
             _env = env;
-            _dbMgr = new DbManager(passwordManager, cache, config);
+            _dbMgr = new DbManager(passwordManager, cache, config, loggerFactory);
             this.convertManager = new ConvertManager();
         }
 

Web/Controllers/HomeController.cs

@@ -9,6 +9,7 @@
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.Extensions.Caching.Memory;
 using Microsoft.Extensions.Configuration;
+using Microsoft.Extensions.Logging;
 using Microsoft.EntityFrameworkCore;
 using QueryTree.Enums;
 
@@ -26,10 +27,11 @@ public HomeController(
             UserManager<ApplicationUser> userManager,
             IPasswordManager passwordManager,
             IConfiguration config,
-            IMemoryCache cache)
+            IMemoryCache cache,
+            ILoggerFactory loggerFactory)
             : base(userManager, dbContext)
         {
-            _dbMgr = new DbManager(passwordManager, cache, config);
+            _dbMgr = new DbManager(passwordManager, cache, config, loggerFactory);
             _passwordManager = passwordManager;
         }
 

Web/Managers/DbManager.cs

@@ -13,6 +13,7 @@
 using QueryTree.Enums;
 using QueryTree.ViewModels;
 using System.Text.RegularExpressions;
+using Microsoft.Extensions.Logging;
 
 
 namespace QueryTree.Managers
@@ -22,12 +23,14 @@ public class DbManager
         private IPasswordManager _passwordManager;
         private IMemoryCache _cache;
         private IConfiguration _config;
+        private readonly ILogger _logger;
 
-        public DbManager(IPasswordManager passwordManager, IMemoryCache cache, IConfiguration config)
+        public DbManager(IPasswordManager passwordManager, IMemoryCache cache, IConfiguration config, ILoggerFactory loggerFactory)
         {
             _passwordManager = passwordManager;
             _cache = cache;
             _config = config;
+            _logger = loggerFactory.CreateLogger<DbManager>();
         }
 
         private DbTable GetDbTable(DatabaseType type, DbConnection conn, int connectionId, string databaseName, string tableName)
@@ -587,10 +590,12 @@ public bool TryUseDbConnection(DatabaseType type, string server, int port, bool
             }
             catch (SqlException e)
             {
+                _logger.LogError(e, "Unable to query db");
                 error = e.Message;
             }
             catch (Exception e)
             {
+                _logger.LogError(e, "Unable to query db");
                 error = e.Message;
             }
 

Web/Managers/ScheduledEmailManager.cs

@@ -8,6 +8,7 @@
 using Microsoft.EntityFrameworkCore;
 using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.Caching.Memory;
+using Microsoft.Extensions.Logging;
 using QueryTree.Services;
 
 namespace QueryTree.Managers
@@ -36,14 +37,15 @@ public ScheduledEmailManager(
             ApplicationDbContext db,
             IWebHostEnvironment env,
             IMemoryCache cache,
-            IPasswordManager passwordManager)
+            IPasswordManager passwordManager,
+            ILoggerFactory loggerFactory)
         {
 			_emailSenderService = emailSenderService;
             _emailSender = emailSender;
             _config = config;
             _db = db;
             _env = env;
-            _dbMgr = new DbManager(passwordManager, cache, config);
+            _dbMgr = new DbManager(passwordManager, cache, config, loggerFactory);
             _convertManager = new ConvertManager();
         }