Added logging and global exception handling features

Core/Eisk.Core/Eisk.Core.csproj

@@ -5,6 +5,7 @@
   </PropertyGroup>
 
   <ItemGroup>
+    <PackageReference Include="log4net" Version="2.0.8" />
     <PackageReference Include="Microsoft.AspNetCore.All" Version="2.0.9" />
     <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="2.0.3" />
   </ItemGroup>

Core/Eisk.Core/Exceptions/GlobalExceptionHandler.cs

@@ -0,0 +1,48 @@
+using System;
+using System.Net;
+using System.Threading.Tasks;
+using Microsoft.AspNetCore.Http;
+using Eisk.Core.Logger;
+using Eisk.Domains.Entities;
+
+namespace Eisk.Core.Exceptions
+{
+    public class GlobalExceptionHandler
+    {
+        private readonly RequestDelegate _next;
+        private readonly ILogger _logger;
+
+        public GlobalExceptionHandler(RequestDelegate next, ILogger logger)
+        {
+            this._next = next;
+            this._logger = logger;
+        }
+
+        public async Task InvokeAsync(HttpContext httpContext)
+        {
+            try
+            {
+                await _next(httpContext);
+            }
+            catch(Exception exception)
+            {
+                _logger.Error($"Error in processing request: {exception}");
+                await HandleExceptionAsync(httpContext, exception);
+            }
+        }
+
+        private Task HandleExceptionAsync(HttpContext context, Exception exception)
+        {
+            context.Response.ContentType = "application/json";
+            context.Response.StatusCode = (int) HttpStatusCode.InternalServerError;
+
+            return context.Response.WriteAsync(new ErrorDetails()
+            {
+                ErrorCode = context.Response.StatusCode,
+                ErrorMessage = exception.Message
+            }.ToString());
+        }
+
+
+    }
+}

Core/Eisk.Core/Logger/Logger.cs

@@ -0,0 +1,56 @@
+using System;
+using System.Collections.Generic;
+using System.Reflection;
+using System.Text;
+using log4net;
+
+namespace Eisk.Core.Logger
+{
+    //wrapper for logger class
+    public interface ILogger
+    {
+        void Info(string message);
+        void Warn(string message);
+        void Debug(string message);
+        void Error(string message);
+    }
+
+    /// <summary>
+    /// Responsible for global logging 
+    /// </summary>
+    public class Logger : ILogger
+    {
+        private ILog _logger;
+
+        public Logger()
+        {
+            this._logger = LogManager.GetLogger(Assembly.GetCallingAssembly(), "logger");
+        }
+
+        public Logger(Type logClass)
+        {
+            this._logger = LogManager.GetLogger(logClass);
+        }
+
+        public void Debug(string message)
+        {
+            _logger.Debug(message);
+        }
+
+        public void Error(string message)
+        {
+            _logger.Error(message);
+        }
+
+        public void Info(string message)
+        {
+            _logger.Info(message);
+        }
+
+        public void Warn(string message)
+        {
+            _logger.Warn(message);
+        }
+    }
+}
+

DomainCore/Eisk.Domains/Entities/ErrorDetails.cs

@@ -0,0 +1,15 @@
+using Newtonsoft.Json;
+
+namespace Eisk.Domains.Entities
+{
+    public class ErrorDetails
+    {
+        public int ErrorCode { get; set; }
+        public string ErrorMessage { get; set; }
+
+        public override string ToString()
+        {
+            return JsonConvert.SerializeObject(this);
+        }
+    }
+}

WebApi/Eisk.WebApi/Controllers/EmployeesController.cs

@@ -8,7 +8,7 @@ public class EmployeesController : WebApiControllerBase<Employee,int>
     {
         public EmployeesController(EmployeeDomainService employeeDomainService):base(employeeDomainService)
         {
-
+            throw new System.Exception("test");
         }
     }
 }

WebApi/Eisk.WebApi/Eisk.WebApi.csproj

@@ -12,6 +12,7 @@
   </ItemGroup>
 
   <ItemGroup>
+    <PackageReference Include="log4net" Version="2.0.8" />
     <PackageReference Include="Microsoft.AspNetCore.All" Version="2.0.9" />
     <PackageReference Include="Swashbuckle.AspNetCore" Version="4.0.1" />
   </ItemGroup>

WebApi/Eisk.WebApi/Program.cs

@@ -1,12 +1,19 @@
-using Microsoft.AspNetCore;
+using System.Reflection;
+using System.IO;
+using Microsoft.AspNetCore;
 using Microsoft.AspNetCore.Hosting;
+using log4net;
+using log4net.Config;
 
 namespace Eisk.WebApi
 {
     public class Program
     {
         public static void Main(string[] args)
         {
+            var logRepository = LogManager.GetRepository(Assembly.GetEntryAssembly());
+            XmlConfigurator.Configure(logRepository, new FileInfo("log4net.config"));
+
             BuildWebHost(args).Run();
         }
 

WebApi/Eisk.WebApi/Startup.cs

@@ -15,6 +15,8 @@ namespace Eisk.WebApi
     using DataServices.Interfaces;
     using DomainServices;
     using EFCore.Setup;
+    using Eisk.Core.Exceptions;
+    using Eisk.Core.Logger;
 
     public class Startup
     {
@@ -46,6 +48,9 @@ public void ConfigureServices(IServiceCollection services)
 
             services.AddTransient<EmployeeDomainService>();
 
+            //Logging service
+            services.AddSingleton<ILogger, Logger>();
+
             services.AddMvc();
 
             // Register the Swagger generator, defining 1 or more Swagger documents
@@ -77,6 +82,9 @@ public void Configure(IApplicationBuilder app, IHostingEnvironment env)
                 app.UseDeveloperExceptionPage();
             }
 
+            //global exception middleware
+            app.UseMiddleware<GlobalExceptionHandler>();
+
             // Enable middleware to serve generated Swagger as a JSON endpoint.
             app.UseSwagger();
 

WebApi/Eisk.WebApi/log4net.config

@@ -0,0 +1,19 @@
+<configuration>
+  <configSections>
+    <section name="log4net" type="log4net.Config.Log4NetConfigurationSectionHandler, log4net" />
+  </configSections>
+  <log4net>
+    <appender name="FileAppender" type="log4net.Appender.FileAppender">
+      <file value="logfile.log" />
+      <appendToFile value="true" />
+      <layout type="log4net.Layout.PatternLayout">
+        <conversionPattern value="%d [%t] %-5p - %m%n" />
+      </layout>
+    </appender>
+    <root>
+      <!--LogLevel: OFF, FATAL, ERROR, WARN, INFO, DEBUG, ALL -->
+      <level value="ALL" />
+      <appender-ref ref="FileAppender" />
+    </root>
+  </log4net>
+</configuration>