fix: use promise allSettled instead of all

adobe · Aug 23, 2024 · eade28a · eade28a
1 parent 7c4cd2f
commit eade28a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 22 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/structured-data/handler.js b/src/structured-data/handler.js
@@ -96,7 +96,10 @@ export async function processStructuredData(baseURL, context, pages) {
     }
   });
 
-  return Promise.all(urlInspectionResult);
+  const results = await Promise.allSettled(urlInspectionResult);
+  return results
+    .filter((result) => result.status === 'fulfilled')
+    .map((result) => result.value);
 }
 
 export async function structuredDataHandler(baseURL, context, site) {