Skip to content
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

feat(py): Add validate_pii_selector #2687

Merged
merged 1 commit into from
Nov 6, 2023
Merged

Conversation

Dav1dde
Copy link
Member

@Dav1dde Dav1dde commented Nov 3, 2023

Expose a function to validate PII selectors directly, so we can use it to validate safe fields in Sentry.

@Dav1dde Dav1dde requested a review from a team November 3, 2023 12:23
@Dav1dde Dav1dde force-pushed the feat/validate-safe-fields branch 2 times, most recently from 28cf517 to 637d6d2 Compare November 3, 2023 12:31
@Dav1dde Dav1dde self-assigned this Nov 3, 2023
py/CHANGELOG.md Outdated Show resolved Hide resolved
py/tests/test_processing.py Outdated Show resolved Hide resolved
@Dav1dde Dav1dde force-pushed the feat/validate-safe-fields branch from 637d6d2 to 5ce698f Compare November 6, 2023 08:00
@Dav1dde Dav1dde force-pushed the feat/validate-safe-fields branch from 5ce698f to 3ee3257 Compare November 6, 2023 08:02
py/tests/test_processing.py Outdated Show resolved Hide resolved
py/sentry_relay/processing.py Outdated Show resolved Hide resolved
py/sentry_relay/processing.py Outdated Show resolved Hide resolved
@Dav1dde Dav1dde changed the title feat(py): Add validate_pii_selector_spec feat(py): Add validate_pii_selector Nov 6, 2023
@Dav1dde Dav1dde force-pushed the feat/validate-safe-fields branch from 0b19a31 to 0b4d84a Compare November 6, 2023 13:08
@Dav1dde Dav1dde enabled auto-merge (squash) November 6, 2023 13:09
@Dav1dde Dav1dde merged commit 5a58efb into master Nov 6, 2023
20 checks passed
@Dav1dde Dav1dde deleted the feat/validate-safe-fields branch November 6, 2023 14:04
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.

4 participants