fix: also handle 'secret' as sensible data

arlac77 · May 6, 2019 · ef1d452 · ef1d452
1 parent c05a9df
commit ef1d452
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/util.mjs b/src/util.mjs
@@ -46,7 +46,7 @@ export function merge(a, b) {
  */
 export function removeSensibleValues(
   object,
-  toBeRemoved = key => key.match(/pass|auth|key|user/)
+  toBeRemoved = key => key.match(/pass|auth|key|user|secret/)
 ) {
   if (
     object === undefined ||

diff --git a/tests/remove-sensible-test.mjs b/tests/remove-sensible-test.mjs
@@ -6,8 +6,9 @@ test("removeSensibleValues", t => {
     removeSensibleValues({
       a: { value: 7 },
       key: "secure",
-      password: "secure"
+      password: "secure",
+      secret: "secure"
     }),
-    { a: { value: 7 }, key: "...", password: "..." }
+    { a: { value: 7 }, key: "...", password: "...", secret: "..." }
   );
 });