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

Fix traceback parsing under Safari 6. #15

Closed
wants to merge 4 commits into from

Conversation

jeamland
Copy link
Contributor

@jeamland jeamland commented Aug 2, 2012

Safari (possibly just Safari 6) ends up in the Firefox traceback parser. This
actually works with two minor fixes:

  • Function arguments are not supplied so don't attempt to parse them out.
  • Tracebacks appear to end in a '[native code]' entry. Deal with this.

Verified

This commit was signed with the committer’s verified signature.
setchy Adam Setch
Safari (possibly just Safari 6) ends up in the Firefox traceback parser. This
actually works with two minor fixes:

- Function arguments are not supplied so don't attempt to parse them out.
- Tracebacks appear to end in a '[native code]' entry. Deal with this.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@ipmb
Copy link
Contributor

ipmb commented Aug 15, 2012

Tests still fail on Safari 6 after applying c368a84.

@ipmb
Copy link
Contributor

ipmb commented Aug 15, 2012

Fixed test in e18c7df and merged.

@ipmb ipmb closed this Aug 15, 2012
kamilogorek pushed a commit that referenced this pull request Jun 12, 2018
Tell Sentry to shut up, fixes #13 #14
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.

None yet

2 participants