Skip to content

feat(aiohttp): Apply data_collection filtering to URL query strings#6833

Open
ericapisani wants to merge 1 commit into
py-2583-query-params-asgifrom
py-2583-query-params-aiohttp
Open

feat(aiohttp): Apply data_collection filtering to URL query strings#6833
ericapisani wants to merge 1 commit into
py-2583-query-params-asgifrom
py-2583-query-params-aiohttp

Conversation

@ericapisani

@ericapisani ericapisani commented Jul 16, 2026

Copy link
Copy Markdown
Member

Use the new data_collection settings to filter url.query and related
span attributes for both the aiohttp server and client integrations,
falling back to should_send_default_pii() when data_collection is not
configured.

Refs #6743
Refs #6743

Use the new data_collection settings to filter url.query and related
span attributes for both the aiohttp server and client integrations,
falling back to should_send_default_pii() when data_collection is not
configured.

Refs PY-2583
Refs #6743
@linear-code

linear-code Bot commented Jul 16, 2026

Copy link
Copy Markdown

PY-2583

@ericapisani ericapisani marked this pull request as ready for review July 16, 2026 18:04
@ericapisani ericapisani requested a review from a team as a code owner July 16, 2026 18:04
@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

93548 passed | ⏭️ 6302 skipped | Total: 99850 | Pass Rate: 93.69% | Execution Time: 328m 46s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +1694
Passed Tests 📈 +1694
Failed Tests
Skipped Tests

All tests are passing successfully.

✅ Patch coverage is 92.31%. Project has 2457 uncovered lines.
✅ Project coverage is 89.81%. Comparing base (base) to head (head).

Files with missing lines (1)
File Patch % Lines
sentry_sdk/integrations/aiohttp.py 92.31% ⚠️ 2 Missing and 5 partials
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.68%    89.81%    +0.13%
==========================================
  Files          193       193         —
  Lines        24004     24105      +101
  Branches      8340      8424       +84
==========================================
+ Hits         21526     21648      +122
- Misses        2478      2457       -21
- Partials      1387      1381        -6

Generated by Codecov Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant