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

Add ajaxError handler so we get better Sentry errors for API calls #920

Closed
kollivier opened this issue Aug 25, 2018 · 1 comment
Closed
Assignees

Comments

@kollivier
Copy link
Contributor

kollivier commented Aug 25, 2018

Summary

Right now, when a timeout or 500 error occurs during an API call, Sentry appears to get an error object that is actually an object with the jQuery callbacks for the API call.

We should instead (after testing it works, of course!) add a $(document).ajaxError handler to catch and report these errors properly, so we can figure out exactly which API calls are erroring or timing out, and why.

Category

ENHANCEMENT

Real-life consequences (anything community should be aware of, for instance how it affects end users)

Right now we can see that API calls are either timing out or failing, but we don't know which calls and under which circumstances these events are happening. This makes us unable to address these issues effectively.

@kollivier kollivier self-assigned this Aug 25, 2018
@kollivier kollivier removed the URGENT label Sep 4, 2018
@kollivier
Copy link
Contributor Author

This is implemented now, looks like I just lost track of this and forgot to close!

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

1 participant