feat(logging): add 404 logging

fossapps · Jan 23, 2020 · 78d7939 · 78d7939
1 parent 88bb07a
commit 78d7939
Show file tree

Hide file tree

Showing 4 changed files with 35 additions and 3 deletions.
diff --git a/Micro.KeyStore.Api/Middlewares/RequestResponseLoggingMiddleware.cs b/Micro.KeyStore.Api/Middlewares/RequestResponseLoggingMiddleware.cs
@@ -0,0 +1,28 @@
+using System.Threading.Tasks;
+using Microsoft.AspNetCore.Http;
+using Microsoft.Extensions.Logging;
+
+namespace Micro.KeyStore.Api.Middlewares
+{
+    public class RequestResponseLoggingMiddleware
+    {
+        private readonly RequestDelegate _next;
+        private readonly ILogger<RequestResponseLoggingMiddleware> _logger;
+
+        public RequestResponseLoggingMiddleware(RequestDelegate next, ILogger<RequestResponseLoggingMiddleware> logger)
+        {
+            _next = next;
+            _logger = logger;
+        }
+
+        public async Task Invoke(HttpContext context)
+        {
+            await _next(context);
+            if (context.Response.StatusCode != StatusCodes.Status404NotFound)
+            {
+                return;
+            }
+            _logger.LogInformation($"404: {context.Request.Path}");
+        }
+    }
+}
diff --git a/Micro.KeyStore.Api/Program.cs b/Micro.KeyStore.Api/Program.cs
@@ -27,9 +27,10 @@ public static async Task Main(string[] args)
                 var logger = scope.ServiceProvider.GetRequiredService<ILogger<Program>>();
                 try
                 {
-                    var autoMigrate = host.Services.GetRequiredService<IOptions<DatabaseConfig>>().Value.AutoMigrate;
-                    if (autoMigrate)
+                    var dbConfig = host.Services.GetRequiredService<IOptions<DatabaseConfig>>().Value;
+                    if (dbConfig.AutoMigrate)
                     {
+                        logger.LogInformation($"automatically migrating database: {dbConfig.Name}{dbConfig.User}@{dbConfig.Host}");
                         var db = scope.ServiceProvider.GetRequiredService<ApplicationContext>().Database;
                         await db.MigrateOrFail(logger);
                     }

diff --git a/Micro.KeyStore.Api/Startup.cs b/Micro.KeyStore.Api/Startup.cs
@@ -1,4 +1,5 @@
 using Micro.KeyStore.Api.Configs;
+using Micro.KeyStore.Api.Middlewares;
 using Micro.KeyStore.Api.StartupExtensions;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Hosting;
@@ -40,6 +41,8 @@ public void Configure(IApplicationBuilder app, IWebHostEnvironment env, ILoggerF
             {
                 app.UseDeveloperExceptionPage();
             }
+
+            app.UseMiddleware<RequestResponseLoggingMiddleware>();
             app.UseRouting();
             app.UseAuthorization();
             app.SetupSwaggerAndUi();

diff --git a/Micro.KeyStore.Api/Workers/CleanupKeysWorker.cs b/Micro.KeyStore.Api/Workers/CleanupKeysWorker.cs
@@ -72,7 +72,7 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)
                         }
                     }
 
-                    _logger.LogInformation($"Archived {keys.Count} keys at {DateTime.Now}");
+                    _logger.LogDebug($"Archived {keys.Count} keys at {DateTime.Now}");
                     if (stoppingToken.IsCancellationRequested) return;
                     await Task.Delay(TimeSpan.FromSeconds(_config.BatchIntervalInSeconds), stoppingToken);
                 }