fix: make secure dependent on __prod__

thenicekat · Feb 26, 2024 · 9b1b44b · 9b1b44b
1 parent fb1842e
commit 9b1b44b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/backend/src/index.ts b/backend/src/index.ts
@@ -3,7 +3,7 @@ import 'dotenv/config'
 import express, { Express } from 'express'
 import session from 'express-session'
 const cors = require('cors');
-import { PORT, SESSIONKEY } from './constants'
+import { PORT, SESSIONKEY, __prod__ } from './constants'
 // Middleware
 import { loggerMiddleware } from './middleware/logger.middleware'
 import { errorsMiddleware } from './middleware/error.middleware'
@@ -37,7 +37,7 @@ app.use(session({
     resave: false,
     saveUninitialized: true,
     cookie: {
-        secure: true,
+        secure: __prod__,
         maxAge: 1000 * 60 * 60 * 24 * 1 // 1 day
     },
 }))

diff --git a/backend/src/routes/user.route.ts b/backend/src/routes/user.route.ts
@@ -11,7 +11,7 @@ userRouter.post("/signup", async (req, res) => {
     const userEmail = req.session.email as string;
 
     data["email"] = userEmail;
-    console.log(data)
+
     const createUser = await newUser(data);
 
     if (createUser.error) {
@@ -27,7 +27,7 @@ userRouter.post("/signup", async (req, res) => {
         message: "User created successfully",
         data: createUser.data
     }
-    res.redirect('/');
+
     return res.status(HttpCodes.CREATED).json(response);
 });