Skip to content

Commit

Permalink
test
Browse files Browse the repository at this point in the history
  • Loading branch information
rugglcon committed Aug 27, 2019
1 parent 7af8f0d commit 046f120
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 10 deletions.
8 changes: 4 additions & 4 deletions src/app.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ class App {
}

config(): void {
this.app.use(cors());
// this.app.use(cors());
this.app.options('*', cors());
this.app.use((_req, res, next) => {
res.header('Access-Control-Allow-Origin', '*');
Expand Down Expand Up @@ -96,23 +96,23 @@ class App {
/**
* BUDGET ROUTES
*/
this.app.use('/api/budgets', passport.authenticate('jwt', {
this.app.use('/api/budgets', cors(), passport.authenticate('jwt', {
session: true
}), budgetRoutes(cors, this.budgetLogic, this.expenseLogic));
logger.info('instantiated budget routes');

/**
* EXPENSE ROUTES
*/
this.app.use('/api/expense', passport.authenticate('jwt', {
this.app.use('/api/expense', cors(), passport.authenticate('jwt', {
session: true
}), expenseRoutes(cors, this.budgetLogic, this.expenseLogic));
logger.info('instantiated expense routes');

/**
* USER ROUTES
*/
this.app.use('/api/user', userRoutes(cors, passport, this.userLogic, this.authLogic));
this.app.use('/api/user', cors(), userRoutes(cors, passport, this.userLogic, this.authLogic));
logger.info('instantiated user routes');
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/routes/budget.routes.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ export const budgetRoutes = (appCors: (options?: cors.CorsOptions | cors.CorsOpt
budgetLogic: BudgetLogic, expenseLogic: ExpenseLogic): Router => {
const budgetsRouter = Router();

// budgetsRouter.all('*', appCors({ origin: true, credentials: true }));
budgetsRouter.options('*', appCors({ origin: true, credentials: true }));

// validates that id is a number
budgetsRouter.param('id', (_req, res, next, id) => {
Expand Down
8 changes: 4 additions & 4 deletions src/routes/expense.routes.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@ export const expenseRoutes = (appCors: (options?: cors.CorsOptions | cors.CorsOp
}
});

// expensesRouter.all('*', appCors({
// origin: true,
// credentials: true
// }));
expensesRouter.options('*', appCors({
origin: true,
credentials: true
}));

// creates an expense
expensesRouter.post('/', async (req, res) => {
Expand Down
2 changes: 1 addition & 1 deletion src/routes/user.routes.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ export const userRoutes = (appCors: (options?: cors.CorsOptions | cors.CorsOptio
}
});

// userRouter.all('*', appCors({ origin: true, credentials: true }));
userRouter.options('*', appCors({ origin: true, credentials: true }));

// logs a user in
userRouter.post(
Expand Down

0 comments on commit 046f120

Please sign in to comment.