From 9d2b2405d5a93f2ee7bb37e72471eca392e9e4c0 Mon Sep 17 00:00:00 2001 From: csimonapastore Date: Sun, 23 Mar 2025 22:05:32 +0100 Subject: [PATCH] Changed CreateUserAsync to avoid silent transaction failure - 2 --- MainProject/Services/UserService.cs | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/MainProject/Services/UserService.cs b/MainProject/Services/UserService.cs index b245a3b..d88cf34 100644 --- a/MainProject/Services/UserService.cs +++ b/MainProject/Services/UserService.cs @@ -20,6 +20,7 @@ public interface IUserService public class UserService : BaseService, IUserService { + private readonly NLog.Logger Logger = NLog.LogManager.GetCurrentClassLogger(); public UserService( IHttpContextAccessor httpContextAccessor, IConfiguration configuration, @@ -111,7 +112,9 @@ public class UserService : BaseService, IUserService { User? user = null; - using (var transaction = await _sqlServerContext.Database.BeginTransactionAsync()) + using var transaction = await _sqlServerContext.Database.BeginTransactionAsync(); + + try { var tempUser = CreateUserData(data, role); await _sqlServerContext.Users.AddAsync(tempUser); @@ -119,6 +122,11 @@ public class UserService : BaseService, IUserService await transaction.CommitAsync(); user = tempUser; } + catch (Exception exception) + { + Logger.Error(exception, $"[UserService][CreateUserAsync]"); + } + return user; }