feat: Cancel session --> cancel by user, not session id

CS3219-AY2425S1 · Oct 19, 2024 · a031863 · a031863
1 parent 33f813d
commit a031863
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/peerprep/backend/matching-service/src/controllers/runnerController.ts b/peerprep/backend/matching-service/src/controllers/runnerController.ts
@@ -155,10 +155,10 @@ export const cancelSession = async (
   res: Response
 ): Promise<void> => {
   try {
-    const { sessionId } = req.body;
+    const { userId } = req.body;
 
     // Cancel the timeout and remove session from Redis
-    await redis.delSession(sessionId);
+    await redis.delSessionByUser(userId);
 
     res.status(200).json({ message: "Session timeout cancelled." });
   } catch (error) {

diff --git a/peerprep/backend/matching-service/src/utils/redisUtils.ts b/peerprep/backend/matching-service/src/utils/redisUtils.ts
@@ -335,6 +335,14 @@ export const delSession = async (sessionId: string): Promise<void> => {
   } 
 };
 
+export const delSessionByUser = async (userId: string): Promise<void> => {
+  const session = await findSessionByUser(userId);
+  if (!session) {
+    throw new Error("User is not in a session");
+  }
+  await delSession(session.sessionId);
+}
+
 export const findSession = async (
   sessionId: string
 ): Promise<Session | null> => {