wip

Author: silesky

.changeset/two-pillows-wonder.md

@@ -0,0 +1,5 @@
+---
+'@segment/analytics-signals': patch
+---
+
+Redact formData

packages/signals/signals/src/core/client/__tests__/redact.test.ts

@@ -79,11 +79,12 @@ describe(redactSignalData, () => {
   it('should redact the value in the "target" property if the type is "interaction"', () => {
     const signal = factories.createInteractionSignal({
       eventType: 'change',
-      target: { value: 'secret' },
+      target: { value: 'secret', formData: { password: '123' } },
     })
     const expected = factories.createInteractionSignal({
       eventType: 'change',
       target: { value: 'XXX' },
+      formData: { password: 'XXX' },
     })
     expect(redactSignalData(signal)).toEqual(expected)
   })

packages/signals/signals/src/core/client/redact.ts

@@ -3,8 +3,15 @@ import { Signal } from '@segment/analytics-signals-runtime'
 export const redactSignalData = (signalArg: Signal): Signal => {
   const signal = structuredClone(signalArg)
   if (signal.type === 'interaction') {
-    if ('target' in signal.data && 'value' in signal.data.target) {
-      signal.data.target.value = redactJsonValues(signal.data.target.value)
+    if ('target' in signal.data) {
+      if ('value' in signal.data.target) {
+        signal.data.target.value = redactJsonValues(signal.data.target.value)
+      }
+      if ('formData' in signal.data.target) {
+        signal.data.target.formData = redactJsonValues(
+          signal.data.target.formData
+        )
+      }
     }
   } else if (signal.type === 'network') {
     signal.data = redactJsonValues(signal.data, 2)

packages/signals/signals/src/core/signal-generators/dom-gen.ts

@@ -60,8 +60,8 @@ interface ParsedElementBase {
   nodeName: string
   tagName: string
   title: string
-  type: string
-  value: string
+  type?: string
+  value?: string
   textContent?: string
   innerText?: string
 }
@@ -88,6 +88,8 @@ interface ParsedMediaElement extends ParsedElementBase {
 
 interface ParsedHTMLFormElement extends ParsedElementBase {
   formData: Record<string, string>
+  innerText: never
+  textContent: never
 }
 
 type AnyParsedElement =
@@ -147,6 +149,8 @@ const parseElement = (el: HTMLElement): AnyParsedElement => {
   } else if (el instanceof HTMLFormElement) {
     return {
       ...base,
+      innerText: undefined,
+      textContent: undefined,
       formData: parseFormData(new FormData(el)),
     }
   }