Skip to content
This repository was archived by the owner on Dec 17, 2024. It is now read-only.

Commit 3769788

Browse files
committed
Fixed Security Issue
1 parent d4474a6 commit 3769788

File tree

3 files changed

+13
-6
lines changed

3 files changed

+13
-6
lines changed

server/package.json

+4
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,18 @@
1414
"@mikro-orm/migrations": "^4.5.9",
1515
"@mikro-orm/postgresql": "^4.5.9",
1616
"argon2": "^0.28.2",
17+
"cookie-parser": "^1.4.5",
1718
"cors": "^2.8.5",
19+
"csurf": "^1.11.0",
1820
"express": "^4.17.1",
1921
"pg": "^8.7.1",
2022
"ts-node": "^10.3.0",
2123
"typescript": "^4.4.3"
2224
},
2325
"devDependencies": {
26+
"@types/cookie-parser": "^1.4.2",
2427
"@types/cors": "^2.8.12",
28+
"@types/csurf": "^1.11.2",
2529
"@types/dotenv": "^8.2.0",
2630
"@types/express": "^4.17.13",
2731
"@types/express-session": "^1.17.4",

server/src/index.ts

+2
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import { MyContext } from "./types";
77
import userRouter from "./routers/user";
88
import cors from "cors";
99
import session from "express-session";
10+
import cookieParser from "cookie-parser";
1011

1112
dotenv.config();
1213
const app = express();
@@ -27,6 +28,7 @@ const main = async () => {
2728
app.listen(PORT, () => console.log(`Alive on http://localhost:${PORT}`));
2829

2930
app.use(express.json());
31+
app.use(cookieParser());
3032
app.use(
3133
session({
3234
secret: process.env.COOKIE_SECRET ?? "",

server/src/routers/user.ts

+7-6
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,14 @@
11
import express from "express";
22
import { Delete, GetAll, GetOne, Login, SignUp, Me } from "../controllers/user";
3+
import csurf from "csurf";
34

45
const router = express.Router();
56

6-
router.get("/me", Me);
7-
router.get("/:id", GetOne);
8-
router.get("/", GetAll);
9-
router.post("/create", SignUp);
10-
router.post("/login", Login);
11-
router.get("/delete/:id", Delete);
7+
router.get("/me", csurf, Me);
8+
router.get("/:id", csurf, GetOne);
9+
router.get("/", csurf, GetAll);
10+
router.post("/create", csurf, SignUp);
11+
router.post("/login", csurf, Login);
12+
router.get("/delete/:id", csurf, Delete);
1213

1314
export default router;

0 commit comments

Comments
 (0)