diff options
author | eug-vs <eug-vs@keemail.me> | 2020-06-25 21:25:18 +0300 |
---|---|---|
committer | eug-vs <eug-vs@keemail.me> | 2020-06-25 21:25:18 +0300 |
commit | fd484a217c77dba42c29fae1cfdb2390422da847 (patch) | |
tree | b11c527bbc0a268f0361a4d56f6aff7011960f98 /hooks | |
parent | 050d3e087b3ce2c9dd892a96daf215402292aa2c (diff) | |
download | which-api-fd484a217c77dba42c29fae1cfdb2390422da847.tar.gz |
feat: add logger and handle eveyrthing nicely
Diffstat (limited to 'hooks')
-rw-r--r-- | hooks/handleErrors.ts | 11 | ||||
-rw-r--r-- | hooks/logging.ts | 4 | ||||
-rw-r--r-- | hooks/requireAuth.ts | 5 |
3 files changed, 17 insertions, 3 deletions
diff --git a/hooks/handleErrors.ts b/hooks/handleErrors.ts new file mode 100644 index 0000000..2a3c728 --- /dev/null +++ b/hooks/handleErrors.ts @@ -0,0 +1,11 @@ +import { HookContext } from '@feathersjs/feathers'; +import logger from '../logger'; + + +export default async (context: HookContext): Promise<HookContext> => { + context.result = context.error.message; + context.statusCode = context.error.code; + logger.error(context.error); + return context; +}; + diff --git a/hooks/logging.ts b/hooks/logging.ts index 840873c..8babe9a 100644 --- a/hooks/logging.ts +++ b/hooks/logging.ts @@ -1,13 +1,13 @@ import { HookContext } from '@feathersjs/feathers'; +import logger from '../logger'; export default async (context: HookContext): Promise<HookContext> => { if (context.params.provider) { const { method, path, id } = context; - const timestamp = new Date().toLocaleString('default', { timeStyle: 'medium', dateStyle: 'short' }); const message = `${method.toUpperCase()}: /${path}/${id || ''}`; const username = context.params.user?.username || 'anonymous'; - console.log(`[${timestamp}] ${message} ${username}`); + logger.log(`${message} ${username}`); } return context; }; diff --git a/hooks/requireAuth.ts b/hooks/requireAuth.ts index a7b0e96..52e8974 100644 --- a/hooks/requireAuth.ts +++ b/hooks/requireAuth.ts @@ -1,7 +1,10 @@ +import { NotAuthenticated } from '@feathersjs/errors'; import { HookContext } from '@feathersjs/feathers'; export default async (context: HookContext): Promise<HookContext> => { - if (!context.params.user) throw new Error('This endpoint requires auth!'); + if (!context.params.authenticated) { + throw new NotAuthenticated('This endpoint requires auth!'); + } return context; }; |