Merge pull request #1687 from simov/fix-caseless-bug

Fix caseless bug - content-type not being set for multipart/form-data
request · Jul 21, 2015 · ff3a2cc · ff3a2cc
2 parents 33b15ee + 27722fe
commit ff3a2cc
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/request.js b/request.js
@@ -573,7 +573,7 @@ Request.prototype.init = function (options) {
 
     if (self._form && !self.hasHeader('content-length')) {
       // Before ending the request, we had to compute the length of the whole form, asyncly
-      self.setHeader(self._form.getHeaders())
+      self.setHeader(self._form.getHeaders(), true)
       self._form.getLength(function (err, length) {
         if (!err) {
           self.setHeader('content-length', length)

diff --git a/tests/test-form-data.js b/tests/test-form-data.js
@@ -29,6 +29,9 @@ function runTest(t, options) {
       }
     }
 
+    t.ok(/multipart\/form-data; boundary=--------------------------\d+/
+      .test(req.headers['content-type']))
+
     // temp workaround
     var data = ''
     req.setEncoding('utf8')

diff --git a/tests/test-form.js b/tests/test-form.js
@@ -21,6 +21,9 @@ tape('multipart form append', function(t) {
       return
     }
 
+    t.ok(/multipart\/form-data; boundary=--------------------------\d+/
+      .test(req.headers['content-type']))
+
     // temp workaround
     var data = ''
     req.setEncoding('utf8')