Skip to content
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

Add meta options to logging and error logging. #91

Merged
merged 1 commit into from
Dec 2, 2015

Conversation

conradoqg
Copy link
Contributor

Organizes the Readme.md, puts logging before error logging and separate logging options from error logging options.

Add baseMeta and metaField to better control how meta is send to winston, allowing better logging features when using external logger as loggly.
Corrects minor typo in test-coverage script command at package.json.
Add tests for the metaField option.

…te logging options from error logging options.

Add baseMeta and metaField to better control how meta is send to winston, allowing better logging features when using external logger as loggly.
Corrects minor typo in test-coverage script command at package.json.
Add tests for the metaField option.
@floatingLomas
Copy link
Collaborator

So this allows you to define meta constants that get included with every log message, and to also allow moving the response data from the original meta to a specific property within the meta?

@conradoqg
Copy link
Contributor Author

Yes for both. Sorry for my English.

@thecatontheflat
Copy link

Thank you. This PR actually helped me to understand the configuration better

bithavoc added a commit that referenced this pull request Dec 2, 2015
Add meta options to logging and error logging.
@bithavoc bithavoc merged commit e49150d into bithavoc:master Dec 2, 2015
@bithavoc
Copy link
Owner

bithavoc commented Dec 2, 2015

Thank you @conradoqg

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants