diff --git a/src/WorkflowCore/Services/BackgroundTasks/EventConsumer.cs b/src/WorkflowCore/Services/BackgroundTasks/EventConsumer.cs index dd7323b01..26e630ea5 100644 --- a/src/WorkflowCore/Services/BackgroundTasks/EventConsumer.cs +++ b/src/WorkflowCore/Services/BackgroundTasks/EventConsumer.cs @@ -114,7 +114,7 @@ private async Task SeedSubscription(Event evt, EventSubscription sub, Hash if (!await _lockProvider.AcquireLock(sub.WorkflowId, cancellationToken)) { - Logger.LogInformation("Workflow locked {0}", sub.WorkflowId); + Logger.LogInformation("Workflow locked {WorkflowId}", sub.WorkflowId); return false; } @@ -151,4 +151,4 @@ private async Task SeedSubscription(Event evt, EventSubscription sub, Hash } } } -} +} \ No newline at end of file diff --git a/src/WorkflowCore/Services/BackgroundTasks/RunnablePoller.cs b/src/WorkflowCore/Services/BackgroundTasks/RunnablePoller.cs index fcd2abd92..29b76837c 100644 --- a/src/WorkflowCore/Services/BackgroundTasks/RunnablePoller.cs +++ b/src/WorkflowCore/Services/BackgroundTasks/RunnablePoller.cs @@ -94,7 +94,7 @@ await _persistenceStore.ScheduleCommand(new ScheduledCommand() _logger.LogDebug($"Got greylisted workflow {item}"); continue; } - _logger.LogDebug("Got runnable instance {0}", item); + _logger.LogDebug("Got runnable instance {Item}", item); _greylist.Add($"wf:{item}"); await _queueProvider.QueueWork(item, QueueType.Workflow); } @@ -218,4 +218,4 @@ await _persistenceStore.ProcessCommands(new DateTimeOffset(_dateTimeProvider.Utc } } } -} +} \ No newline at end of file diff --git a/src/WorkflowCore/Services/BackgroundTasks/WorkflowConsumer.cs b/src/WorkflowCore/Services/BackgroundTasks/WorkflowConsumer.cs index b092f41f6..ad9c9e6f6 100644 --- a/src/WorkflowCore/Services/BackgroundTasks/WorkflowConsumer.cs +++ b/src/WorkflowCore/Services/BackgroundTasks/WorkflowConsumer.cs @@ -33,7 +33,7 @@ protected override async Task ProcessItem(string itemId, CancellationToken cance { if (!await _lockProvider.AcquireLock(itemId, cancellationToken)) { - Logger.LogInformation("Workflow locked {0}", itemId); + Logger.LogInformation("Workflow locked {ItemId}", itemId); return; } @@ -165,4 +165,4 @@ private async void FutureQueue(WorkflowInstance workflow, CancellationToken canc } } } -} +} \ No newline at end of file diff --git a/src/WorkflowCore/Services/WorkflowController.cs b/src/WorkflowCore/Services/WorkflowController.cs index 6edb63aa7..79272e084 100755 --- a/src/WorkflowCore/Services/WorkflowController.cs +++ b/src/WorkflowCore/Services/WorkflowController.cs @@ -107,7 +107,7 @@ await _eventHub.PublishNotification(new WorkflowStarted public async Task PublishEvent(string eventName, string eventKey, object eventData, DateTime? effectiveDate = null) { - _logger.LogDebug("Creating event {0} {1}", eventName, eventKey); + _logger.LogDebug("Creating event {EventName} {EventKey}", eventName, eventKey); Event evt = new Event(); if (effectiveDate.HasValue) @@ -241,4 +241,4 @@ public void RegisterWorkflow() _registry.RegisterWorkflow(wf); } } -} +} \ No newline at end of file diff --git a/src/WorkflowCore/Services/WorkflowExecutor.cs b/src/WorkflowCore/Services/WorkflowExecutor.cs index 145f02d41..da3e9cd85 100755 --- a/src/WorkflowCore/Services/WorkflowExecutor.cs +++ b/src/WorkflowCore/Services/WorkflowExecutor.cs @@ -47,7 +47,7 @@ public async Task Execute(WorkflowInstance workflow, Can var def = _registry.GetDefinition(workflow.WorkflowDefinitionId, workflow.Version); if (def == null) { - _logger.LogError("Workflow {0} version {1} is not registered", workflow.WorkflowDefinitionId, workflow.Version); + _logger.LogError("Workflow {WorkflowDefinitionId} version {Version} is not registered", workflow.WorkflowDefinitionId, workflow.Version); return wfResult; } @@ -61,7 +61,7 @@ public async Task Execute(WorkflowInstance workflow, Can var step = def.Steps.FindById(pointer.StepId); if (step == null) { - _logger.LogError("Unable to find step {0} in workflow definition", pointer.StepId); + _logger.LogError("Unable to find step {StepId} in workflow definition", pointer.StepId); pointer.SleepUntil = _datetimeProvider.UtcNow.Add(_options.ErrorRetryInterval); wfResult.Errors.Add(new ExecutionError { @@ -83,7 +83,7 @@ public async Task Execute(WorkflowInstance workflow, Can } catch (Exception ex) { - _logger.LogError(ex, "Workflow {0} raised error on step {1} Message: {2}", workflow.Id, pointer.StepId, ex.Message); + _logger.LogError(ex, "Workflow {WorkflowId} raised error on step {StepId} Message: {Message}", workflow.Id, pointer.StepId, ex.Message); wfResult.Errors.Add(new ExecutionError { WorkflowId = workflow.Id, @@ -158,14 +158,14 @@ private async Task ExecuteStep(WorkflowInstance workflow, WorkflowStep step, Exe using (var scope = _scopeProvider.CreateScope(context)) { - _logger.LogDebug("Starting step {0} on workflow {1}", step.Name, workflow.Id); + _logger.LogDebug("Starting step {StepName} on workflow {WorkflowId}", step.Name, workflow.Id); IStepBody body = step.ConstructBody(scope.ServiceProvider); var stepExecutor = scope.ServiceProvider.GetRequiredService(); if (body == null) { - _logger.LogError("Unable to construct step body {0}", step.BodyType.ToString()); + _logger.LogError("Unable to construct step body {BodyType}", step.BodyType.ToString()); pointer.SleepUntil = _datetimeProvider.UtcNow.Add(_options.ErrorRetryInterval); wfResult.Errors.Add(new ExecutionError { @@ -275,4 +275,4 @@ private async Task DetermineNextExecutionTime(WorkflowInstance workflow, Workflo }); } } -} +} \ No newline at end of file