Merge pull request #23 from datapartyjs/nconf-config-defaults

fix issue #22

Author: sevenbitbyte

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@dataparty/api",
   "private": false,
-  "version": "1.2.0",
+  "version": "1.2.1",
   "main": "src/index.js",
   "browser": "src/index-browser.js",
   "files": [

src/config/nconf.js

@@ -20,7 +20,6 @@ class NconfConfig {
 
   constructor(defaults, whitelist){
     this.whitelist = whitelist || []
-    defaults = defaults || {}
     this.basePath = defaults.basePath || BASE_PATH
     this.defaults = defaults || {}
     this.defaults.logicalSeparator = '.'
@@ -54,6 +53,8 @@ class NconfConfig {
       whitelist: this.whitelist,
     })
 
+    nconf.defaults(this.defaults)
+
     nconf.load()
     logger(`config ready: ${this.basePath}/${file_name}`)
     this.started = true