fixed duped email and updated /search route

yale-swe · Oct 28, 2023 · 17d07f9 · 17d07f9
1 parent 56b747b
commit 17d07f9
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 8 deletions.
diff --git a/server/controllers/auth.js b/server/controllers/auth.js
@@ -18,7 +18,7 @@ export const register = async (req, res) => {
 			userName,
 			firstName,
 			lastName,
-			email,
+			email: email.toLowerCase(),
 			password: passwordHash,
 		});
 

diff --git a/server/controllers/user.js b/server/controllers/user.js
@@ -24,14 +24,13 @@ export const getUserById = async (req, res) => {
 
 export const getUserByEmailOrUsername = async (req, res) => {
 	try {
-		const { email, username } = req.body;
-
-		const conditions = [];
-		if (email) conditions.push({ email });
-		if (username) conditions.push({ userName: username });
+		const { email, userName } = req.body;
 
 		const user = await UserModel.findOne({
-			$or: conditions,
+			$or: [
+				{ email: { $regex: new RegExp(`^${email}$`, "i") } },
+				{ userName: userName },
+			],
 		});
 
 		if (!user) {

diff --git a/server/routes/user.js b/server/routes/user.js
@@ -13,7 +13,7 @@ import {
 
 const router = express.Router();
 
-router.post("/search", getUserByEmailOrUsername);
+router.get("/search", getUserByEmailOrUsername);
 router.get("/:userId", getUserById);
 
 router.get("/:userId/friends", getUserFriends);