Skip to content
This repository has been archived by the owner on Jan 17, 2025. It is now read-only.

Commit

Permalink
Om loggmeldinga inneholder 'access to this (student/class)' skal det …
Browse files Browse the repository at this point in the history
…sendes warning i stedet for error, så ikke error-loggen får unødvendige oppføringer
  • Loading branch information
MatsAnd committed Feb 3, 2021
1 parent ca160b0 commit 20312de
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion Classes/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,8 @@ const handleClasses = async (context, req) => {
// No matching method found
throw new HTTPError(404, 'Method not found', { method, id, action })
} catch (error) {
logger('error', ['handle-classes', 'user', user, 'id', id, 'err', error.message])
const level = error.message && error.message.includes('access to this') ? 'warn' : 'error'
logger(level, ['handle-classes', 'user', user, 'id', id, 'err', error.message])

if (error instanceof HTTPError) return error.toJSON()
return new HTTPError(500, 'An unknown error occured', error).toJSON()
Expand Down
3 changes: 2 additions & 1 deletion Students/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,8 @@ const handleStudents = async (context, req) => {
// No matching method found
throw new HTTPError(404, 'Method not found', { method, id, action })
} catch (error) {
logger('error', ['handle-students', 'user', user, 'id', id, 'err', error.message])
const level = error.message && error.message.includes('access to this') ? 'warn' : 'error'
logger(level, ['handle-students', 'user', user, 'id', id, 'err', error.message])

if (error instanceof HTTPError) return error.toJSON()
return new HTTPError(500, 'An unknown error occured', error).toJSON()
Expand Down
3 changes: 2 additions & 1 deletion YFF/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,8 @@ const handleYFF = async (context, req) => {
logger('info', ['handle-yff', 'method', method, 'student', student, 'user', user, 'type', type, 'id', `${id || 'alle'}`, 'result', result.length])
return getResponse(result)
} catch (error) {
logger('error', ['handle-yff', 'method', method, 'student', student, 'user', user, 'id', `${id || 'alle'}`, 'err', error.message])
const level = error.message && error.message.includes('access to this') ? 'warn' : 'error'
logger(level, ['handle-yff', 'method', method, 'student', student, 'user', user, 'id', `${id || 'alle'}`, 'err', error.message])
if (error instanceof HTTPError) return error.toJSON()
return new HTTPError(500, 'An unknown error occured', error)
}
Expand Down

0 comments on commit 20312de

Please # to comment.