Skip to content

Commit 497464c

Browse files
authored
Merge pull request #63 from ik079/feature/cart-routes
Migrate routes to use authenticated user
2 parents d8dd685 + 0c51e75 commit 497464c

1 file changed

Lines changed: 6 additions & 13 deletions

File tree

src/routes/cart.routes.js

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,13 @@
11
import express from 'express';
22
import { getCart, addToCart, updateCartItem, removeFromCart } from '../controllers/cart.controller.js';
3+
import { authenticateToken } from '../middleware/auth.middleware.js';
34

45
const router = express.Router();
56

6-
// GET /api/cart/:userId
7-
router.get('/:userId', getCart);
8-
9-
// POST /api/cart/:userId
10-
router.post('/:userId', addToCart);
11-
12-
// PUT /api/cart/:userId
13-
router.put('/:userId', updateCartItem);
14-
15-
// DELETE /api/cart/:userId/:productId
16-
router.delete('/:userId/:productId', removeFromCart);
7+
// Authenticated routes - use req.user.id instead of path param
8+
router.get('/', authenticateToken, getCart);
9+
router.post('/', authenticateToken, addToCart);
10+
router.put('/', authenticateToken, updateCartItem);
11+
router.delete('/:productId', authenticateToken, removeFromCart);
1712

1813
export default router;
19-
20-

0 commit comments

Comments
 (0)