-
Notifications
You must be signed in to change notification settings - Fork 885
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
Can't log to stderr in tricky case #85
Comments
mcollina
added a commit
that referenced
this issue
Sep 11, 2016
Very nice catch |
This was referenced Sep 28, 2016
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Hello Matteo,
While playing with pino, I noticed a subtle bug, when stderr is redirected from bash like this:
And index.js looks like
It seems related to the property
_writableState
which isundefined
in this case.So the condition
opts && opts._writableState
to assign opts to the stream, can't pass.Note that it just happens when options are omitted!
Cheers,
Benoit
The text was updated successfully, but these errors were encountered: