Skip to content

fix(aiohttp): Gate server request query_string via data_collection option - #7300

Open
ericapisani wants to merge 1 commit into
masterfrom
py-2752-gate-aiohttp-query-string
Open

fix(aiohttp): Gate server request query_string via data_collection option#7300
ericapisani wants to merge 1 commit into
masterfrom
py-2752-gate-aiohttp-query-string

Conversation

@ericapisani

Copy link
Copy Markdown
Member

Filter the request event's query_string through the url_query_params data_collection behaviour, mirroring the existing gating for the client span's url.query attribute.

Fixes PY-2752
Fixes #7299

…tion

Filter the request event's query_string through the
url_query_params data_collection behaviour, mirroring the
existing gating for the client span's url.query attribute.

Fixes PY-2752
Fixes #7299
@ericapisani
ericapisani requested a review from a team as a code owner August 28, 2026 19:19
@linear-code

linear-code Bot commented Aug 28, 2026

Copy link
Copy Markdown

PY-2752

@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

119159 passed | ⏭️ 6762 skipped | Total: 125921 | Pass Rate: 94.63% | Execution Time: 422m 48s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +20520
Passed Tests 📈 +20237
Failed Tests
Skipped Tests 📈 +283

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2494 uncovered lines.
❌ Project coverage is 90.26%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    90.29%    90.26%    -0.03%
==========================================
  Files          193       193         —
  Lines        25601     25606        +5
  Branches      9436      9442        +6
==========================================
+ Hits         23116     23112        -4
- Misses        2485      2494        +9
- Partials      1435      1437        +2

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.

Gate aiohttp query string in error event on data collection

1 participant