-
Notifications
You must be signed in to change notification settings - Fork 639
Comparing changes
Open a pull request
base repository: getsentry/sentry-python
base: master
head repository: getsentry/sentry-python
compare: py-2581-cookies
- 12 commits
- 30 files changed
- 1 contributor
Commits on Jul 8, 2026
-
ref(integrations): route HTTP header filtering through data collectio…
…n config `_filter_headers` previously used a hardcoded sensitive-header tuple and a `send_default_pii`/`use_annotated_value` toggle. It now delegates to `_apply_key_value_collection_filtering` from `sentry_sdk.data_collection`, so header scrubbing respects the new `data_collection.http_headers.request` allowlist/denylist/off configuration. Cookie and set-cookie headers are always redacted regardless of mode. Drops the now-unused `use_annotated_value` parameter from all call sites. Work to scrub cookies in a more granular way will be tackled as part of PY-2581/#6741. Fixes PY-2584 Fixes #6744
Configuration menu - View commit details
-
Copy full SHA for 7541460 - Browse repository at this point
Copy the full SHA 7541460View commit details
Commits on Jul 9, 2026
-
Configuration menu - View commit details
-
Copy full SHA for 5e9f610 - Browse repository at this point
Copy the full SHA 5e9f610View commit details -
Make changes to adjust to the data collection option being within the…
… _experiments property in the client
Configuration menu - View commit details
-
Copy full SHA for a590c8e - Browse repository at this point
Copy the full SHA a590c8eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 16ed0c8 - Browse repository at this point
Copy the full SHA 16ed0c8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 486f0bc - Browse repository at this point
Copy the full SHA 486f0bcView commit details -
add test coverage for edge case around host header being dropped but …
…still being needed for the url attribute
Configuration menu - View commit details
-
Copy full SHA for 7d14a81 - Browse repository at this point
Copy the full SHA 7d14a81View commit details -
Configuration menu - View commit details
-
Copy full SHA for bb98334 - Browse repository at this point
Copy the full SHA bb98334View commit details -
fix(aws-lambda): Remove vendored deps from new embedded-sdk test fixt…
…ures The new lambda_functions_with_embedded_sdk fixture directories were missing the .gitignore that the other fixtures use to keep everything except index.py untracked. As a result, certifi and urllib3 packages installed by the test setup got committed, and ruff failed CI linting against them since they're unmodified third-party code. Add the missing .gitignore to each new fixture directory and remove the committed vendored packages; they are regenerated automatically at test time via `uv pip install --target`.
Configuration menu - View commit details
-
Copy full SHA for 61d1618 - Browse repository at this point
Copy the full SHA 61d1618View commit details -
Configuration menu - View commit details
-
Copy full SHA for 951c408 - Browse repository at this point
Copy the full SHA 951c408View commit details
Commits on Jul 10, 2026
-
feat(integrations): apply data_collection cookie filtering to wsgi, s…
…tarlette, litestar, starlite Extends the granular cookie collection controls (data_collection.cookies) to _wsgi_common, starlette, litestar, and starlite, matching the behavior already used elsewhere. Falls back to should_send_default_pii() when data_collection is not configured for cookies. HTTP "Cookie" and "set-cookie" headers will continue to be completely filtered with the "[Filtered]" value. Fixes PY-2581 Fixes #6741
Configuration menu - View commit details
-
Copy full SHA for a19be39 - Browse repository at this point
Copy the full SHA a19be39View commit details -
fix(integrations): omit cookies field when data_collection cookies mo…
…de is off Previously the async request extractors attached an empty cookies dict when the data_collection cookies mode was off, while sync route handlers omitted it entirely. Make all integrations consistent by not attaching the cookies field at all when filtering yields no cookies.
Configuration menu - View commit details
-
Copy full SHA for 620e222 - Browse repository at this point
Copy the full SHA 620e222View commit details -
Configuration menu - View commit details
-
Copy full SHA for ec6f520 - Browse repository at this point
Copy the full SHA ec6f520View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff master...py-2581-cookies