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

fix(pii): filter the IP address when setting is turned on #4522

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

Litarnus
Copy link
Contributor

@Litarnus Litarnus commented Feb 25, 2025

This PR removes the client_ip from the normalization settings if the proper flag is set. This way it cannot be accidentally copied somewhere down the line before scrubbing happens.

#skip-changelog

@Litarnus Litarnus marked this pull request as ready for review February 25, 2025 15:35
@Litarnus Litarnus requested a review from a team as a code owner February 25, 2025 15:35
@Litarnus Litarnus merged commit 080159d into master Feb 26, 2025
26 of 27 checks passed
@Litarnus Litarnus deleted the martinl/filter-ip-address-by-setting branch February 26, 2025 08:30
@Litarnus Litarnus self-assigned this Feb 26, 2025
Litarnus added a commit that referenced this pull request Feb 26, 2025
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.

2 participants