Added get message + get users endpoints

Author: Keagan Aromin

backend/controllers/message.controller.js

@@ -41,4 +41,26 @@ export const sendMessage = async (req, res) => {
 		console.log("Error in sendMessage controller: ", error.message);
 		res.status(500).json({ error: "Internal server error" });
 	}
-};
+};
+
+export const getMessages = async (req, res) => {
+    try {
+        const {id:userToChatId} = req.params.id; //get req.params.id and rename to userToChatId
+        const senderId = req.user._id;
+
+        //conversation document, has array of mesage references as an attribute, use mongoose populate method to get actual contents of each message
+        const conversation = await Conversation.findOne({
+            participants: {$all: [senderId, userToChatId]},
+        }).populate("messages");
+
+        //if not messages, successful request but returned array will be empty
+        if (!conversation) return res.status(200).json([]);
+
+        const messages = conversation.messages;
+        res.status(200).json(messages);
+
+    } catch (error) {
+        console.log("Error in getMessage controller: ", error.message);
+		res.status(500).json({ error: "Internal server error" });
+    }
+}

backend/controllers/user.controller.js

@@ -0,0 +1,14 @@
+import User from "../models/user.model.js"
+
+export const getUsersForSidebar = async (req, res) => {
+    try {
+        const loggedInUserId = req.user._id
+        //User.find finds ALL users rather than ONE | $ne = not equal | .select("-password") excludes password attribute from each user in json response
+        // we are finding any users besides the current one
+        const filteredUsers = await User.find({ _id: {$ne: loggedInUserId}})
+        res.status(200).json(filteredUsers);
+    } catch (error) {
+        console.error("Error in getUsersForSidebar: ", error.message)
+        res.status(500).json({error: "Internal server error"})
+    }
+}

backend/middleware/protectRoute.js

@@ -26,6 +26,7 @@ const protectRoute = async (req, res, next) => {
 		}
 
         //assign request user attribute to the user we decoded using mongoose model + jwt
+        //because of this line of code, we can use req.user._id instead of req.user.Id
 		req.user = user;
 
         //this says that once everything has run, exit this function and call the next one

backend/routes/message.routes.js

@@ -5,6 +5,7 @@ import protectRoute from "../middleware/protectRoute";
 const router = express.Router();
 
 //when we have a post request, we first run protectRoute until the next() call is reached, then we run sendMessage
+//protectRoute makes sure that only authenticated users can send a message
 router.post("/send/:id", protectRoute, sendMessage)
 
 export default router;

backend/routes/user.routes.js

@@ -0,0 +1,11 @@
+import express from "express";
+import protectRoute from "../middleware/protectRoute.js";
+import { getUsersForSidebar } from "../controllers/user.controller.js";
+
+const router = express.Router();
+
+//when we have a get request, we first run protectRoute until the next() call is reached, then we run getUsersForSidebar
+//protectRoute makes sure that only authenticated users can get users
+router.get("/", protectRoute, getUsersForSidebar)
+
+export default router;

backend/server.js

@@ -1,9 +1,11 @@
 import express from "express";
 import dotenv from "dotenv";
 import cookieParser from "cookie-parser";
+
 import authRoutes from "./routes/auth.routes.js";
 import messageRoutes from "./routes/auth.routes.js";
 import userRoutes from "./routes/auth.routes.js";
+
 import connectToMongoDB from "./db/connectToMongoDB.js";
 
 const app = express();
@@ -26,4 +28,6 @@ app.use("/api/users", userRoutes);
 app.listen(PORT, () => {
     connectToMongoDB();
     console.log(`Server Running on port ${PORT}`)
-});
+});
+
+//The reason we separate auth route + controller from user route + controller is that authentication is its own service & we want to keep our app structure clean