Logger

AscarGb · Nov 20, 2018 · 44384f9 · 44384f9
1 parent 53671ab
commit 44384f9
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 9 deletions.
diff --git a/cmanager/Program.cs b/cmanager/Program.cs
@@ -8,6 +8,7 @@
 using Microsoft.EntityFrameworkCore;
 using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Logging;
 
 namespace cmanager
 {
@@ -37,6 +38,8 @@ static int Main(string[] args)
             services.AddTransient<RenameRoleVerb>();
             services.AddSingleton<ConsoleWriter>();
 
+            services.AddLogging(configure => configure.AddConsole())
+                .Configure<LoggerFilterOptions>(options => options.MinLevel = LogLevel.Information);
 
             var connectionString = configuration.GetConnectionString("DefaultConnection");
             var dbType = configuration.GetSection("DbType").Get<string>();
@@ -72,7 +75,8 @@ static int Main(string[] args)
             }
             catch (Exception e)
             {
-                provider.GetService<ConsoleWriter>().WriteText(e.Message);
+                var logger = provider.GetRequiredService<ILogger<Program>>();
+                logger.LogError(e, "An error occurred");
                 return 1;
             }
         }

diff --git a/cmanager/Verbs/AddRoleVerb.cs b/cmanager/Verbs/AddRoleVerb.cs
@@ -19,8 +19,7 @@ public override int Execute(AddRoleOptions opts)
 
             if (user == null)
             {
-                _consoleWriter.WriteText("User not found");
-                return 1;
+                throw new Exception("User not found");                
             }
 
             foreach (var r in opts.Roles)

diff --git a/cmanager/Verbs/DeleteUserVerb.cs b/cmanager/Verbs/DeleteUserVerb.cs
@@ -18,8 +18,7 @@ override public int Execute(DeleteUserOptions opts)
             var u = _userManager.FindByNameAsync(opts.UserName).Result;
             if (u == null)
             {
-                _consoleWriter.WriteText("Not found");
-                return 1;
+                throw new Exception("User not found");
             }
 
             var result = _userManager.DeleteAsync(u).Result;

diff --git a/cmanager/Verbs/FindUserVerb.cs b/cmanager/Verbs/FindUserVerb.cs
@@ -23,8 +23,7 @@ override public int Execute(FindUserOptions opts)
                 var u = _userManager.FindByNameAsync(opts.UserName).Result;
                 if (u == null)
                 {
-                    _consoleWriter.WriteText("Not found");
-                    return 1;
+                    throw new Exception("User not found");
                 }
                 else
                 {

diff --git a/cmanager/Verbs/RenameRoleVerb.cs b/cmanager/Verbs/RenameRoleVerb.cs
@@ -18,8 +18,7 @@ public override int Execute(RenameRoleOptions opts)
 
             if (role == null)
             {
-                _consoleWriter.WriteText("Role not found");
-                return 1;
+                throw new Exception("Role not found");
             }
 
             role.Name = opts.NewRoleName;

diff --git a/cmanager/Verbs/RolesListVerb.cs b/cmanager/Verbs/RolesListVerb.cs
@@ -15,6 +15,7 @@ public RolesListVerb(
         public override int Execute()
         {
             _consoleWriter.WriteText(string.Join(", ", _roleManager.Roles));
+
             return 0;
         }
     }

diff --git a/cmanager/cmanager.csproj b/cmanager/cmanager.csproj
@@ -13,6 +13,7 @@
     <PackageReference Include="Microsoft.AspNetCore.Identity.EntityFrameworkCore" Version="2.1.6" />
     <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="2.1.4" />
     <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="2.1.1" />
+    <PackageReference Include="Microsoft.Extensions.Logging.Console" Version="2.1.1" />
     <PackageReference Include="Pomelo.EntityFrameworkCore.MySql" Version="2.1.2" />
   </ItemGroup>