http-proxy-middleware 0.17.4 → 0.19.2

package/lib/path-rewriter.js

@@ -1,72 +1,79 @@
-var _      = require('lodash');
-var logger = require('./logger').getInstance();
+var _ = require('lodash')
+var logger = require('./logger').getInstance()
+var ERRORS = require('./errors')
 
 module.exports = {
-    create: createPathRewriter
-};
+  create: createPathRewriter
+}
 
 /**
  * Create rewrite function, to cache parsed rewrite rules.
  *
- * @returns {function} Function to rewrite paths; This function should accept `path` (request.url) as parameter
+ * @param {Object} rewriteConfig
+ * @return {Function} Function to rewrite paths; This function should accept `path` (request.url) as parameter
  */
 function createPathRewriter(rewriteConfig) {
-    var rulesCache;
+  var rulesCache
 
-    if (!isValidRewriteConfig(rewriteConfig)) {
-        return;
-    }
+  if (!isValidRewriteConfig(rewriteConfig)) {
+    return
+  }
 
-    if (_.isFunction(rewriteConfig)) {
-        var customRewriteFn = rewriteConfig;
-        return customRewriteFn;
-    } else {
-        rulesCache = parsePathRewriteRules(rewriteConfig);
-        return rewritePath;
-    }
+  if (_.isFunction(rewriteConfig)) {
+    var customRewriteFn = rewriteConfig
+    return customRewriteFn
+  } else {
+    rulesCache = parsePathRewriteRules(rewriteConfig)
+    return rewritePath
+  }
 
-    function rewritePath(path) {
-        var result = path;
+  function rewritePath(path) {
+    var result = path
 
-        _.forEach(rulesCache, function(rule) {
-            if (rule.regex.test(path)) {
-                result = result.replace(rule.regex, rule.value);
-                logger.debug('[HPM] Rewriting path from "%s" to "%s"', path, result);
-                return false;
-            }
-        });
+    _.forEach(rulesCache, function(rule) {
+      if (rule.regex.test(path)) {
+        result = result.replace(rule.regex, rule.value)
+        logger.debug('[HPM] Rewriting path from "%s" to "%s"', path, result)
+        return false
+      }
+    })
 
-        return result;
-    }
+    return result
+  }
 }
 
 function isValidRewriteConfig(rewriteConfig) {
-    if (_.isFunction(rewriteConfig)) {
-        return true;
-    } else if (!_.isEmpty(rewriteConfig) && _.isPlainObject(rewriteConfig)) {
-        return true;
-    } else if (_.isUndefined(rewriteConfig) ||
-               _.isNull(rewriteConfig) ||
-               _.isEqual(rewriteConfig, {})) {
-        return false;
-    } else {
-        throw new Error('[HPM] Invalid pathRewrite config. Expecting object with pathRewrite config or a rewrite function');
-    }
+  if (_.isFunction(rewriteConfig)) {
+    return true
+  } else if (!_.isEmpty(rewriteConfig) && _.isPlainObject(rewriteConfig)) {
+    return true
+  } else if (
+    _.isUndefined(rewriteConfig) ||
+    _.isNull(rewriteConfig) ||
+    _.isEqual(rewriteConfig, {})
+  ) {
+    return false
+  } else {
+    throw new Error(ERRORS.ERR_PATH_REWRITER_CONFIG)
+  }
 }
 
 function parsePathRewriteRules(rewriteConfig) {
-    var rules = [];
+  var rules = []
 
-    if (_.isPlainObject(rewriteConfig)) {
-        _.forIn(rewriteConfig, function(value, key) {
-            rules.push({
-                regex: new RegExp(key),
-                value: rewriteConfig[key]
-            });
-            logger.info('[HPM] Proxy rewrite rule created: "%s" ~> "%s"', key, rewriteConfig[key]);
-        });
-    }
+  if (_.isPlainObject(rewriteConfig)) {
+    _.forIn(rewriteConfig, function(value, key) {
+      rules.push({
+        regex: new RegExp(key),
+        value: rewriteConfig[key]
+      })
+      logger.info(
+        '[HPM] Proxy rewrite rule created: "%s" ~> "%s"',
+        key,
+        rewriteConfig[key]
+      )
+    })
+  }
 
-    return rules;
+  return rules
 }
-

package/lib/router.js

