comment out other defs of cors and configure it in app

rugglcon · Aug 23, 2019 · 80b3e52 · 80b3e52
1 parent 48796ea
commit 80b3e52
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/src/app.ts b/src/app.ts
@@ -28,7 +28,7 @@ class App {
     }
 
     config(): void {
-        this.app.use(cors());
+        this.app.use(cors({ origin: true, credentials: true }));
         this.app.use((_req, res, next) => {
             res.header('Access-Control-Allow-Origin', '*');
             res.header('Access-Control-Allow-Headers', 'Origin, X-Requested-With, Content-Type, Accept, Authorization');

diff --git a/src/routes/budget.routes.ts b/src/routes/budget.routes.ts
@@ -8,7 +8,7 @@ import { ExpenseLogic } from 'logic/expenses';
 export const budgetRoutes = (cors: () => RequestHandler, budgetLogic: BudgetLogic, expenseLogic: ExpenseLogic): Router => {
     const budgetsRouter = Router();
 
-    budgetsRouter.options('*', cors());
+    // budgetsRouter.all('*', cors());
 
     // validates that id is a number
     budgetsRouter.param('id', (_req, res, next, id) => {

diff --git a/src/routes/expense.routes.ts b/src/routes/expense.routes.ts
@@ -22,7 +22,7 @@ export const expenseRoutes = (cors: () => RequestHandler,
         }
     });
 
-    expensesRouter.options('*', cors());
+    // expensesRouter.all('*', cors());
 
     // creates an expense
     expensesRouter.post('/', async (req, res) => {

diff --git a/src/routes/user.routes.ts b/src/routes/user.routes.ts
@@ -22,7 +22,7 @@ export const userRoutes = (cors: () => RequestHandler, passport: PassportStatic,
         }
     });
 
-    userRouter.options('*', cors());
+    // userRouter.all('*', cors({ origin: true, credentials: true }));
 
     // logs a user in
     userRouter.post(