Skip to content

Commit 88abd95

Browse files
Merge pull request #1 from chenxidev1129/feature/add-logger
implement logger service
2 parents 78dba75 + 25067ed commit 88abd95

File tree

2 files changed

+30
-2
lines changed

2 files changed

+30
-2
lines changed

taskmaster-api/Services/AuthService.cs

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
using Microsoft.IdentityModel.Tokens;
88
using System.Text;
99
using taskmaster_api.Data.Models;
10+
using Microsoft.Extensions.Logging;
1011

1112
namespace taskmaster_api.Services
1213
{
@@ -15,12 +16,14 @@ public class AuthService : IAuthService
1516
private readonly UserManager<IdentityUser> _userManager;
1617
private readonly RoleManager<IdentityRole> _roleManager;
1718
private readonly IConfiguration _configuration;
19+
private readonly ILogger<AuthService> _logger;
1820

19-
public AuthService(UserManager<IdentityUser> userManager, RoleManager<IdentityRole> roleManager, IConfiguration configuration)
21+
public AuthService(UserManager<IdentityUser> userManager, RoleManager<IdentityRole> roleManager, IConfiguration configuration, ILogger<AuthService> logger)
2022
{
2123
_userManager = userManager;
2224
_roleManager = roleManager;
2325
_configuration = configuration;
26+
_logger = logger;
2427
}
2528

2629
public ICoreActionResult<LoginResponseDto> Login(LoginDto loginDto)
@@ -45,17 +48,21 @@ public ICoreActionResult<LoginResponseDto> Login(LoginDto loginDto)
4548

4649
var token = GetToken(authClaims);
4750

51+
_logger.LogInformation("Login Successful.");
52+
4853
return CoreActionResult<LoginResponseDto>.Success(new LoginResponseDto
4954
{
5055
Token = new JwtSecurityTokenHandler().WriteToken(token),
5156
Expiration = token.ValidTo,
5257
});
5358
}
5459

60+
_logger.LogInformation("Login Failed.");
5561
return CoreActionResult<LoginResponseDto>.Failure("Login Failed.");
5662
}
5763
catch (Exception ex)
5864
{
65+
_logger.LogInformation(ex.Message);
5966
return CoreActionResult<LoginResponseDto>.Exception(ex);
6067
}
6168
}
@@ -67,6 +74,7 @@ public ICoreActionResult<RegisterDto> Register(RegisterDto registerDto)
6774
var userExists = _userManager.FindByNameAsync(registerDto.Username).Result;
6875
if (userExists != null)
6976
{
77+
_logger.LogInformation("User already exists!");
7078
return CoreActionResult<RegisterDto>.Failure("User already exists!");
7179
}
7280

@@ -80,13 +88,17 @@ public ICoreActionResult<RegisterDto> Register(RegisterDto registerDto)
8088

8189
if (!result.Succeeded)
8290
{
91+
_logger.LogInformation("User creation failed! Please check user details and try again.");
8392
return CoreActionResult<RegisterDto>.Failure("User creation failed! Please check user details and try again.");
8493
}
8594

95+
_logger.LogInformation("Register Successful.");
96+
8697
return CoreActionResult<RegisterDto>.Success(registerDto);
8798
}
8899
catch (Exception ex)
89100
{
101+
_logger.LogInformation(ex.Message);
90102
return CoreActionResult<RegisterDto>.Exception(ex);
91103
}
92104
}
@@ -98,6 +110,7 @@ public ICoreActionResult<RegisterDto> RegisterAdmin(RegisterDto registerDto)
98110
var userExists = _userManager.FindByNameAsync(registerDto.Username).Result;
99111
if (userExists != null)
100112
{
113+
_logger.LogInformation("User already exists!");
101114
return CoreActionResult<RegisterDto>.Failure("User already exists!");
102115
}
103116

@@ -111,6 +124,7 @@ public ICoreActionResult<RegisterDto> RegisterAdmin(RegisterDto registerDto)
111124
var result = _userManager.CreateAsync(user, registerDto.Password).Result;
112125
if (!result.Succeeded)
113126
{
127+
_logger.LogInformation("User creation failed! Please check user details and try again.");
114128
return CoreActionResult<RegisterDto>.Failure("User creation failed! Please check user details and try again.");
115129
}
116130

