Simplify url generation

regalijan · Oct 19, 2023 · 25a8623 · 25a8623
1 parent 28ea352
commit 25a8623
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/functions/api/reports/submit.ts b/functions/api/reports/submit.ts
@@ -182,7 +182,7 @@ export async function onRequestPost(context: RequestContext) {
     );
   }
 
-  const uploadUrls = await Promise.allSettled(uploadUrlPromises);
+  const uploadUrlResults = await Promise.allSettled(uploadUrlPromises);
 
   const reportId = `${Date.now()}${context.request.headers.get(
     "cf-ray",
@@ -196,12 +196,15 @@ export async function onRequestPost(context: RequestContext) {
     { expirationTtl: 3600 },
   );
 
-  if (uploadUrls.find((uploadUrl) => uploadUrl.status === "rejected"))
+  if (uploadUrlResults.find((uploadUrl) => uploadUrl.status === "rejected"))
     return errorResponse("Failed to generate upload url", 500);
 
   const attachments: string[] = [];
+  const uploadUrls: string[] = [];
+
+  for (const urlResult of uploadUrlResults as PromiseFulfilledResult<string>[]) {
+    uploadUrls.push(urlResult.value);
 
-  for (const urlResult of uploadUrls) {
     let url = urlResult.toString().replace("t/", "");
     const extension = (url.split(".").at(-1) as string).toLowerCase();