Check online status first

Closes #3037
pixiebrix · May 12, 2022 · 7257e52 · 7257e52
1 parent 5cf0203
commit 7257e52
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/utils/enrichAxiosErrors.ts b/src/utils/enrichAxiosErrors.ts
@@ -55,6 +55,10 @@ async function enrichBusinessRequestError(error: unknown): Promise<never> {
 
   console.trace("enrichBusinessRequestError", { error });
 
+  if (!navigator.onLine) {
+    throw new ClientNetworkError(NO_INTERNET_MESSAGE, { cause: error });
+  }
+
   // This should have already been called before attempting the request because Axios does not actually catch invalid URLs
   const url = assertHttpsUrl(error.config.url, error.config.baseURL);
 
@@ -71,10 +75,6 @@ async function enrichBusinessRequestError(error: unknown): Promise<never> {
     throw new RemoteServiceError(getErrorMessage(error), { cause: error });
   }
 
-  if (!navigator.onLine) {
-    throw new ClientNetworkError(NO_INTERNET_MESSAGE, { cause: error });
-  }
-
   const hasPermissions = await browser.permissions.contains({
     origins: [url.href],
   });