381 (#386)

* [#381] Fixes proxyReq is empty ({}) in userResHeaderDecorator.

* Remove old comment.

Author: monkpow

app/steps/sendProxyRequest.js

@@ -10,7 +10,7 @@ function sendProxyRequest(Container) {
 
   return new Promise(function(resolve, reject) {
     var protocol = Container.proxy.requestModule;
-    var proxyReq = protocol.request(reqOpt, function(rsp) {
+    var proxyReq = Container.proxy.req = protocol.request(reqOpt, function(rsp) {
       if (options.stream) {
         Container.proxy.res = rsp;
         return resolve(Container);

index.js

@@ -30,11 +30,6 @@ module.exports = function proxy(host, userOptions) {
     debug('[start proxy] ' + req.path);
     var container = new ScopeContainer(req, res, next, host, userOptions);
 
-    // Skip proxy if filter is falsey.  Loose equality so filters can return
-    // false, null, undefined, etc.
-
-    //if (!container.options.filter(req, res)) { return next(); }
-
     filterUserRequest(container)
       .then(buildProxyReq)
       .then(resolveProxyHost)

lib/scopeContainer.js

@@ -18,8 +18,8 @@ function Container(req, res, next, host, userOptions) {
       next: next,
     },
     proxy: {
-      req: {},
-      res: {},
+      req: undefined,
+      res: undefined,
       resData: undefined, // from proxy res
       bodyContent: undefined, // for proxy req
       reqBuilder: {},  // reqOpt, intended as first arg to http(s)?.request