-
Notifications
You must be signed in to change notification settings - Fork 74
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
[feature] Added server side logging #239
Conversation
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.
I think by default we should do:
- log level:
debug
if development,warning
if production (as you did) but should be possible to override this in some way (env var? what's the best practice here?) - output: console by default, but should be possible to configure it (following the same method that we'l chose for log level) to a different transport
Can we log HTTP requests by default during dev?
Please also add information to the README on how to override logging configuration.
|
||
| Environment Variable | Detail | | ||
| -------------------- | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | ||
| **LOG_LEVEL** | (optional) This can be used to set the level of logging. The available values are `warn` and `debug`. By default log level is set to `debug` during development and `warn` during production. | |
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.
The available values are
warn
anddebug
.
What about info
and error
?
Please open the PR from a branch of this repository. |
Closes #82
Tasks done: