error msg fix

gatewayservice/gateway-service.js

@@ -128,9 +128,7 @@ app.patch('/update-user', async (req, res) => {
     );
     res.json(userResponse.data);
   } catch (error) {
-    res
-      .status(error.response.status)
-      .json({ error: error.response.data.error });
+    res.status(error.response.status).json({ msg: error.response.data.msg });
   }
 });
 

middleware/auth-middleware.js

@@ -13,9 +13,6 @@ export const authenticateUser = (req, res, next) => {
     const { userId, role } = verifyJWT(token);
     req.user = { userId, role };
 
-    console.log('auth mw req.user');
-    console.log(req.user);
-
     next();
   } catch (error) {
     throw new UnauthenticatedError('Authentication invalid');

middleware/errorhandler-middleware.js

@@ -1,8 +1,6 @@
 import { StatusCodes } from 'http-status-codes';
 
 const errorHandlerMiddleware = (err, req, res, next) => {
-  console.log('error handler');
-  console.log(err);
   const statusCode = err.statusCode || StatusCodes.INTERNAL_SERVER_ERROR;
   const msg = err.message || 'something went wrong';
   res.status(statusCode).json({ msg });

middleware/validation-middleware.js

@@ -9,8 +9,6 @@ const withValidationErrors = (validateValues) => {
       const errors = validationResult(req);
       if (!errors.isEmpty()) {
         const errorMessages = errors.array().map((error) => error.msg);
-        console.log('validation mw');
-        console.log(errorMessages);
         throw new BadRequestError(errorMessages);
       }
       next();
@@ -83,8 +81,6 @@ export const validateUpdateUserInput = withValidationErrors([
     .withMessage('email must be no more than 20 characters long')
     .custom(async (email, { req }) => {
       const user = await User.findOne({ email });
-      console.log(user);
-      console.log(req.user);
       if (user && user._id.toString() !== req.user.userId) {
         throw new BadRequestError('email already exists');
       }

users/userservice/user-controller.js

@@ -55,12 +55,8 @@ export const getCurrentUserController = async (req, res) => {
 };
 
 export const updateUserController = async (req, res) => {
-  try {
-    const newUser = { ...req.body };
-    delete newUser.password;
-    await User.findByIdAndUpdate(req.user.userId, newUser);
-    res.status(StatusCodes.OK).json({ msg: 'update user' });
-  } catch (error) {
-    res.status(500).json({ error: error.message });
-  }
+  const newUser = { ...req.body };
+  delete newUser.password;
+  await User.findByIdAndUpdate(req.user.userId, newUser);
+  res.status(StatusCodes.OK).json({ msg: 'update user' });
 };

users/userservice/user-service.js

@@ -2,6 +2,7 @@
 import express from 'express';
 import mongoose from 'mongoose';
 import userRouter from './user-router.js';
+import errorHandlerMiddleware from './middleware/errorhandler-middleware.js';
 
 const app = express();
 const port = 8001;
@@ -14,6 +15,7 @@ mongoose.connect(mongoUri);
 
 // userRouter
 app.use('/', userRouter);
+app.use(errorHandlerMiddleware);
 
 const server = app.listen(port, () => {
   console.log(`User Service listening at http://localhost:${port}`);