minimal change to perform additional redactions (#8232)

redwoodjs · May 5, 2023 · 02b34a4 · 02b34a4
1 parent 28bf392
commit 02b34a4
Showing 1 changed file with 27 additions and 0 deletions.
diff --git a/packages/telemetry/src/sendTelemetry.ts b/packages/telemetry/src/sendTelemetry.ts
@@ -21,21 +21,31 @@ interface SensitiveArgPositions {
     positions: Array<number>
     options?: never
     redactWith: Array<string>
+    allowOnly: Array<string>
   }
   g: {
     positions: Array<number>
     options?: never
     redactWith: Array<string>
+    allowOnly?: Array<string>
   }
   generate: {
     positions: Array<number>
     options?: never
     redactWith: Array<string>
+    allowOnly?: Array<string>
   }
   prisma: {
     positions?: never
     options: Array<string>
     redactWith: Array<string>
+    allowOnly?: Array<string>
+  }
+  lint: {
+    positions?: Array<number>
+    options?: never
+    redactWith: Array<string>
+    allowOnly: Array<string>
   }
 }
 
@@ -45,6 +55,7 @@ const SENSITIVE_ARG_POSITIONS: SensitiveArgPositions = {
   exec: {
     positions: [1],
     redactWith: ['[script]'],
+    allowOnly: ['exec'],
   },
   g: {
     positions: [2, 3],
@@ -58,6 +69,10 @@ const SENSITIVE_ARG_POSITIONS: SensitiveArgPositions = {
     options: ['--name'],
     redactWith: ['[name]'],
   },
+  lint: {
+    allowOnly: ['lint', '--fix'],
+    redactWith: ['[path]'],
+  },
 }
 
 interface Args {
@@ -149,6 +164,18 @@ export const sanitizeArgv = (
         }
       })
     }
+
+    // allow only clause
+    if (sensitiveCommand.allowOnly) {
+      args.forEach((arg: string, index: number) => {
+        if (
+          !sensitiveCommand.allowOnly?.includes(arg) &&
+          !sensitiveCommand.redactWith.includes(arg)
+        ) {
+          args[index] = sensitiveCommand.redactWith[0]
+        }
+      })
+    }
   }
 
   return args.join(' ')