Adds support for context arrays in #17

Author: drewzboto

Gruntfile.js

@@ -18,6 +18,7 @@ module.exports = function(grunt) {
       all: [
         'Gruntfile.js',
         'tasks/*.js',
+        'lib/*.js',
         '<%= nodeunit.tests %>',
       ],
       options: {
@@ -77,7 +78,11 @@ module.exports = function(grunt) {
             },
             {
               host: 'www.missingcontext.com',
-            }
+            },
+            {
+              context: ['/array1','/array2'],
+              host: 'www.defaults.com'
+            },
       ],
       server2: {
         proxies: [

README.md

@@ -116,9 +116,11 @@ For the server task, add the configureProxies task before the connect task
 The available configuration options from a given proxy are generally the same as what is provided by the underlying [httpproxy](https://github.com/nodejitsu/node-http-proxy) library
 
 #### options.context
-Type: `String`
+Type: `String` or `Array`
 
-The context to match requests against. Matching requests will be proxied. Should start with /. Should not end with /
+The context(s) to match requests against. Matching requests will be proxied. Should start with /. Should not end with /
+Multiple contexts can be matched for the same proxy rule via an array such as:
+context: ['/api', 'otherapi'] 
 
 #### options.host
 Type: `String`
@@ -236,4 +238,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
+* 0.1.6 Add xforward option, added support for context arrays, added debug logging

lib/utils.js

@@ -1,6 +1,7 @@
 'use strict';
 var utils = module.exports;
 var httpProxy = require('http-proxy');
+var _ = require('lodash');
 var proxies = [];
 var rewrite = function (req) {
     return function (rule) {
@@ -54,11 +55,25 @@ utils.processRewrites = function (rewrites) {
     return rules;
 };
 
+utils.matchContext = function(context, url) {
+    var contexts = context;
+    var matched = false;
+    if (!_.isArray(contexts)) {
+        contexts = [contexts];
+    }
+    contexts.forEach(function(testContext) {
+        if (url.lastIndexOf(testContext, 0) === 0) {
+            matched = true;
+        }
+    });
+    return matched;
+};
+
 utils.proxyRequest = function (req, res, next) {
     var proxied = false;
 
     proxies.forEach(function(proxy) {
-        if (!proxied && req && req.url.lastIndexOf(proxy.config.context, 0) === 0) {
+        if (!proxied && req && utils.matchContext(proxy.config.context, req.url)) {
             if (proxy.config.rules.length) {
                 proxy.config.rules.forEach(rewrite(req));
             }

package.json

@@ -1,7 +1,7 @@
 {
   "name": "grunt-connect-proxy",
   "description": "Provides a http proxy as middleware for grunt connect.",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "homepage": "https://github.com/drewzboto/grunt-connect-proxy",
   "author": {
     "name": "Drewz",
@@ -28,7 +28,8 @@
     "test": "grunt test"
   },
   "dependencies": {
-    "http-proxy": "~0.10.2"
+    "http-proxy": "~0.10.3",
+    "lodash": "~0.9.0"
   },
   "devDependencies": {
     "grunt-contrib-jshint": "~0.1.1",

tasks/connect_proxy.js

@@ -8,11 +8,11 @@
 
 'use strict';
 var utils = require('../lib/utils');
+var _ = require('lodash');
 
 module.exports = function(grunt) {
   grunt.registerTask('configureProxies', 'Configure any specified connect proxies.', function(config) {
     // setup proxy
-    var _ = grunt.util._;
     var httpProxy = require('http-proxy');
     var proxyOption;
     var proxyOptions = [];

test/connect_proxy_test.js

@@ -32,7 +32,7 @@ exports.connect_proxy = {
     test.expect(9);
     var proxies = utils.proxies();
 
-    test.equal(proxies.length, 4, 'should return four valid proxies');
+    test.equal(proxies.length, 5, 'should return five valid proxies');
     test.notEqual(proxies[0].server, null, 'server should be configured');
     test.equal(proxies[0].config.context, '/defaults', 'should have context set from config');
     test.equal(proxies[0].config.host, 'www.defaults.com', 'should have host set from config');
@@ -48,7 +48,7 @@ exports.connect_proxy = {
     test.expect(12);
     var proxies = utils.proxies();
 
-    test.equal(proxies.length, 4, 'should return four valid proxies');
+    test.equal(proxies.length, 5, 'should return five valid proxies');
     test.notEqual(proxies[1].server, null, 'server should be configured');
     test.equal(proxies[1].config.context, '/full', 'should have context set from config');
     test.equal(proxies[1].config.host, 'www.full.com', 'should have host set from config');
@@ -77,7 +77,7 @@ exports.connect_proxy = {
     test.expect(5);
     var proxies = utils.proxies();
 
-    test.equal(proxies.length, 4, 'should not add the 2 invalid proxies');
+    test.equal(proxies.length, 5, 'should not add the 2 invalid proxies');
     test.notEqual(proxies[0].config.context, '/missinghost', 'should not have context set from config with missing host');
     test.notEqual(proxies[0].config.host, 'www.missingcontext.com', 'should not have host set from config with missing context');
     test.notEqual(proxies[1].config.context, '/missinghost', 'should not have context set from config with missing host');
@@ -88,7 +88,7 @@ exports.connect_proxy = {
   invalid_rewrite: function(test) {
     test.expect(3);
     var proxies = utils.proxies();
-    test.equal(proxies.length, 4, 'proxies should still be valid');
+    test.equal(proxies.length, 5, 'proxies should still be valid');
     test.equal(proxies[3].config.rules.length, 1, 'rules array should have one valid item');
     test.deepEqual(proxies[3].config.rules[0], { from: new RegExp('^/in'), to: '/thisis'}, 'rules object should be converted to regex');
 

test/server2_proxy_test.js

@@ -9,12 +9,12 @@ exports.server2_proxy_test = {
     test.expect(10);
     var proxies = utils.proxies();
 
-    test.equal(proxies.length, 5, 'should return one valid proxy');
+    test.equal(proxies.length, 6, 'should return six valid proxies');
     test.notEqual(proxies[0].server, null, 'server should be configured');
     test.equal(proxies[0].config.context, '/defaults', 'should have context set from config');
     test.equal(proxies[0].config.host, 'www.defaults.com', 'should have host set from config');
-    test.equal(proxies[4].config.context, '/', 'should have context set from config');
-    test.equal(proxies[4].config.host, 'www.server2.com', 'should have host set from config');
+    test.equal(proxies[5].config.context, '/', 'should have context set from config');
+    test.equal(proxies[5].config.host, 'www.server2.com', 'should have host set from config');
     test.equal(proxies[0].config.port, 80, 'should have default port 80');
     test.equal(proxies[0].config.https, false, 'should have default http');
     test.equal(proxies[0].config.changeOrigin, false, 'should have default change origin');