Capture otel metrics for OAuth errors and add logging to stderr

ee/branding/api/index.ts

@@ -15,8 +15,8 @@ const handler = async (req: NextApiRequest, res: NextApiResponse) => {
         res.setHeader('Allow', 'GET');
         res.status(405).json({ error: { message: `Method ${method} Not Allowed` } });
     }
-  } catch (error: any) {
-    const { message, statusCode = 500 } = error;
+  } catch (err: any) {
+    const { message, statusCode = 500 } = err;
 
     res.status(statusCode).json({ error: { message } });
   }

ee/identity-federation/api/metadata.ts

@@ -32,8 +32,8 @@ const handleGET = async (req: NextApiRequest, res: NextApiResponse) => {
     }
 
     res.status(200).send(metadata.xml);
-  } catch (error: any) {
-    const { message, statusCode = 500 } = error;
+  } catch (err: any) {
+    const { message, statusCode = 500 } = err;
 
     return res.status(statusCode).json({
       error: { message },

ee/product/api/[productId].ts

@@ -15,8 +15,8 @@ const handler = async (req: NextApiRequest, res: NextApiResponse) => {
         res.setHeader('Allow', 'GET,DELETE');
         res.status(405).json({ error: { message: `Method ${req.method} Not Allowed` } });
     }
-  } catch (error: any) {
-    const { message, statusCode = 500 } = error;
+  } catch (err: any) {
+    const { message, statusCode = 500 } = err;
     res.status(statusCode).json({ error: { message } });
   }
 };

ee/product/api/index.ts

@@ -12,8 +12,8 @@ const handler = async (req: NextApiRequest, res: NextApiResponse) => {
         res.setHeader('Allow', 'POST');
         res.status(405).json({ error: { message: `Method ${req.method} Not Allowed` } });
     }
-  } catch (error: any) {
-    const { message, statusCode = 500 } = error;
+  } catch (err: any) {
+    const { message, statusCode = 500 } = err;
     res.status(statusCode).json({ error: { message } });
   }
 };

internal-ui/src/identity-federation/EditIdentityFederationApp.tsx

@@ -55,8 +55,8 @@ export const EditIdentityFederationApp = ({
       await fetch(urls.deleteApp, { method: 'DELETE', headers: defaultHeaders });
       setDelModalVisible(false);
       onDelete?.();
-    } catch (error: any) {
-      onError?.(error);
+    } catch (err: any) {
+      onError?.(err);
     }
   };
 

lib/api/default.ts

@@ -21,9 +21,9 @@ export const defaultHandler = async (req: NextApiRequest, res: NextApiResponse,
     // Call the handler
     await handler(req, res);
     return;
-  } catch (error: any) {
-    const message = error.message || 'Internal Server Error';
-    const status = error.statusCode || 500;
+  } catch (err: any) {
+    const message = err.message || 'Internal Server Error';
+    const status = err.statusCode || 500;
 
     console.error(`${req.method} ${req.url} - ${status} - ${message}`);
 

npm/src/controller/connection/saml.ts

@@ -32,8 +32,8 @@ async function fetchMetadata(resource: string) {
       timeout: 8000,
     });
     return response.data;
-  } catch (error: any) {
-    throw new JacksonError("Couldn't fetch XML data", error.response?.status || 400);
+  } catch (err: any) {
+    throw new JacksonError("Couldn't fetch XML data", err.response?.status || 400);
   }
 }