diff --git a/Samples.sln b/Samples.sln index 9eb8a4f82..e1235a835 100644 --- a/Samples.sln +++ b/Samples.sln @@ -238,6 +238,7 @@ Global {E20238CE-5C64-4F87-8753-3B4E1EC9A740}.Debug|Any CPU.Build.0 = Debug|Any CPU {E20238CE-5C64-4F87-8753-3B4E1EC9A740}.Release|Any CPU.ActiveCfg = Release|Any CPU {E20238CE-5C64-4F87-8753-3B4E1EC9A740}.Release|Any CPU.Build.0 = Release|Any CPU + EndGlobalSection GlobalSection(SolutionProperties) = preSolution HideSolutionNode = FALSE diff --git a/src/providers/Elsa.DistributedLocking.Redis/RedisLockProvider.cs b/src/providers/Elsa.DistributedLocking.Redis/RedisLockProvider.cs index dbb41ffad..10d3c8ec5 100644 --- a/src/providers/Elsa.DistributedLocking.Redis/RedisLockProvider.cs +++ b/src/providers/Elsa.DistributedLocking.Redis/RedisLockProvider.cs @@ -34,7 +34,7 @@ namespace Elsa.DistributedLocking.Redis private async Task CreateLockAsync(string name, CancellationToken cancellationToken = default) { var resourceName = $"{Prefix}:{name}"; - _logger.LogInformation($"Lock provider will try to acquire lock for {resourceName}"); + _logger.LogInformation("Lock provider will try to acquire lock for {resourceName}",resourceName); try { var redLock = await _distributedLockFactory.CreateLockAsync(resourceName, _lockTimeout, @@ -50,7 +50,7 @@ namespace Elsa.DistributedLocking.Redis { RedLockInstance.Add(redLock); } - _logger.LogInformation($"Lock provider acquired lock for {resourceName}"); + _logger.LogInformation("Lock provider acquired lock for {resourceName}",resourceName); return true; } @@ -58,7 +58,7 @@ namespace Elsa.DistributedLocking.Redis } catch (Exception ex) { - _logger.LogWarning($"Failed to acquire lock for {resourceName}. Reason > {ex}"); + _logger.LogWarning("Failed to acquire lock for {resourceName}. Reason > {ex}",resourceName,ex); return false; } } @@ -71,7 +71,8 @@ namespace Elsa.DistributedLocking.Redis } var resourceName = $"{Prefix}:{name}"; - _logger.LogInformation($"Lock provider will try to release lock for {resourceName}"); + _logger.LogInformation("Lock provider will try to release lock for {resourceName}", resourceName); + try { lock (RedLockInstance) @@ -82,7 +83,8 @@ namespace Elsa.DistributedLocking.Redis { redLock.Dispose(); RedLockInstance.Remove(redLock); - _logger.LogInformation($"Lock provider released lock for {resourceName}"); + _logger.LogInformation("Lock provider released lock for {resourceName}",resourceName); + break; } } @@ -90,7 +92,7 @@ namespace Elsa.DistributedLocking.Redis } catch (Exception ex) { - _logger.LogWarning($"Failed to release lock for {resourceName}. Reason > {ex}"); + _logger.LogWarning("Failed to release lock for {resourceName}. Reason > {ex}",resourceName,ex); } return Task.CompletedTask; } diff --git a/src/samples/Elsa.Samples.DistributedLock/Program.cs b/src/samples/Elsa.Samples.DistributedLock/Program.cs index 11075e576..77b643049 100644 --- a/src/samples/Elsa.Samples.DistributedLock/Program.cs +++ b/src/samples/Elsa.Samples.DistributedLock/Program.cs @@ -12,7 +12,7 @@ namespace Elsa.Samples.DistributedLock { private static async Task Main() { - await CreateHostBuilder().UseConsoleLifetime().Build().RunAsync(); + await CreateHostBuilder().UseConsoleLifetime().Build().RunAsync(); } public static IHostBuilder CreateHostBuilder() =>