Better handling of auth errors (#1879)

* Better handling of auth errors

* Remove newline

* Update app/frontend/src/authConfig.ts

Author: pamelafox

app/frontend/src/api/api.ts

@@ -30,9 +30,12 @@ export async function askApi(request: ChatAppRequest, idToken: string | undefine
         body: JSON.stringify(request)
     });
 
-    const parsedResponse: ChatAppResponseOrError = await response.json();
     if (response.status > 299 || !response.ok) {
-        throw Error(parsedResponse.error || "Unknown error");
+        throw Error(`Request failed with status ${response.status}`);
+    }
+    const parsedResponse: ChatAppResponseOrError = await response.json();
+    if (parsedResponse.error) {
+        throw Error(parsedResponse.error);
     }
 
     return parsedResponse as ChatAppResponse;

app/frontend/src/pages/chat/Chat.tsx

@@ -186,13 +186,16 @@ const Chat = () => {
             if (!response.body) {
                 throw Error("No response body");
             }
+            if (response.status > 299 || !response.ok) {
+                throw Error(`Request failed with status ${response.status}`);
+            }
             if (shouldStream) {
                 const parsedResponse: ChatAppResponse = await handleAsyncRequest(question, answers, response.body);
                 setAnswers([...answers, [question, parsedResponse]]);
             } else {
                 const parsedResponse: ChatAppResponseOrError = await response.json();
-                if (response.status > 299 || !response.ok) {
-                    throw Error(parsedResponse.error || "Unknown error");
+                if (parsedResponse.error) {
+                    throw Error(parsedResponse.error);
                 }
                 setAnswers([...answers, [question, parsedResponse as ChatAppResponse]]);
             }