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

TraceKit.computeStackTrace evaluates error to {} #313

Closed
Myztiq opened this issue Jan 30, 2015 · 6 comments
Closed

TraceKit.computeStackTrace evaluates error to {} #313

Myztiq opened this issue Jan 30, 2015 · 6 comments

Comments

@Myztiq
Copy link

Myztiq commented Jan 30, 2015

I have created a reproduction hosted here: http://raven-undefined.divshot.io/
With source here: https://github.com/Myztiq/raven-undefined

A request to an invalid file triggers an error with the message Unexpected token < and when TraceKit.computeStackTrace is done it results in {} being spat out. This results in sentry getting errors with undefined as the value and message.

@Myztiq
Copy link
Author

Myztiq commented Jan 30, 2015

Should I instead file this ticket in the Tracekit repo? I realized that it's actually a standalone dependency moments after submitting.

@Myztiq
Copy link
Author

Myztiq commented Jan 30, 2015

Moved issue to TraceKit: occ/TraceKit#68

@Myztiq Myztiq closed this as completed Jan 30, 2015
@mattrobenolt
Copy link
Contributor

@Myztiq We maintain our own fork, and TraceKit upstream is unmaintained.

@mattrobenolt mattrobenolt reopened this Jan 30, 2015
@niemyjski
Copy link

@mattrobenolt, @Myztiq, I'm helping maintain TraceKit upstream.. If you want to recreate all the issues / pull requests in the master I'll participate in the discussion and merge them in!

@mattrobenolt
Copy link
Contributor

I'm going to assume this has been fixed since we've pulled in latest TraceKit recently.

@niemyjski
Copy link

If it's not let me know

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

No branches or pull requests

3 participants