-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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 lost details on HttpProcessingError #9052
Conversation
for more information, see https://pre-commit.ci
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## master #9052 +/- ##
==========================================
+ Coverage 98.30% 98.31% +0.01%
==========================================
Files 107 107
Lines 34355 34510 +155
Branches 4066 4100 +34
==========================================
+ Hits 33771 33929 +158
+ Misses 412 410 -2
+ Partials 172 171 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Our codecov doesn't actually include any other types of exceptions... |
There doesn't appear to be any reason we should be catching other exceptions... |
Backport to 3.10: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 8911419 on top of patchback/backports/3.10/89114196040e6b4e435799939dfde6141223cc12/pr-9052 Backporting merged PR #9052 into master
🤖 @patchback |
Backport to 3.11: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 8911419 on top of patchback/backports/3.11/89114196040e6b4e435799939dfde6141223cc12/pr-9052 Backporting merged PR #9052 into master
🤖 @patchback |
(cherry picked from commit 8911419)
(cherry picked from commit 8911419)
Fixes #8395.