Move auth to router.use

This commit is contained in:
Jori Lallo
2017-12-26 15:01:16 +02:00
parent 4406ec8e15
commit a74e90fc09
4 changed files with 13 additions and 8 deletions

View File

@ -8,8 +8,9 @@ import { presentApiKey } from '../presenters';
import { ApiKey } from '../models';
const router = new Router();
router.use(auth());
router.post('apiKeys.create', auth(), async ctx => {
router.post('apiKeys.create', async ctx => {
const { name } = ctx.body;
ctx.assertPresent(name, 'name is required');
@ -25,7 +26,7 @@ router.post('apiKeys.create', auth(), async ctx => {
};
});
router.post('apiKeys.list', auth(), pagination(), async ctx => {
router.post('apiKeys.list', pagination(), async ctx => {
const user = ctx.state.user;
const keys = await ApiKey.findAll({
where: {
@ -46,7 +47,7 @@ router.post('apiKeys.list', auth(), pagination(), async ctx => {
};
});
router.post('apiKeys.delete', auth(), async ctx => {
router.post('apiKeys.delete', async ctx => {
const { id } = ctx.body;
ctx.assertPresent(id, 'id is required');

View File

@ -12,6 +12,7 @@ import documents from './documents';
import views from './views';
import hooks from './hooks';
import apiKeys from './apiKeys';
import team from './team';
import validation from './middlewares/validation';
import methodOverride from '../middlewares/methodOverride';
@ -64,6 +65,7 @@ router.use('/', documents.routes());
router.use('/', views.routes());
router.use('/', hooks.routes());
router.use('/', apiKeys.routes());
router.use('/', team.routes());
// Router is embedded in a Koa application wrapper, because koa-router does not
// allow middleware to catch any routes which were not explicitly defined.

View File

@ -7,12 +7,13 @@ import auth from './middlewares/authentication';
import { presentUser } from '../presenters';
const router = new Router();
router.use(auth());
router.post('user.info', auth(), async ctx => {
router.post('user.info', async ctx => {
ctx.body = { data: await presentUser(ctx, ctx.state.user) };
});
router.post('user.update', auth(), async ctx => {
router.post('user.update', async ctx => {
const { user } = ctx.state;
const { name, avatarUrl } = ctx.body;
const endpoint = publicS3Endpoint();
@ -28,7 +29,7 @@ router.post('user.update', auth(), async ctx => {
ctx.body = { data: await presentUser(ctx, user) };
});
router.post('user.s3Upload', auth(), async ctx => {
router.post('user.s3Upload', async ctx => {
const { filename, kind, size } = ctx.body;
ctx.assertPresent(filename, 'filename is required');
ctx.assertPresent(kind, 'kind is required');

View File

@ -6,8 +6,9 @@ import { presentView } from '../presenters';
import { View, Document } from '../models';
const router = new Router();
router.use(auth());
router.post('views.list', auth(), async ctx => {
router.post('views.list', async ctx => {
const { id } = ctx.body;
ctx.assertPresent(id, 'id is required');
@ -36,7 +37,7 @@ router.post('views.list', auth(), async ctx => {
};
});
router.post('views.create', auth(), async ctx => {
router.post('views.create', async ctx => {
const { id } = ctx.body;
ctx.assertPresent(id, 'id is required');