review: wrap in try/catch

wazo-platform · Jun 4, 2024 · 25d565a · 25d565a
1 parent 79aacd4
commit 25d565a
Showing 1 changed file with 12 additions and 8 deletions.
diff --git a/src/simple/Auth.ts b/src/simple/Auth.ts
@@ -167,18 +167,22 @@ export class Auth {
   }
 
   async initiateIdpAuthentication(domain: string, redirectUrl: string): Promise<{ location: string, saml_session_id: string }> {
-    const response = await getApiClient().auth.initiateIdpAuthentication(domain, redirectUrl);
+    try {
+      const response = await getApiClient().auth.initiateIdpAuthentication(domain, redirectUrl);
+
+      if (!response.ok) {
+        if (response.status === 404) {
+          throw new NoSamlRouteError('No route found for saml sso');
+        }
 
-    if (!response.ok) {
-      if (response.status === 404) {
-        throw new NoSamlRouteError('No route found for saml sso');
+        const error = await response.json();
+        throw new Error(error.message || error.reason);
       }
 
-      const error = await response.json();
-      throw new Error(error.message || error.reason);
+      return response.json();
+    } catch (e: any) {
+      throw new Error(e.message);
     }
-
-    return response.json();
   }
 
   async logInViaRefreshToken(refreshToken: string): Promise<Session | null> {