ref(aiohttp): Remove support for send_default_pii - #7632
Conversation
Codecov Results 📊✅ 66270 passed | ❌ 1 failed | ⏭️ 3019 skipped | Total: 69290 | Pass Rate: 95.64% | Execution Time: 192m 36s 📊 Comparison with Base Branch
➕ New Tests (1)View new tests
❌ Failed Tests
|
| File | Patch % | Lines |
|---|---|---|
| sentry_sdk/integrations/aiohttp.py | 90.32% |
Coverage diff
@@ Coverage Diff @@
## master #PR +/-##
==========================================
- Coverage 90.22% 90.10% -0.12%
==========================================
Files 194 180 -14
Lines 25997 21106 -4891
Branches 9662 7624 -2038
==========================================
+ Hits 23454 19016 -4438
- Misses 2543 2090 -453
- Partials 1451 1198 -253Generated by Codecov Action
send_default_pii
|
|
||
| @pytest.mark.asyncio | ||
| @pytest.mark.parametrize("send_pii", [True, False]) | ||
| async def test_url_query_attribute( |
There was a problem hiding this comment.
There's an extensive data_collection version of this test
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 627b51d. Configure here.

Closes https://linear.app/getsentry/issue/PY-2809/remove-send-default-pii-from-aiohttp