Merge pull request #1505 from simov/fix-multipart-boundary

Fix multipart boundary extraction regexp
request · Mar 27, 2015 · 5b03774 · 5b03774
2 parents 2d95c75 + fd545b3
commit 5b03774
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/multipart.js b/lib/multipart.js
@@ -56,7 +56,7 @@ Multipart.prototype.setHeaders = function (chunked) {
     self.request.setHeader('content-type', 'multipart/related; boundary=' + self.boundary)
   } else {
     if (header.indexOf('boundary') !== -1) {
-      self.boundary = header.replace(/.*boundary=([^\s;])+.*/, '$1')
+      self.boundary = header.replace(/.*boundary=([^\s;]+).*/, '$1')
     } else {
       self.request.setHeader('content-type', header + '; boundary=' + self.boundary)
     }

diff --git a/tests/test-multipart.js b/tests/test-multipart.js
@@ -59,8 +59,8 @@ function runTest(t, a) {
       // check for http://localhost:6767/file traces
       t.ok(data.indexOf('Photoshop ICC') !== -1)
 
-      if (a.header && a.header.indexOf('mixed') !== -1) {
-        t.ok(data.indexOf('boundary=XXX'))
+      if (a.header && a.header.indexOf('boundary=XXX') !== -1) {
+        t.ok(data.indexOf('--XXX') !== -1)
       }
 
       res.writeHead(200)