Fix parsing of content-disposition header

Author: Matt Mason

src/WebJobs.Script/Binding/HttpBinding.cs

@@ -302,7 +302,11 @@ internal static void AddResponseHeader(HttpResponseMessage response, KeyValuePai
                         }
                         break;
                     case "content-disposition":
-                        response.Content.Headers.ContentDisposition = new ContentDispositionHeaderValue(header.Value.ToString());
+                        ContentDispositionHeaderValue contentDisposition = null;
+                        if (ContentDispositionHeaderValue.TryParse(header.Value.ToString(), out contentDisposition))
+                        {
+                            response.Content.Headers.ContentDisposition = contentDisposition;
+                        }
                         break;
                     case "content-encoding":
                     case "content-language":

test/WebJobs.Script.Tests/Binding/HttpBindingTests.cs

@@ -41,6 +41,19 @@ public void AddResponseHeader_ContentMD5_AddsExpectedHeader()
             Assert.Equal(base64, Convert.ToBase64String(response.Content.Headers.ContentMD5));
         }
 
+        [Fact]
+        public void AddResponseHeader_ContentDisposition_AddsExpectedHeader()
+        {
+            HttpResponseMessage response = new HttpResponseMessage()
+            {
+                Content = new StringContent("Test")
+            };
+            var cd = "attachment; filename=\"test.txt\"";
+            var header = new KeyValuePair<string, object>("content-disposition", cd);
+            HttpBinding.AddResponseHeader(response, header);
+            Assert.Equal(cd, response.Content.Headers.ContentDisposition.ToString());
+        }
+
         [Fact]
         public void ParseResponseObject_ReturnsExpectedResult()
         {