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

err can be a string #396

Closed
wants to merge 1 commit into from
Closed

err can be a string #396

wants to merge 1 commit into from

Conversation

gdw2
Copy link
Contributor

@gdw2 gdw2 commented May 20, 2014

See #204

@jcrugzz
Copy link
Contributor

jcrugzz commented May 20, 2014

@gdw2 could you show the case where this can be a string? this just seems horribly wrong if an error was somehow stringified.

@gdw2
Copy link
Contributor Author

gdw2 commented May 20, 2014

See the issue I linked (#204). I'll need to track down exactly what's producing it since it isn't outputting a stack trace.

@gdw2
Copy link
Contributor Author

gdw2 commented May 20, 2014

I looked into it a little more and it appears that err isn't a string, but instead is an array of two errors: [err, err]. It's bubbling up from the node.js:process._processFatal() method. Does this seem more acceptable, and if so, how do we handle it?

I'm still not sure from where it's originating. I have the debugger stopping at a breakpoint, but nothing traces back further than the _processFatal() method. I wonder if this is any indicator that its coming from native code?

@jcrugzz
Copy link
Contributor

jcrugzz commented May 21, 2014

@gdw2 😖 that is just ridiculous. Are you running on the latest node by any chance? This seems like a silly bug in node core or something of the sort. While we can add some crufty code to handle this case, this should just not happen and should be fixed at the root cause.

@indexzero
Copy link
Member

👍 this should go into the next [email protected] release.

indexzero added a commit that referenced this pull request Oct 6, 2014
@indexzero
Copy link
Member

Cherry-picked as 122fb75

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.

3 participants