peafowl - npm Package Compare versions

Comparing version 0.1.0 to 0.2.0

lib/index.js

@@ -1,17 +0,14 @@
-'use strict';
+const map = new WeakMap();
 
-const fs = require('fs');
-
-let getEnvironment = () => {
-		return process.env.NODE_ENV || 'development';
-	},
-	settings;
-
 class Config {
 	constructor() {
-		settings = JSON.parse(fs.readFileSync('config/' + getEnvironment() + '.json', 'utf8'));
+		map.set(this, require(`${__dirname}/../config/${this.getEnvironment()}.json`));
 	}
 
+	getEnvironment() {
+		return process.env.NODE_ENV || 'development';
+	}
+
 	get(path) {
-		let obj = settings;
+		let obj = map.get(this);
 
@@ -23,3 +20,3 @@ path.split('/').forEach((key) => {
 
-			throw new Error('Trying to get "' + path + '" but key "' + key + '" was not found');
+			throw new Error(`Trying to get "${path}" but key "${key}" was not found`);
 		});
@@ -33,3 +30,3 @@
 			last = keys.pop(),
-			obj = settings;
+			obj = map.get(this);
 
@@ -36,0 +33,0 @@ keys.forEach((key) => {

package.json

 {
   "name": "peafowl",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "description": "",
@@ -13,5 +13,5 @@ "main": "lib/index.js",
     "code": "^4.0.0",
-    "lab": "^11.1.0",
+    "lab": "^14.3.1",
     "plato": "^1.7.0"
   }
 }

test/config.js

@@ -1,7 +0,6 @@
-'use strict';
-
-const Lab = require('lab'),
+const
+	Lab = require('lab'),
 	Code = require('code'),
 	lab = exports.lab = Lab.script(),
-	config = require('lib/index');
+	config = require(`${__dirname}/../lib/index`);
 
@@ -8,0 +7,0 @@ lab.experiment('Config', () => {

New alerts

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Fixed alerts

Filesystem access

Supply chain risk

Accesses the file system, and could potentially read sensitive data.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

4577

decreased by-0.89%

Lines of code

84

decreased by-2.33%

Number of low supply chain risk alerts

4

increased by33.33%

No dependency changes