Fixed bug in error handler, finished implementing update user endpoint
This commit is contained in:
1 parent
734efb1e29
commit
78abdd7f70
3 files changed
+2
-3
No files matched your search
@@ -132,7 +132,6 @@ const userEditController = async (req, res, next) => {
|
||||
return;
|
||||
}
|
||||
|
||||
//Find and update user
|
||||
try {
|
||||
const updatedUser = await req.db.updateUser(req, res);
|
||||
res
|
||||
|
||||
@@ -69,7 +69,7 @@ const updateUser = async (req, res) => {
|
||||
const candidateUser = req.body;
|
||||
|
||||
try {
|
||||
const updatedUser = await user.findByIdAndUpdate(
|
||||
const updatedUser = await UserModel.findByIdAndUpdate(
|
||||
candidateUserId,
|
||||
candidateUser,
|
||||
{ new: true } // Returns updated user instead of pre-update user
|
||||
|
||||
@@ -3,7 +3,7 @@ const logger = require("../utils/logger");
|
||||
const handleErrors = (error, req, res, next) => {
|
||||
const status = error.status || 500;
|
||||
const message = error.message || "Something went wrong";
|
||||
const service = error.errorService || "Unknown service";
|
||||
const service = error.service || "Unknown service";
|
||||
|
||||
logger.error(error.message, { service: service });
|
||||
res.status(status).json({ success: false, msg: message });
|
||||
|
||||
Reference in new issue
Block a user