From f98f5a445b01837ce2bbe7b0a227d109275e49c7 Mon Sep 17 00:00:00 2001 From: ash1shkumar Date: Sat, 6 Jun 2026 00:16:18 +0530 Subject: [PATCH] refactor: centralize task route authentication --- backend/routes/tasks.routes.js | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/backend/routes/tasks.routes.js b/backend/routes/tasks.routes.js index 5e0c542..edce0d0 100644 --- a/backend/routes/tasks.routes.js +++ b/backend/routes/tasks.routes.js @@ -8,21 +8,22 @@ import { } from "../controllers/tasks.controller.js"; import { authenticateUser } from "../middleware/auth.middleware.js"; -import { validateTask } from "../middleware/validation.middleware.js"; const router = express.Router(); +const protectedRouter = express.Router(); router.get("/", getTasks); -// Centralized protection for all task mutation routes -router.use(authenticateUser); +/** +* Protected task mutation routes + */ +protectedRouter.use(authenticateUser); -router.post("/", createTask); +protectedRouter.post("/", createTask); +protectedRouter.patch("/:id", updateTaskStatus); +protectedRouter.patch("/:id/edit", updateTask); +protectedRouter.delete("/:id", deleteTask); -router.patch("/:id", updateTaskStatus); - -router.patch("/:id/edit", updateTask); - -router.delete("/:id", deleteTask); +router.use("/", protectedRouter); export default router; \ No newline at end of file