Ava: Display internal Node.js stacktrace if no user stacktrace is present

[szmarczak]

Current behavior

Expected behavior

{ Error: self signed certificate
    at TLSSocket.onConnectSecure (_tls_wrap.js:1177:34)
    at TLSSocket.emit (events.js:193:13)
    at TLSSocket._finishInit (_tls_wrap.js:668:8) code: 'DEPTH_ZERO_SELF_SIGNED_CERT' }

---

IssueHunt Summary

bunysae has been rewarded.

Backers (Total: $40.00)

• issuehunt ($40.00)

Submitted pull Requests

- #2420 Show node internal stack trace

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.
  

[sindresorhus]

Makes sense to me.

I also notice we don't show the .code property. We should show that if it's not empty, no matter what.

[szmarczak]

It should also display No user stacktrace present. IMHO

[novemberborn]

It's a little jarring when there is no stack trace. But, if the Node.js trace lines are useful here, why wouldn't they be useful in other situations?

Should we perhaps color code the trace lines but not filter any?

Agreed we should show all properties of the error.

[sindresorhus]

It's the difference between the error originating from your own code or from Node.js itself. If it originates from your own code, you don't really care or want the Node.js code frames, but if the error originates from Node.js, you definitely want them for context or being able to report it to the Node.js team.

[issuehunt-app[bot]]

@issuehunt has funded $40.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[issuehunt-app[bot]]

@novemberborn has rewarded $36.00 to @bunysae. See it on IssueHunt

• :moneybag: Total deposit: $40.00
• :tada: Repository reward(0%): $0.00
• :wrench: Service fee(10%): $4.00