@@ -1,53 +1,51 @@
-var _      = require('lodash');
-var logger = require('./logger.js').getInstance();
+var _ = require('lodash')
+var logger = require('./logger.js').getInstance()
 
 module.exports = {
-    getTarget: getTarget
-};
+  getTarget: getTarget
+}
 
 function getTarget(req, config) {
-    var newTarget;
-    var router = config.router;
+  var newTarget
+  var router = config.router
 
-    if (_.isPlainObject(router)) {
-        newTarget = getTargetFromProxyTable(req, router);
-    } else if (_.isFunction(router)) {
-        newTarget = router(req);
-    }
+  if (_.isPlainObject(router)) {
+    newTarget = getTargetFromProxyTable(req, router)
+  } else if (_.isFunction(router)) {
+    newTarget = router(req)
+  }
 
-    return newTarget;
+  return newTarget
 }
 
 function getTargetFromProxyTable(req, table) {
-    var result;
-    var host = req.headers.host;
-    var path = req.url;
-
-    var hostAndPath = host + path;
-
-    _.forIn(table, function(value, key) {
-        if (containsPath(key)) {
-
-            if (hostAndPath.indexOf(key) > -1) {    // match 'localhost:3000/api'
-                result = table[key];
-                logger.debug('[HPM] Router table match: "%s"', key);
-                return false;
-            }
-        } else {
-
-            if (key === host) {                     // match 'localhost:3000'
-                result = table[key];
-                logger.debug('[HPM] Router table match: "%s"', host);
-                return false;
-            }
-
-        }
-
-    });
+  var result
+  var host = req.headers.host
+  var path = req.url
+
+  var hostAndPath = host + path
+
+  _.forIn(table, function(value, key) {
+    if (containsPath(key)) {
+      if (hostAndPath.indexOf(key) > -1) {
+        // match 'localhost:3000/api'
+        result = table[key]
+        logger.debug('[HPM] Router table match: "%s"', key)
+        return false
+      }
+    } else {
+      if (key === host) {
+        // match 'localhost:3000'
+        result = table[key]
+        logger.debug('[HPM] Router table match: "%s"', host)
+        return false
+      }
+    }
+  })
 
-    return result;
+  return result
 }
 
 function containsPath(v) {
-    return v.indexOf('/') > -1;
+  return v.indexOf('/') > -1
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "http-proxy-middleware",
-  "version": "0.17.4",
+  "version": "0.19.2",
   "description": "The one-liner node.js proxy middleware for connect, express and browser-sync",
   "main": "index.js",
   "files": [
@@ -9,6 +9,8 @@
   ],
   "scripts": {
     "clean": "rm -rf coverage",
+    "lint": "prettier \"**/*.{js,md}\" --list-different",
+    "lint:fix": "prettier \"**/*.{js,md}\" --write",
     "test": "mocha --recursive --colors --reporter spec",
     "cover": "npm run clean && istanbul cover ./node_modules/mocha/bin/_mocha -- --recursive",
     "coveralls": "istanbul cover ./node_modules/mocha/bin/_mocha --report lcovonly -- --recursive --reporter spec && istanbul-coveralls && npm run clean"
@@ -39,22 +41,41 @@
   },
   "homepage": "https://github.com/chimurai/http-proxy-middleware",
   "devDependencies": {
-    "browser-sync": "^2.18.2",
-    "chai": "^3.5.0",
-    "connect": "^3.5.0",
-    "coveralls": "^2.11.15",
-    "express": "^4.14.0",
+    "@commitlint/cli": "^7.2.1",
+    "@commitlint/config-conventional": "^7.1.2",
+    "browser-sync": "^2.26.3",
+    "chai": "^4.2.0",
+    "connect": "^3.6.6",
+    "coveralls": "^3.0.2",
+    "express": "^4.16.4",
+    "husky": "^1.2.0",
     "istanbul": "^0.4.5",
     "istanbul-coveralls": "^1.0.3",
-    "mocha": "^3.2.0",
-    "mocha-lcov-reporter": "1.2.0",
-    "opn": "^4.0.2",
-    "ws": "^1.1.1"
+    "mocha": "^5.2.0",
+    "mocha-lcov-reporter": "1.3.0",
+    "opn": "^5.4.0",
+    "precise-commits": "^1.0.2",
+    "prettier": "^1.15.2",
+    "ws": "^6.1.2"
   },
   "dependencies": {
-    "http-proxy": "^1.16.2",
-    "is-glob": "^3.1.0",
-    "lodash": "^4.17.2",
-    "micromatch": "^2.3.11"
+    "http-proxy": "^1.18.1",
+    "is-glob": "^4.0.0",
+    "lodash": "^4.17.11",
+    "micromatch": "^3.1.10"
+  },
+  "engines": {
+    "node": ">=4.0.0"
+  },
+  "husky": {
+    "hooks": {
+      "commit-msg": "commitlint -E HUSKY_GIT_PARAMS",
+      "pre-commit": "precise-commits"
+    }
+  },
+  "commitlint": {
+    "extends": [
+      "@commitlint/config-conventional"
+    ]
   }
 }