Skip to content

Commit 4e9157c

Browse files
committed
Give more specific error message when it is either username or email duplicate
1 parent d1a575e commit 4e9157c

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

user-service/controller/user-controller.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,13 @@ export async function createUser(req, res) {
1616
try {
1717
const { username, email, password } = req.body;
1818
if (username && email && password) {
19-
const existingUser = await _findUserByUsernameOrEmail(username, email);
19+
let existingUser = await _findUserByUsername(username)
2020
if (existingUser) {
21-
return res.status(409).json({ message: "username or email already exists" });
21+
return res.status(409).json({ message: "Username already exists" });
22+
}
23+
existingUser = await _findUserByEmail(email);
24+
if (existingUser) {
25+
return res.status(409).json({ message: "Email already exists" });
2226
}
2327

2428
const salt = bcrypt.genSaltSync(10);

0 commit comments

Comments
 (0)