sentry don't capture response

[nichtsam]

I got this from sentry:

Error: Object captured as exception with keys: data, internal, status, statusText

Test Plan

Checklist

• Tests updated
• Docs updated

Screenshots

[kentcdodds]

Thanks for this contribution. Can you verify that we are still getting Sentry errors from the server when we hit a 404 or a 500?

[nichtsam]

Thanks for this contribution. Can you verify that we are still getting Sentry errors from the server when we hit a 404 or a 500?

Thanks for reviewing! I tested it, and I believe we won’t get Sentry errors for 404s since they’re thrown manually, which I considered handled. However, 500 errors should still be captured by Sentry since they’re actual errors. Essentially, throw new Response() won’t be captured, but throw new Error() will.

[kentcdodds]

Ok, I think 400s probably shouldn't be reported anyway so we're probably fine. Thanks!