From 5de04dafc6c21cbfedc0f89859a21066e2741295 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Felix=20Geisend=C3=B6rfer?= <felix@debuggable.com>
Date: Sun, 20 Dec 2009 20:27:06 +0100
Subject: [PATCH] Do not pause request for multipart parsing

This is actually undesireable as it takes away control from the user who
may want to pause/resume to throttle the upload stream, or synchronize
it with disk flushing.

I actually ran into memory issues when trying to stream huge files to
disc as the file module was building up a huge action buffer. This can
now easily be avoided like this:

part.addListener('body', function(chunk) {
  req.pause();
  file.write(chunk).addCallback(function() {
    req.resume();
  });
}
---
 lib/multipart.js | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/lib/multipart.js b/lib/multipart.js
index c700788cdad..899b2d14997 100644
--- a/lib/multipart.js
+++ b/lib/multipart.js
@@ -53,11 +53,7 @@ proto.init = function(options) {
     var self = this;
     req
       .addListener('body', function(chunk) {
-        req.pause();
         self.write(chunk);
-        setTimeout(function() {
-          req.resume();
-        });
       })
       .addListener('complete', function() {
         self.emit('complete');