Provides xforward setting to fix #24

Author: drewzboto

Gruntfile.js

@@ -50,6 +50,7 @@ module.exports = function(grunt) {
               https: true,
               rejectUnauthorized: true,
               changeOrigin: true,
+              xforward: true,
               rewrite: {
                 '^/full': '/anothercontext'
               }

README.md

@@ -37,7 +37,8 @@ grunt.initConfig({
                     host: '10.10.2.202',
                     port: 8080,
                     https: false,
-                    changeOrigin: false
+                    changeOrigin: false,
+                    xforward: false
                 }
             ]
         }
@@ -148,6 +149,16 @@ Default: false
 
 Whether to reject self-signed certificates when https: true is set. Defaults to accept self-signed certs since proxy is meant for development environments.
 
+#### options.xforward: 
+Type: `Boolean`
+Default: false
+
+Whether to add x-forward headers to the proxy request, such as
+  "x-forwarded-for": "127.0.0.1",
+  "x-forwarded-port": 50892,
+  "x-forwarded-proto": "http"
+
+
 #### options.appendProxies
 Type: `Boolean`
 Default: true
@@ -225,3 +236,4 @@ grunt.registerTask('e2etest', function (target) {
 * 0.1.3 Bumped http-proxy dependency to 0.10.2
 * 0.1.4 Added proxy rewrite support (thanks to @slawrence)
 * 0.1.5 Default rejectUnauthorized to false to allow self-signed certificates over SSL
+* 0.1.6 Add xforward option

lib/utils.js

@@ -23,7 +23,7 @@ utils.reset = function() {
 };
 
 utils.validateRewrite = function (rule) {
-    if (!rule || 
+    if (!rule ||
         typeof rule.from === 'undefined' ||
         typeof rule.to === 'undefined' ||
         typeof rule.from !== 'string' ||
@@ -68,7 +68,7 @@ utils.proxyRequest = function (req, res, next) {
 
             var source = req.originalUrl;
             var target = (proxy.server.target.https ? 'https://' : 'http://') + proxy.server.target.host + ':' + proxy.server.target.port + req.url;
-            utils.log.verbose.writeln('Proxied request: ' + source + ' -> ' + target);
+            utils.log.verbose.writeln('Proxied request: ' + source + ' -> ' + target + '\n' + JSON.stringify(req.headers, true, 2));
         }
     });
     if (!proxied) {

tasks/connect_proxy.js

@@ -43,6 +43,7 @@ module.exports = function(grunt) {
             port: 80,
             https: false,
             changeOrigin: false,
+            xforward: false,
             rejectUnauthorized: false,
             rules: []
         });
@@ -51,7 +52,10 @@ module.exports = function(grunt) {
             utils.registerProxy({
               server: new httpProxy.HttpProxy({
                 target: proxyOption,
-                changeOrigin: proxyOption.changeOrigin
+                changeOrigin: proxyOption.changeOrigin,
+                enable : {
+                    xforward: proxyOption.xforward // enables X-Forwarded-For
+                }
               }),
               config: proxyOption
             });

test/connect_proxy_test.js

@@ -29,7 +29,7 @@ exports.connect_proxy = {
     done();
   },
   default_options: function(test) {
-    test.expect(8);
+    test.expect(9);
     var proxies = utils.proxies();
 
     test.equal(proxies.length, 4, 'should return four valid proxies');
@@ -40,11 +40,12 @@ exports.connect_proxy = {
     test.equal(proxies[0].config.https, false, 'should have default http');
     test.equal(proxies[0].config.rejectUnauthorized, false, 'should have reject unauthorized default to false');
     test.equal(proxies[0].config.changeOrigin, false, 'should have default change origin');
+    test.equal(proxies[0].config.xforward, false, 'should have default xforward');
 
     test.done();
   },
   full_options: function(test) {
-    test.expect(11);
+    test.expect(12);
     var proxies = utils.proxies();
 
     test.equal(proxies.length, 4, 'should return four valid proxies');
@@ -55,6 +56,7 @@ exports.connect_proxy = {
     test.equal(proxies[1].config.https, true, 'should have http set from config');
     test.equal(proxies[1].config.rejectUnauthorized, true, 'should have reject unauthorized set from config');
     test.equal(proxies[1].config.changeOrigin, true, 'should have change origin set from config');
+    test.equal(proxies[1].config.xforward, true, 'should have xforward set from config');
     test.deepEqual(proxies[1].config.rewrite, { '^/full': '/anothercontext' }, 'should have rewrite set from config');
     test.equal(proxies[1].config.rules.length, 1, 'rules array should have an item');
     test.deepEqual(proxies[1].config.rules[0], { from: new RegExp('^/full'), to: '/anothercontext'}, 'rules object should be converted to regex');