@@ -134,10 +148,13 @@ public ICoreActionResult<RegisterDto> RegisterAdmin(RegisterDto registerDto)
134148
var userIdentityResult = _userManager.AddToRoleAsync(user, UserRoles.User).Result;
135149
}
136150

151+
_logger.LogInformation("Admin Register Successful.");
152+
137153
return CoreActionResult<RegisterDto>.Success(registerDto);
138154
}
139155
catch (Exception ex)
140156
{
157+
_logger.LogInformation(ex.Message);
141158
return CoreActionResult<RegisterDto>.Exception(ex);
142159
}
143160
}

taskmaster-api/Services/TaskService.cs

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
using Microsoft.AspNetCore.Mvc;
2+
using Microsoft.Extensions.Logging;
23
using taskmaster_api.Data.DTOs;
34
using taskmaster_api.Data.DTOs.Interface;
45
using taskmaster_api.Data.Entities;
@@ -10,8 +11,9 @@ namespace taskmaster_api.Services
1011
public class TaskService : ITaskService
1112
{
1213
private readonly ITaskRepository _taskRepository;
14+
private readonly ILogger<TaskService> _logger;
1315

14-
public TaskService(ITaskRepository taskRepository)
16+
public TaskService(ITaskRepository taskRepository, ILogger<TaskService> logger)
1517
{
1618
_taskRepository = taskRepository;
1719
}
@@ -23,12 +25,15 @@ public ICoreActionResult<TaskDto> GetTaskById(int id)
2325
var task = _taskRepository.GetTaskById(id);
2426
if (task == null)
2527
{
28+
_logger.LogInformation("Task not found");
2629
return CoreActionResult<TaskDto>.Failure("Task not found", "NotFound");
2730
}
31+
2832
return CoreActionResult<TaskDto>.Success(task.ToDto());
2933
}
3034
catch (Exception ex)
3135
{
36+
_logger.LogInformation(ex.Message);
3237
return CoreActionResult<TaskDto>.Exception(ex);
3338
}
3439
}
@@ -42,6 +47,7 @@ public ICoreActionResult<List<TaskDto>> GetAllTasks()
4247
}
4348
catch (Exception ex)
4449
{
50+
_logger.LogInformation(ex.Message);
4551
return CoreActionResult<List<TaskDto>>.Exception(ex);
4652
}
4753
}
@@ -56,6 +62,7 @@ public ICoreActionResult<TaskDto> CreateTask(TaskDto taskDto)
5662
}
5763
catch (Exception ex)
5864
{
65+
_logger.LogInformation(ex.Message);
5966
return CoreActionResult<TaskDto>.Exception(ex);
6067
}
6168
}
@@ -68,6 +75,7 @@ public ICoreActionResult<TaskDto> UpdateTask(int id, TaskDto taskDto)
6875
var existingTask = _taskRepository.GetTaskById(id);
6976
if (existingTask == null)
7077
{
78+
_logger.LogInformation("Task not found");
7179
return CoreActionResult<TaskDto>.Failure("Task not found", "NotFound");
7280
}
7381

@@ -76,6 +84,7 @@ public ICoreActionResult<TaskDto> UpdateTask(int id, TaskDto taskDto)
7684
}
7785
catch (Exception ex)
7886
{
87+
_logger.LogInformation(ex.Message);
7988
return CoreActionResult<TaskDto>.Exception(ex);
8089
}
8190
}
@@ -87,12 +96,14 @@ public ICoreActionResult DeleteTask(int id)
8796
var deletedTaskId = _taskRepository.DeleteTask(id);
8897
if (deletedTaskId == 0)
8998
{
99+
_logger.LogInformation("Task not found");
90100
return CoreActionResult.Ignore("Task not found", "NotFound");
91101
}
92102
return CoreActionResult.Success();
93103
}
94104
catch (Exception ex)
95105
{
106+
_logger.LogInformation(ex.Message);
96107
return CoreActionResult.Exception(ex);
97108
}
98109
}

0 commit comments

Comments
 (0)