CHE-41 Adjustment made to route to use req.query for the token string

Code-Hammers · Mar 26, 2024 · 3ca3d5d · 3ca3d5d
1 parent cf2d108
commit 3ca3d5d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/server/controllers/userController.ts b/server/controllers/userController.ts
@@ -13,7 +13,7 @@ const registerUser = async (
   next: NextFunction
 ) => {
   const { firstName, lastName, email, password } = req.body;
-  const { token } = req.params;
+  const { token } = req.query;
 
   try {
     const isValidEmail = email.match(/[\w\d\.]+@[a-z]+\.[\w]+$/gim);
@@ -27,6 +27,8 @@ const registerUser = async (
       tokenExpiry: { $gt: new Date() },
       isRegistered: false,
     });
+
+    //TODO Needs better error handling - this can trigger with situaions other than bad or missing token
     if (!invitation) {
       return res
         .status(400)

diff --git a/server/routes/userRoutes.ts b/server/routes/userRoutes.ts
@@ -9,7 +9,7 @@ import {
 const router = express.Router();
 
 router.post("/login", authUser);
-router.post("/register/:token", registerUser);
+router.post("/register", registerUser);
 router.delete("/:email", deleteUserByEmail);
 router.get("/:userId", getUserById);