-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Task/add access matches (about users, headers, path. .. ) to access Logger #544
Conversation
…ers related with proxys
Co-authored-by: Fermín Galán Márquez <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
' | SubService=' + req.subService + | ||
' | Action=' + req.action + | ||
' | Path=' + req.path + | ||
' | Query=' + JSON.stringify(req.query) + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En el fichero lib/middleware/proxy.js
se trunca el body a 100 caracteres ( ' | Body=' + JSON.stringify(req.body).slice(0, 100) + // not all body
), y aquí no, ¿cual es el motivo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hablado con Álvaro, por lo visto se loga el body entero cuando hay error, y solo una parte cuando no lo hay. NTC
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
El body se trunca a 100 caracteres en caso de acceso valido y se deja entero en caso de error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Faltaba documentarlo: #545
#541
Add to access logger info about:
Add Access match configuration in a file with a watchdog for updates
Check access match by