Remove lodash.reduce dependency

helmetjs · Jul 20, 2018 · 6941b55 · 6941b55
1 parent 7c89159
commit 6941b55
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 15 deletions.
diff --git a/lib/parse-dynamic-directives.js b/lib/parse-dynamic-directives.js
@@ -1,8 +1,11 @@
-var reduce = require('lodash.reduce')
 var isFunction = require('./is-function')
 
 module.exports = function parseDynamicDirectives (directives, functionArgs) {
-  return reduce(directives, function (result, value, key) {
+  var result = {}
+
+  Object.keys(directives).forEach(function (key) {
+    var value = directives[key]
+
     if (Array.isArray(value)) {
       result[key] = value.map(function (element) {
         if (isFunction(element)) {
@@ -16,7 +19,7 @@ module.exports = function parseDynamicDirectives (directives, functionArgs) {
     } else if (value !== false) {
       result[key] = value
     }
+  })
 
-    return result
-  }, {})
+  return result
 }
diff --git a/lib/transform-directives-for-browser.js b/lib/transform-directives-for-browser.js
@@ -1,7 +1,9 @@
-var reduce = require('lodash.reduce')
-
 function createFirefoxPreCSP10Directives (directives, basePolicy) {
-  return reduce(directives, function (result, value, key) {
+  var result = Object.assign({}, basePolicy)
+
+  Object.keys(directives).forEach(function (key) {
+    var value = directives[key]
+
     if (key === 'connectSrc') {
       result.xhrSrc = value
     } else {
@@ -22,9 +24,9 @@ function createFirefoxPreCSP10Directives (directives, basePolicy) {
         result.options = optionsValues
       }
     }
+  })
 
-    return result
-  }, basePolicy)
+  return result
 }
 
 var handlers = {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -33,7 +33,6 @@
     "camelize": "1.0.0",
     "content-security-policy-builder": "2.0.0",
     "dasherize": "2.0.0",
-    "lodash.reduce": "4.6.0",
     "platform": "1.3.5"
   },
   "devDependencies": {