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

[Content]: Trade Desk doc not aligned with functionality #7468

Open
bailey-trotter-segment opened this issue Feb 28, 2025 · 0 comments
Open
Assignees
Labels
content When the bug is about content that needs to get fixed triaged

Comments

@bailey-trotter-segment
Copy link

What article on segment.com/docs is affected?

https://segment.com/docs/connections/destinations/catalog/actions-the-trade-desk-crm/#faqs:~:text=If%20any%20of,the%20ingestion%20process.

What part(s) of the article would you like to see updated?

It would seem Trade Desk prefers you send clear text emails over which we can handle just fine, where it gets confusing is here.

We seem to imply if you are storing the hashed version in Segment, to set the PII Type to "Hashed Email". When doing testing of the code, I discovered we actually do a lot more and the docs to not make it clear.

TTD expects an email to be normalized, sha256 hashed then hex base64 encoded. What we do, when you select the "hashed email" in the drop down is a handful of things:

  • if its a plain text email, we normalize and hash as needed for TTD
  • if the email is sha256 hash, we assume you normalized it correctly per TTD requirements and hex base64 encode it
  • if the email is base64 encoded, we assume you normalized and sha256 hash it correctly per TTD requirements and send it on as is

Additional information

  • Here, if user selects "hash email" we set pii_type to EmailHashedUnifiedId2

  • here if pii_type is EmailHashedUnifiedId2 we call the hashing function

@bailey-trotter-segment bailey-trotter-segment added content When the bug is about content that needs to get fixed triage labels Feb 28, 2025
@forstisabella forstisabella self-assigned this Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content When the bug is about content that needs to get fixed triaged
Projects
None yet
Development

No branches or pull requests

2 participants