[CustomHandler]Fix DateTime serialization (#6707) (#6710)

Author: pragnagopa

release_notes.md

@@ -21,10 +21,12 @@
     }
     ```
     Exception is thrown if HttpOutputBindingResponse is not valid Json.
-
-    - Issue [#6606](https://github.com/Azure/azure-functions-host/issues/6606): We identified a couple of inconsistencies in the request schema and next release will include following changes.
-        - Query property will change from a JSON serialized string to a dictionary
-        - Identities property will change from a JSON serialized string to an array
+    - We identified a couple of inconsistencies in the request schema and next release will include following changes.
+        - Issue [#6606](https://github.com/Azure/azure-functions-host/issues/6606): 
+            - Query property will change from a JSON serialized string to a dictionary
+            - Identities property will change from a JSON serialized string to an array
+        - Issue [#6574](https://github.com/Azure/azure-functions-host/issues/6574)
+            - Metadata / Input binding data of type DateTime will not be serialzed as string
     If you are using these properties, please ensure your app is able to detect and handle the new schema.
 
 **Release sprint:** Sprint 84

src/WebJobs.Script/Workers/MessageExtensions/ScriptInvocationContextExtensions.cs

@@ -125,6 +125,14 @@ internal static object GetHttpScriptInvocationContextValue(object inputValue, Da
                 }
                 return Convert.ToBase64String(byteArray);
             }
+            if (inputValue is DateTime)
+            {
+                return DateTime.Parse(inputValue.ToString());
+            }
+            if (inputValue is DateTimeOffset)
+            {
+                return DateTimeOffset.Parse(inputValue.ToString());
+            }
             try
             {
                 return JObject.FromObject(inputValue);

test/WebJobs.Script.Tests/HttpWorker/DefaultHttpWorkerServiceTests.cs

@@ -513,7 +513,18 @@ private async void ValidateDefaultInvocationRequest(HttpRequestMessage httpReque
             Assert.Equal(expectedMetadata.Count(), httpScriptInvocationContext.Metadata.Count());
             foreach (var key in expectedMetadata.Keys)
             {
-                Assert.Equal(JsonConvert.SerializeObject(expectedMetadata[key]), httpScriptInvocationContext.Metadata[key]);
+                if (expectedMetadata[key] is DateTime)
+                {
+                    Assert.Equal(DateTime.Parse(expectedMetadata[key].ToString()), httpScriptInvocationContext.Metadata[key]);
+                }
+                else if (expectedMetadata[key] is DateTimeOffset)
+                {
+                    Assert.Equal(DateTimeOffset.Parse(expectedMetadata[key].ToString()), httpScriptInvocationContext.Metadata[key]);
+                }
+                else
+                {
+                    Assert.Equal(JsonConvert.SerializeObject(expectedMetadata[key]), httpScriptInvocationContext.Metadata[key]);
+                }
             }
 
             // Verify Data

test/WebJobs.Script.Tests/HttpWorker/HttpWorkerTestUtilities.cs

@@ -120,6 +120,7 @@ public static Dictionary<string, object> GetScriptInvocationBindingData()
             Dictionary<string, object> bindingData = new Dictionary<string, object>();
             bindingData["dequeueCount"] = 4;
             bindingData["VisibleTime"] = new DateTime(2019, 10, 1);
+            bindingData["VisibleTimeOffset"] = new DateTimeOffset(new DateTime(2019, 10, 1));
             bindingData["helloString"] = "helloMetadata";
             return bindingData;
         }