From a738b51d87aaf7d915dd95205375f177b6fa9773 Mon Sep 17 00:00:00 2001 From: Tom Moor Date: Tue, 16 Aug 2022 19:49:15 +0200 Subject: [PATCH] chore: Add additional logging for unknown request errors --- app/utils/ApiClient.ts | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/app/utils/ApiClient.ts b/app/utils/ApiClient.ts index 6a98fba60..f5d9e8625 100644 --- a/app/utils/ApiClient.ts +++ b/app/utils/ApiClient.ts @@ -5,6 +5,7 @@ import queryString from "query-string"; import EDITOR_VERSION from "@shared/editor/version"; import stores from "~/stores"; import isCloudHosted from "~/utils/isCloudHosted"; +import Logger from "./Logger"; import download from "./download"; import { AuthorizationError, @@ -188,7 +189,14 @@ class ApiClient { ); } - throw new RequestError(`Error ${response.status}: ${error.message}`); + const err = new RequestError(`Error ${response.status}`); + Logger.error("Request failed", err, { + ...error, + url: urlToFetch, + }); + + // Still need to throw to trigger retry + throw err; }; get = (