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.

Sign up for GitHub

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

Jump to bottom

Sentry should not track legit HTTP exceptions (30x, 40x) #774

Closed
noirbizarre opened this issue Feb 17, 2017 · 1 comment · Fixed by #1116
Closed

Sentry should not track legit HTTP exceptions (30x, 40x) #774

noirbizarre opened this issue Feb 17, 2017 · 1 comment · Fixed by #1116
Labels
bug
Milestone
1.1.5

Comments

@noirbizarre
Copy link
Contributor

Sentry should not track HTTP exception which are standard cases and properly handled (ie. 30x and 40x).

Currently, this is poluting the sentry logs:
https://sentry.data.gouv.fr/etalab/www-datagouvfr/issues/74480/
https://sentry.data.gouv.fr/share/issue/332e3734323639/
...

NotFound: 404: Not Found
(5 additional frame(s) were not displayed)
...
  File "udata/api/__init__.py", line 88, in wrapper
    return func(*args, **kwargs)
  File "udata/core/spatial/api.py", line 131, in get
    zone = GeoZone.objects.get_or_404(id=id)

NotFound: 404: Not Found
@noirbizarre noirbizarre modified the milestone: 1.0.2 Feb 17, 2017
@noirbizarre
Copy link
Contributor Author

Related to getsentry/raven-python#963

@noirbizarre noirbizarre modified the milestones: Future, 1.0.2 Feb 20, 2017
@noirbizarre noirbizarre modified the milestones: 1.1.5, Future Sep 8, 2017
@noirbizarre noirbizarre mentioned this issue Sep 8, 2017
Merged
noirbizarre added a commit that referenced this issue Sep 9, 2017
@noirbizarre
Merge pull request #1116 from noirbizarre/gh-774-sentry-exceptions-ha…
7c75eda 
…ndling

Raise raven minimum supported version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
1.1.5
Development

Successfully merging a pull request may close this issue.

Raise raven minimum supported version
1 participant
@noirbizarre