config 3.2.1 → 3.2.5

package/History.md

@@ -1,3 +1,23 @@
+3.2.5 / 2020-01-16
+==================
+
+  * Fixed issue with getCustomEnvVars and multiple config dirs #585 - @dekelev
+
+3.2.4 / 2019-10-25
+==================
+
+  * Improved error handling of env variables value parse - @leonardovillela
+
+3.2.3 / 2019-10-03
+==================
+
+  * Fixed strict mode warning #460 - @fedulovivan
+
+3.2.2 / 2019-07-20
+==================
+
+  * Fixed delimiter bug in configDirs to match O/S delimiter - @iMoses
+
 3.2.1 / 2019-07-18
 ==================
 

package/README.md

@@ -126,7 +126,7 @@ Articles
 
 Further Information
 ---------------------
-If you still don't see what you are looking for, here more resources to check: 
+If you still don't see what you are looking for, here are some more resources to check: 
 
  * The [wiki may have more pages](https://github.com/lorenwest/node-config/wiki) which are not directly linked from here.
  * Review [questions tagged with node-config](https://stackexchange.com/filters/207096/node-config) on StackExchange. These are monitored by `node-config` contributors.
@@ -136,34 +136,34 @@ Contributors
 ------------
 <table id="contributors"><tr><td><img src=https://avatars2.githubusercontent.com/u/373538?v=4><a href="https://github.com/lorenwest">lorenwest</a></td>
 <td><img src=https://avatars1.githubusercontent.com/u/25829?v=4><a href="https://github.com/markstos">markstos</a></td>
+<td><img src=https://avatars1.githubusercontent.com/u/1083065?v=4><a href="https://github.com/iMoses">iMoses</a></td>
 <td><img src=https://avatars3.githubusercontent.com/u/447151?v=4><a href="https://github.com/elliotttf">elliotttf</a></td>
 <td><img src=https://avatars1.githubusercontent.com/u/8839447?v=4><a href="https://github.com/jfelege">jfelege</a></td>
 <td><img src=https://avatars0.githubusercontent.com/u/66902?v=4><a href="https://github.com/leachiM2k">leachiM2k</a></td>
-<td><img src=https://avatars1.githubusercontent.com/u/791137?v=4><a href="https://github.com/josx">josx</a></td>
-</tr><tr><td><img src=https://avatars2.githubusercontent.com/u/133277?v=4><a href="https://github.com/enyo">enyo</a></td>
+</tr><tr><td><img src=https://avatars1.githubusercontent.com/u/791137?v=4><a href="https://github.com/josx">josx</a></td>
+<td><img src=https://avatars2.githubusercontent.com/u/133277?v=4><a href="https://github.com/enyo">enyo</a></td>
 <td><img src=https://avatars1.githubusercontent.com/u/4307697?v=4><a href="https://github.com/leosuncin">leosuncin</a></td>
 <td><img src=https://avatars3.githubusercontent.com/u/1077378?v=4><a href="https://github.com/arthanzel">arthanzel</a></td>
 <td><img src=https://avatars2.githubusercontent.com/u/1656140?v=4><a href="https://github.com/eheikes">eheikes</a></td>
 <td><img src=https://avatars3.githubusercontent.com/u/138707?v=4><a href="https://github.com/th507">th507</a></td>
-<td><img src=https://avatars2.githubusercontent.com/u/506460?v=4><a href="https://github.com/Osterjour">Osterjour</a></td>
-</tr><tr><td><img src=https://avatars0.githubusercontent.com/u/842998?v=4><a href="https://github.com/nsabovic">nsabovic</a></td>
+</tr><tr><td><img src=https://avatars2.githubusercontent.com/u/506460?v=4><a href="https://github.com/Osterjour">Osterjour</a></td>
+<td><img src=https://avatars0.githubusercontent.com/u/842998?v=4><a href="https://github.com/nsabovic">nsabovic</a></td>
 <td><img src=https://avatars0.githubusercontent.com/u/5138570?v=4><a href="https://github.com/BadgerBadgerBadgerBadger">BadgerBadgerBadgerBadger</a></td>
 <td><img src=https://avatars2.githubusercontent.com/u/2529835?v=4><a href="https://github.com/simon-scherzinger">simon-scherzinger</a></td>
+<td><img src=https://avatars3.githubusercontent.com/u/8650543?v=4><a href="https://github.com/leonardovillela">leonardovillela</a></td>
 <td><img src=https://avatars1.githubusercontent.com/u/175627?v=4><a href="https://github.com/axelhzf">axelhzf</a></td>
-<td><img src=https://avatars3.githubusercontent.com/u/7782055?v=4><a href="https://github.com/benkroeger">benkroeger</a></td>
+</tr><tr><td><img src=https://avatars3.githubusercontent.com/u/7782055?v=4><a href="https://github.com/benkroeger">benkroeger</a></td>
 <td><img src=https://avatars3.githubusercontent.com/u/1872824?v=4><a href="https://github.com/fgheorghe">fgheorghe</a></td>
-</tr><tr><td><img src=https://avatars3.githubusercontent.com/u/1443067?v=4><a href="https://github.com/IvanVergiliev">IvanVergiliev</a></td>
+<td><img src=https://avatars3.githubusercontent.com/u/1443067?v=4><a href="https://github.com/IvanVergiliev">IvanVergiliev</a></td>
 <td><img src=https://avatars0.githubusercontent.com/u/1736957?v=4><a href="https://github.com/jpwilliams">jpwilliams</a></td>
 <td><img src=https://avatars2.githubusercontent.com/u/1246875?v=4><a href="https://github.com/jaylynch">jaylynch</a></td>
 <td><img src=https://avatars1.githubusercontent.com/u/145742?v=4><a href="https://github.com/jberrisch">jberrisch</a></td>
-<td><img src=https://avatars1.githubusercontent.com/u/9355665?v=4><a href="https://github.com/kgoerlitz">kgoerlitz</a></td>
+</tr><tr><td><img src=https://avatars1.githubusercontent.com/u/9355665?v=4><a href="https://github.com/kgoerlitz">kgoerlitz</a></td>
 <td><img src=https://avatars0.githubusercontent.com/u/8525267?v=4><a href="https://github.com/bertho-zero">bertho-zero</a></td>
-</tr><tr><td><img src=https://avatars3.githubusercontent.com/u/8650543?v=4><a href="https://github.com/leonardovillela">leonardovillela</a></td>
 <td><img src=https://avatars3.githubusercontent.com/u/1918551?v=4><a href="https://github.com/nitzan-shaked">nitzan-shaked</a></td>
 <td><img src=https://avatars3.githubusercontent.com/u/3058150?v=4><a href="https://github.com/robertrossmann">robertrossmann</a></td>
 <td><img src=https://avatars2.githubusercontent.com/u/498929?v=4><a href="https://github.com/roncli">roncli</a></td>
 <td><img src=https://avatars2.githubusercontent.com/u/1355559?v=4><a href="https://github.com/superoven">superoven</a></td>
-<td><img src=https://avatars2.githubusercontent.com/u/54934?v=4><a href="https://github.com/wmertens">wmertens</a></td>
 </tr></table>
 
 License

package/lib/config.js

@@ -666,7 +666,7 @@ util.loadFileConfigs = function(configDir) {
  * @returns {string[]}          fullFilenames - path + filename
  */
 util.locateMatchingFiles = function(configDirs, allowedFiles) {
-  return configDirs.split(':')
+  return configDirs.split(Path.delimiter)
     .reduce(function(files, configDir) {
       if (configDir) {
         try {
@@ -768,7 +768,8 @@ util.parseFile = function(fullFilename) {
   }
   catch (e2) {
     if (e2.code !== 'ENOENT') {
-      throw new Error('Config file ' + fullFilename + ' cannot be read');
+      throw new Error('Config file ' + fullFilename + ' cannot be read. Error code is: '+e2.code
+                        +'. Error message is: '+e2.message);
     }
     return null;  // file doesn't exists
   }
@@ -1038,7 +1039,12 @@ util.substituteDeep = function (substitutionMap, variables) {
       }
       else if (util.isObject(value)) { // work on the subtree, giving it a clone of the pathTo
         if('__name' in value && '__format' in value && vars[value.__name]) {
-          var parsedValue = util.parseString(vars[value.__name], value.__format);
+          try {
+            var parsedValue = util.parseString(vars[value.__name], value.__format);
+          } catch(err) {
+            err.message = '__format parser error in ' + value.__name + ': ' + err.message;
+            throw err;
+          }
           util.setPath(result, pathTo.concat(prop), parsedValue);
         } else {
           _substituteVars(value, vars, pathTo.concat(prop));
@@ -1067,8 +1073,13 @@ util.substituteDeep = function (substitutionMap, variables) {
  */
 util.getCustomEnvVars = function (CONFIG_DIR, extNames) {
   var result = {};
+  var resolutionIndex = 1;
+  var allowedFiles = {};
   extNames.forEach(function (extName) {
-    var fullFilename = Path.join(CONFIG_DIR , 'custom-environment-variables' + '.' + extName);
+    allowedFiles['custom-environment-variables' + '.' + extName] = resolutionIndex++;
+  });
+  var locatedFiles = util.locateMatchingFiles(CONFIG_DIR, allowedFiles);
+  locatedFiles.forEach(function (fullFilename) {
     var configObj = util.parseFile(fullFilename);
     if (configObj) {
       var environmentSubstitutions = util.substituteDeep(configObj, process.env);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "config",
-  "version": "3.2.1",
+  "version": "3.2.5",
   "main": "./lib/config.js",
   "description": "Configuration control for production node deployments",
   "author": "Loren West <open_source@lorenwest.com>",