fix(api.js): fix http.fetch throwing error if response body is empty,

closes #2831
tauri-apps · Dec 1, 2021 · ae2bca8 · ae2bca8
1 parent b4fcb36
commit ae2bca8
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/.changes/api-fetch-empty-reponse.md b/.changes/api-fetch-empty-reponse.md
@@ -0,0 +1,5 @@
+---
+"api": patch
+---
+
+Fix `http.fetch` throwing error if the repsone is successfult but the body is empty.
diff --git a/tooling/api/src/http.ts b/tooling/api/src/http.ts
@@ -203,7 +203,11 @@ class Client {
           // @ts-expect-error
           response.data = JSON.parse(response.data as string)
         } catch (e) {
-          if (response.ok) {
+          // @ts-expect-error
+          if (response.ok && (response.data as string) === '') {
+            // @ts-expect-error
+            response.data = {}
+          } else {
             throw Error(
               `Failed to parse response \`${response.data}\` as JSON: ${e};
               try setting the \`responseType\` option to \`ResponseType.Text\` or \`ResponseType.Binary\` if the API does not return a JSON response.`