fix: remove raw error responses

Author: kitlord

server/controllers/aws.controller.js

@@ -149,7 +149,8 @@ export async function copyObjectInS3RequestHandler(req, res) {
     const newUrl = await copyObjectInS3(url, req.user.id);
     res.json({ url: newUrl });
   } catch (error) {
-    res.status(500).json({ error: error.message });
+    console.error('Error copying object in S3:', error.message);
+    res.status(500).json({ error: 'Internal server error' });
   }
 }
 

server/controllers/user.controller/apiKey.ts

@@ -77,10 +77,9 @@ export const createApiKey: RequestHandler<
     res.json({ apiKeys });
   } catch (err) {
     if (err instanceof Error) {
-      res.status(500).json({ error: err.message });
-    } else {
-      res.status(500).json({ error: 'Internal server error' });
+      console.error('Could not create API key:', err.message);
     }
+    res.status(500).json({ error: 'Internal server error' });
   }
 };
 
@@ -122,9 +121,8 @@ export const removeApiKey: RequestHandler<
     res.status(200).json({ apiKeys: user.apiKeys });
   } catch (err: unknown) {
     if (err instanceof Error) {
-      res.status(500).json({ error: err.message });
-    } else {
-      res.status(500).json({ error: 'Internal server error' });
+      console.error('Could not remove API key:', err.message);
     }
+    res.status(500).json({ error: 'Internal server error' });
   }
 };

server/controllers/user.controller/authManagement.ts

@@ -201,7 +201,8 @@ export const updateSettings: RequestHandler<
       await saveUser(res, user);
     }
   } catch (err) {
-    res.status(500).json({ error: err });
+    console.error('Could not save settings:', err);
+    res.status(500).json({ error: 'Internal server error' });
   }
 };
 

server/controllers/user.controller/helpers.ts

@@ -67,7 +67,8 @@ export async function saveUser(res: Response, user: UserDocument) {
     await user.save();
     res.json(userResponse(user));
   } catch (error) {
-    res.status(500).json({ error });
+    console.error('Could not save user:', error);
+    res.status(500).json({ error: 'Internal server error' });
   }
 }
 

server/controllers/user.controller/signup.ts

@@ -75,8 +75,8 @@ export const createUser: RequestHandler<
       }
     });
   } catch (err) {
-    console.error(err);
-    res.status(500).json({ error: err });
+    console.error('Could not create user:', err);
+    res.status(500).json({ error: 'Internal server error' });
   }
 };
 
@@ -158,7 +158,8 @@ export const emailVerificationInitiate: RequestHandler<
 
     res.json(userResponse(req.user!));
   } catch (err) {
-    res.status(500).json({ error: err });
+    console.error('Could not initiate email verification:', err);
+    res.status(500).json({ error: 'Internal server error' });
   }
 };
 

server/controllers/user.controller/userPreferences.ts

@@ -33,7 +33,8 @@ export const updatePreferences: RequestHandler<
     await user.save();
     res.json(user.preferences);
   } catch (err) {
-    res.status(500).json({ error: err });
+    console.error('Could not save preferences:', err);
+    res.status(500).json({ error: 'Internal server error' });
   }
 };
 
@@ -61,6 +62,7 @@ export const updateCookieConsent: RequestHandler<
     user.cookieConsent = cookieConsent;
     await saveUser(res, user);
   } catch (err) {
-    res.status(500).json({ error: err });
+    console.error('Could not save cookie consent:', err);
+    res.status(500).json({ error: 'Internal server error' });
   }
 };

server/server.js

@@ -168,7 +168,8 @@ app.get('/', (req, res) => {
 // Handle API errors
 app.use('/api', (error, req, res, next) => {
   if (error && error.code && !res.headersSent) {
-    res.status(error.code).json({ error: error.message });
+    console.error('API error:', error.message);
+    res.status(error.code).json({ error: 'Internal server error' });
     return;
   }