Fix empty ddtags being added to logs

lucaspimentel · claude · lucaspimentel · commit 59fcc58bc88c · 2025-10-10T15:24:43.000-04:00
Changed the check to use !string.IsNullOrEmpty(Tags) instead of Tags is not null to prevent writing empty ddtags property to JSON logs. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/tracer/src/Datadog.Trace/Logging/DirectSubmission/Formatting/LogFormatter.cs b/tracer/src/Datadog.Trace/Logging/DirectSubmission/Formatting/LogFormatter.cs
@@ -349,7 +349,7 @@ internal void FormatLog<T>(
 
             EnrichTagsStringWithGitMetadata();
 
-            if (Tags is not null && !renderingDetails.HasRenderedTags)
+            if (!string.IsNullOrEmpty(Tags) && !renderingDetails.HasRenderedTags)
             {
                 writer.WritePropertyName(TagsPropertyName, escape: false);
                 writer.WriteValue(Tags);

Original file line number	Diff line number	Diff line change
`@@ -349,7 +349,7 @@ internal void FormatLog<T>(`
`349`	`349`
`350`	`350`	`EnrichTagsStringWithGitMetadata();`
`351`	`351`
`352`		`- if (Tags is not null && !renderingDetails.HasRenderedTags)`
	`352`	`+ if (!string.IsNullOrEmpty(Tags) && !renderingDetails.HasRenderedTags)`
`353`	`353`	`{`
`354`	`354`	`writer.WritePropertyName(TagsPropertyName, escape: false);`
`355`	`355`	`writer.WriteValue(Tags);`