Skip to content

Commit

Permalink
Merge pull request #1198 from jakenuts/fix-nullref-in-workflow-activity
Browse files Browse the repository at this point in the history
Fixed usage of null WorkflowStep.Name in WorkflowActivity diagnostic service.
  • Loading branch information
danielgerlag authored Nov 8, 2023
2 parents f7fc3aa + 2355322 commit b8c9e2c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions src/Directory.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@
<PackageLicenseUrl>https://github.com/danielgerlag/workflow-core/blob/master/LICENSE.md</PackageLicenseUrl>
<RepositoryType>git</RepositoryType>
<RepositoryUrl>https://github.com/danielgerlag/workflow-core.git</RepositoryUrl>
<Version>3.9.0</Version>
<AssemblyVersion>3.9.0.0</AssemblyVersion>
<FileVersion>3.9.0.0</FileVersion>
<Version>3.9.1</Version>
<AssemblyVersion>3.9.1.0</AssemblyVersion>
<FileVersion>3.9.1.0</FileVersion>
<PackageIconUrl>https://github.com/danielgerlag/workflow-core/raw/master/src/logo.png</PackageIconUrl>
<PackageVersion>3.9.0</PackageVersion>
<PackageVersion>3.9.1</PackageVersion>
</PropertyGroup>
</Project>
2 changes: 1 addition & 1 deletion src/WorkflowCore/Services/WorkflowActivity.cs
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ internal static void Enrich(WorkflowStep workflowStep)

activity.DisplayName += $" step {stepName}";
activity.SetTag("workflow.step.id", workflowStep.Id);
activity.SetTag("workflow.step.name", workflowStep.Name);
activity.SetTag("workflow.step.name", stepName);
activity.SetTag("workflow.step.type", workflowStep.BodyType.Name);
}
}
Expand Down

0 comments on commit b8c9e2c

Please sign in to comment.