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: capture additional errors that shouldn't be reported to Sentry #93

Merged
merged 1 commit into from
Jan 16, 2019

Conversation

bbangert
Copy link
Member

Avoids capturing errors for the following:

  • Unable to send to a client
  • Invalid json text sent to server
  • DynamoDB Internal Server Error that should be retried

Closes #87

jrconlin
jrconlin previously approved these changes Jan 11, 2019
pjenvey
pjenvey previously approved these changes Jan 11, 2019
Avoids capturing errors for the following:
- Unable to send to a client
- Invalid json text sent to server
- DynamoDB Internal Server Error that should be retried

Closes #87
@bbangert bbangert merged commit 9418541 into master Jan 16, 2019
@bbangert bbangert deleted the feat/issue-87 branch January 16, 2019 19:52
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