Add null check for UserName in logging context

Copilot · marcominerva · Copilot · commit 13f4b329da78 · 2025-10-10T12:15:40.000Z
Co-authored-by: marcominerva &lt;3522534+marcominerva@users.noreply.github.com&gt;
diff --git a/src/PdfSmith/Logging/HttpContextLogEventEnricher.cs b/src/PdfSmith/Logging/HttpContextLogEventEnricher.cs
@@ -14,7 +14,11 @@ public void Enrich(LogEvent logEvent, ILogEventPropertyFactory propertyFactory)
             return;
         }
 
-        logEvent.AddPropertyIfAbsent(propertyFactory.CreateProperty("UserName", httpContext.User?.Identity?.Name));
+        var userName = httpContext.User?.Identity?.Name;
+        if (!string.IsNullOrWhiteSpace(userName))
+        {
+            logEvent.AddPropertyIfAbsent(propertyFactory.CreateProperty("UserName", userName));
+        }
 
         if (httpContext.Request?.Headers?.TryGetValue("User-Agent", out var userAgent) == true)
         {

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,11 @@ public void Enrich(LogEvent logEvent, ILogEventPropertyFactory propertyFactory)`
`14`	`14`	`return;`
`15`	`15`	`}`
`16`	`16`
`17`		`- logEvent.AddPropertyIfAbsent(propertyFactory.CreateProperty("UserName", httpContext.User?.Identity?.Name));`
	`17`	`+ var userName = httpContext.User?.Identity?.Name;`
	`18`	`+ if (!string.IsNullOrWhiteSpace(userName))`
	`19`	`+ {`
	`20`	`+ logEvent.AddPropertyIfAbsent(propertyFactory.CreateProperty("UserName", userName));`
	`21`	`+ }`
`18`	`22`
`19`	`23`	`if (httpContext.Request?.Headers?.TryGetValue("User-Agent", out var userAgent) == true)`
`20`	`24`	`{`