bugsnag / bugsnag-python

Official BugSnag error monitoring and error reporting for django, flask, tornado and other python apps.
https://docs.bugsnag.com/platforms/python/
MIT License
84 stars 42 forks source link

Relax checks on response status code #389

Closed imjoehaines closed 3 months ago

imjoehaines commented 3 months ago

Goal

Currently we require a 200 status code for events and a 202 for sessions but there's no need to be so specific as any 2xx status indicates success

Now we only warn on codes outside of the 2xx range, unless a specific code has been set in options['success']. If a specific code is given then it must match exactly, i.e. it has the same behaviour as before

This generally shouldn't affect users as the event & session endpoints always respond with 200 & 202 but can prevent spurious warnings if delivering to a custom service that doesn't use the same codes (e.g. a forwarding agent)