Remove quotes around fields in KubernetesEventGenerator (#9475)

* Remove quotes around columns

* Change NormalizeString signature

* Remove trailing whitespace

Author: AmeyaRele

src/WebJobs.Script.WebHost/Diagnostics/KubernetesEventGenerator.cs

@@ -33,12 +33,12 @@ public override void LogFunctionTraceEvent(LogLevel level, string subscriptionId
                 traceLog.Add("FunctionName", functionName);
                 traceLog.Add("EventName", eventName);
                 traceLog.Add("Source", source);
-                traceLog.Add("Details", NormalizeString(details));
-                traceLog.Add("Summary", NormalizeString(summary));
+                traceLog.Add("Details", NormalizeString(details, addEnclosingQuotes: false));
+                traceLog.Add("Summary", NormalizeString(summary, addEnclosingQuotes: false));
                 traceLog.Add("HostVersion", ScriptHost.Version);
                 traceLog.Add("EventTimeStamp", eventTimestamp.ToString(EventTimestampFormat));
                 traceLog.Add("ExceptionType", exceptionType);
-                traceLog.Add("ExceptionMessage", NormalizeString(exceptionMessage));
+                traceLog.Add("ExceptionMessage", NormalizeString(exceptionMessage, addEnclosingQuotes: false));
                 traceLog.Add("FunctionInvocationId", functionInvocationId);
                 traceLog.Add("HostInstanceId", hostInstanceId);
                 traceLog.Add("ActivityId", activityId);

src/WebJobs.Script.WebHost/Diagnostics/LinuxEventGenerator.cs

@@ -17,7 +17,7 @@ public abstract class LinuxEventGenerator : IEventGenerator
         public static readonly string FunctionsDetailsCategory = "functionsdetails";
         public static readonly string FunctionsExecutionEventsCategory = "functionexecutionevents";
 
-        internal static string NormalizeString(string value)
+        internal static string NormalizeString(string value, bool addEnclosingQuotes = true)
         {
             // Need to remove newlines for csv output
             value = value.Replace(Environment.NewLine, " ");
@@ -34,7 +34,7 @@ internal static string NormalizeString(string value)
             // our delimiter ',', before writing the value we
             // enclose in quotes. This allows us to define matching
             // groups based on quotes for these values.
-            return $"\"{value}\"";
+            return addEnclosingQuotes ? $"\"{value}\"" : value;
         }
 
         /// <summary>

test/WebJobs.Script.Tests/Diagnostics/KubernetesEventGeneratorTests.cs

@@ -47,12 +47,12 @@ public void ParseLogEvents(LogLevel level, string subscriptionId, string appName
                 p => Assert.Equal(functionName, p.Value),
                 p => Assert.Equal(eventName, p.Value),
                 p => Assert.Equal(source, p.Value),
-                p => Assert.Equal(LinuxEventGenerator.NormalizeString(details), p.Value.ToString()),
-                p => Assert.Equal(LinuxEventGenerator.NormalizeString(summary), p.Value.ToString()),
+                p => Assert.Equal(LinuxEventGenerator.NormalizeString(details, addEnclosingQuotes: false), p.Value.ToString()),
+                p => Assert.Equal(LinuxEventGenerator.NormalizeString(summary, addEnclosingQuotes: false), p.Value.ToString()),
                 p => Assert.Equal(ScriptHost.Version, p.Value),
                 p => Assert.True(DateTime.TryParse(p.Value.ToString(), out dt)),
                 p => Assert.Equal(exceptionType, p.Value.ToString()),
-                p => Assert.Equal(LinuxEventGenerator.NormalizeString(exceptionMessage), p.Value.ToString()),
+                p => Assert.Equal(LinuxEventGenerator.NormalizeString(exceptionMessage, addEnclosingQuotes: false), p.Value.ToString()),
                 p => Assert.Equal(functionInvocationId, p.Value.ToString()),
                 p => Assert.Equal(hostInstanceId, p.Value.ToString()),
                 p => Assert.Equal(activityId, p.Value),