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

Redact email addresses as [email] #1638

Merged
merged 1 commit into from
Oct 19, 2023
Merged

Redact email addresses as [email] #1638

merged 1 commit into from
Oct 19, 2023

Conversation

andysellick
Copy link
Contributor

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

What

  • where you enter your email address (https://www.gov.uk/email/manage/authenticate) we have GA4 tracking
  • the tracking currently defaults to not collecting any data from the form, so inputs are listed as [redacted]
  • we want to make the redaction more specific so it says [email]
  • to do this, set an attribute on the form so that values of inputs are collected, but when this occurs we apply our PII stripping code, which means that any email address entered into this field is returned as [email] instead

Why

To make the GA4 data more understandable.

Visual changes

None.

Trello card: https://trello.com/c/AvcrXf8U/703-emails-should-be-email-not-redacted-on-email-subscription-journey

- where you enter your email address (https://www.gov.uk/email/manage/authenticate) we have GA4 tracking
- the tracking currently defaults to not collecting any data from the form, so inputs are listed as [redacted]
- we want to make the redaction more specific so it says [email]
- to do this, set an attribute on the form so that values of inputs are collected, but when this occurs we apply our PII stripping code, which means that any email address entered into this field is returned as [email] instead
@andysellick andysellick requested a review from AshGDS October 19, 2023 10:16
@andysellick andysellick merged commit efec488 into main Oct 19, 2023
7 checks passed
@andysellick andysellick deleted the redact-email branch October 19, 2023 12:22
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