Move #injectPasswordLeakDetection call down, add TODO

stevenwdv · Sep 1, 2022 · 8905263 · 8905263
1 parent 2558dd7
commit 8905263
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/FieldsCollector.ts b/src/FieldsCollector.ts
@@ -659,6 +659,7 @@ export class FieldsCollector extends BaseCollector {
 									: this.options.fill.email, fillTimes);
 						break;
 					case 'password':
+						await this.#injectPasswordLeakDetection(field.handle.executionContext().frame()!);
 						await fillPasswordField(field.handle, this.options.fill.password, fillTimes);
 						break;
 					default:
@@ -673,6 +674,7 @@ export class FieldsCollector extends BaseCollector {
 		}
 	}
 
+	//TODO? Maybe just check once after filling
 	async #injectPasswordLeakDetection(frame: Frame) {
 		try {
 			const page = frame.page();