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

Syslog levels regression #249

Closed
courtneycouch opened this issue May 7, 2013 · 9 comments
Closed

Syslog levels regression #249

courtneycouch opened this issue May 7, 2013 · 9 comments

Comments

@courtneycouch
Copy link

The last commit on the syslog levels reversed the order which breaks how Winston limits log severity. Winston tracks more verbose as being a smaller value normally, and depends on this. Swapping so that debug is a higher value means that debug level is always logged and critical or errors are not logged if you were to set the log level to "info" for example.

651b13e#commitcomment-3163605

@evantahler
Copy link

+1

@jenshimmelreich
Copy link

The syslog-commit has the wrong semantic. It disabled our logging. Together with node 0.10-issues it was complicate to debug. Because of the wrong npm-levels (which would be corrected) I think many people use
syslog-levels.

@Icelle
Copy link

Icelle commented May 28, 2013

+1. This is very confusing now.

@brunomorency
Copy link

+1 on needing a way around this. Setting log level to 'info' usually means you want info, notice, warning ... Now Winston will give you info and debug but not the rest.

@airandfingers
Copy link

+1. Working around this by setting log level to 'emerg'.

@julien51
Copy link

@airandfingers By setting log level to emerg, you get all log messages to be sent to syslog, right?

@airandfingers
Copy link

@julien51 Correct.

@benfleis
Copy link

benfleis commented Mar 3, 2014

+1 on reversing the order.

I just wasted too much time tracking this down :/

@indexzero
Copy link
Member

This absolutely a regression. There is a larger discussion of how we will fix this in #406. Closing this as a duplicate.

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 a pull request may close this issue.

9 participants