fix: fix performance issue

nestjs · Feb 5, 2023 · 555a556 · 555a556
1 parent 43668f4
commit 555a556
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/packages/common/pipes/file/parse-file.pipe.ts b/packages/common/pipes/file/parse-file.pipe.ts
@@ -65,10 +65,15 @@ export class ParseFilePipe implements PipeTransform<any> {
   }
 
   protected async validate(file: any): Promise<any> {
-    for (const validator of this.validators) {
-      await this.validateOrThrow(file, validator);
-    }
+    const validationResults = await Promise.allSettled(
+      this.validators.map(validator => this.validateOrThrow(file, validator)),
+    );
+    const isRejected = (
+      input: PromiseSettledResult<unknown>,
+    ): input is PromiseRejectedResult => input.status === 'rejected';
 
+    const error = await validationResults.find(isRejected)?.reason;
+    if (error) throw error;
     return file;
   }