feat: adds additional manifest attempt logging

cujarrett · Mar 10, 2021 · 7c4e4e6 · 7c4e4e6
1 parent 04c9adb
commit 7c4e4e6
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 16 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/routes/info.js b/src/routes/info.js
@@ -46,14 +46,15 @@ module.exports = async (api) => {
       let manifestRetries = 0
       if (!isValidManifestData) {
         while (manifestRetries < maxRetries && !isValidManifestData) {
+          manifestRetries += 1
+          console.log({ manifestRetries })
           rawManifest = await fetch("https://www.bungie.net/Platform/Destiny2/Manifest/")
           manifestData = await rawManifest.json()
           isValidManifestData = manifestData.Message === "Ok"
-          manifestRetries += 1
         }
 
         if (manifestRetries === maxRetries && !isValidManifestData) {
-          throw new Error("Manifest failed")
+          return { metadata: { error: `The Bungie manifest failed to load ${maxRetries} times` } }
         }
       }
       // eslint-disable-next-line max-len
@@ -144,7 +145,7 @@ ${JSON.stringify(result, null, "  ")}`)
       return JSON.stringify(result, null, "  ")
     } catch (error) {
       console.log(error)
-      return { errorMessage: "An error has occurred" }
+      return { metadata: { error: "An error has occurred" } }
     }
   })
 }