Return calling first before await

Asked Feb 02 '23 at 14:24

Active Feb 02 '23 at 14:25

Viewed 27 times

const isLicensesValid = async (
  licenseList: Promise<string>[]
): Promise<string> => {
  let errorText = "";
  await licenseList.forEach(async (licenseFile) => {
    await licenseFile.then(async (licenseData: string) => {
      const licenseType = await Object.keys(
        JSON.parse(licenseData).Features
      )[0];
      if (!isLicenseValid(licenseData, licenseType.toLowerCase())) {
        errorText = "One or more invalid license file in zip";
      }
    });
  });
  return errorText;
};

not sure why errorText always returns empty. Await is not working in then function. Can someone please suggest what is the problem here

edited Feb 02 '23 at 14:25

Konrad

21,590
4
28
64

asked Feb 02 '23 at 14:24

raj m

1,863
6
21
37

2

An array's forEach method isn't promise-aware, and certainly doesn't _return_ a promise (so awaiting it is pointless). – jonrsharpe Feb 02 '23 at 14:25
`await Promise.all(licenseList.map(...` instead of `await licenseList.forEach` – Konrad Feb 02 '23 at 14:26
Could you please help me with full code – raj m Feb 02 '23 at 14:26
But I need to traverse the list, map would return some array know. – raj m Feb 02 '23 at 14:30
@rajm Just read the [documentation](//developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Promise/all). – Sebastian Simon Feb 02 '23 at 14:40

Return calling first before await

0 Answers0