feat(errors): add status code to error message (#234)

Modern-Treasury · Sep 19, 2023 · 84481e1 · 84481e1
1 parent ac9b156
commit 84481e1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/core.ts b/src/core.ts
@@ -370,7 +370,7 @@ export abstract class APIClient {
         return this.retryRequest(options, retriesRemaining, responseHeaders);
       }
 
-      const errText = await response.text().catch(() => 'Unknown');
+      const errText = await response.text().catch((e) => castToError(e).message);
       const errJSON = safeJSON(errText);
       const errMessage = errJSON ? undefined : errText;
 

diff --git a/src/error.ts b/src/error.ts
@@ -13,7 +13,7 @@ export class APIError extends Error {
     message: string | undefined,
     headers: Headers | undefined,
   ) {
-    super(APIError.makeMessage(error, message));
+    super(`${status} ${APIError.makeMessage(error, message)}`);
     this.status = status;
     this.headers = headers;
     this.error = error;
@@ -25,7 +25,7 @@ export class APIError extends Error {
         typeof error.message === 'string' ? error.message
         : JSON.stringify(error.message)
       : error ? JSON.stringify(error)
-      : message || 'Unknown error occurred'
+      : message || 'status code (no body)'
     );
   }