fix: include content-type in error message for non-JSON responses in fetchExternal

Pavan-Microsoft · Pavan-Microsoft · commit 5756ebde9bb8 · 2026-04-23T15:31:09.000+05:30
diff --git a/src/App/src/utils/httpClient.ts b/src/App/src/utils/httpClient.ts
@@ -70,7 +70,7 @@ class HttpClient {
     }
     const contentType = response.headers.get('content-type') || '';
     if (!contentType.toLowerCase().includes('application/json')) {
-      throw new Error(`${config.method || 'GET'} ${url} returned non-JSON response`);
+      throw new Error(`${config.method || 'GET'} ${url} returned non-JSON response (content-type: ${contentType || 'unknown'})`);
     }
     return response.json();
   }

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ class HttpClient {`
`70`	`70`	`}`
`71`	`71`	`const contentType = response.headers.get('content-type') \|\| '';`
`72`	`72`	`if (!contentType.toLowerCase().includes('application/json')) {`
`73`		- throw new Error(`${config.method \|\| 'GET'} ${url} returned non-JSON response`);
	`73`	+ throw new Error(`${config.method \|\| 'GET'} ${url} returned non-JSON response (content-type: ${contentType \|\| 'unknown'})`);
`74`	`74`	`}`
`75`	`75`	`return response.json();`
`76`	`76`	`}`