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(backport): exclude utm_term from collection #155

Merged
merged 2 commits into from
Apr 18, 2024

Conversation

ramboz
Copy link
Collaborator

@ramboz ramboz commented Apr 17, 2024

The utm_term is typically populated from search queries and can theoretically leak PII. So it's safest to exclude it from collection for now

Fix #153

The utm_term is typically populated from search queries and can theoretically leak PII. So it's safest to exclude it from collection for now

Fix #153
Copy link

This PR will trigger a patch release when merged.

@ramboz ramboz requested review from trieloff and chicharr and removed request for trieloff April 17, 2024 15:30
src/index.js Outdated Show resolved Hide resolved
Co-authored-by: Lars Trieloff <[email protected]>
@ramboz ramboz changed the title fix(backport): exclude utm_term from collection #154 fix(backport): exclude utm_term from collection Apr 18, 2024
@ramboz ramboz merged commit 7a5e5b2 into 1.x Apr 18, 2024
5 checks passed
@ramboz ramboz deleted the exclude-utm_term-backport branch April 18, 2024 15:38
Copy link

🎉 This PR is included in version 1.15.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants