fix(api): do not log password param (#610)

Fixes vatesfr/xo-web#2363
This commit is contained in:
Julien Fontanet 2017-10-16 10:06:24 +02:00 committed by GitHub
parent 38742d0668
commit 34ef2700f2

View File

@ -141,6 +141,16 @@ function resolveParams (method, params) {
})
}
// -------------------------------------------------------------------
const removeSensitiveParams = params =>
typeof params.password === 'string'
? {
...params,
password: '* obfuscated *'
}
: params
// ===================================================================
export default class Api {
@ -277,7 +287,7 @@ export default class Api {
const data = {
userId,
method: name,
params,
params: removeSensitiveParams(params),
duration: Date.now() - startTime,
error: serializeError(error)
}