Preventing double-hashing for TikTok Conversions.

[seg-leonelsanches]

It prevents double-hashing for email and phone when these traits are already hashed (SHA256).

Testing

• Added unit tests for new functionality
• Tested end-to-end using the local server
• [Segmenters] Tested in the staging environment

[joe-ayoub-segment]

hi @jae-rhee-tiktok - any chance you could take a look at this PR please? It was raised by @seg-leonelsanches from the Segment Solution Architect team. Is it something we should deploy?

[jae-rhee-tiktok]

hi @joe-ayoub-segment thank you for the update.

The change looks good to prevent double hashing. Is it common for clients to hash the values before sending them to Segment?

[seg-leonelsanches]

hi @joe-ayoub-segment thank you for the update.

The change looks good to prevent double hashing. Is it common for clients to hash the values before sending them to Segment?

Hi @jae-rhee-tiktok 👋

Yes, our financial sector customers can't send plain data to Segment regarding PII such as email and phone. It's the same for #2034.

[jae-rhee-tiktok]

hi @joe-ayoub-segment thank you for the update.
The change looks good to prevent double hashing. Is it common for clients to hash the values before sending them to Segment?

Hi @jae-rhee-tiktok 👋

Yes, our financial sector customers can't send plain data to Segment regarding PII such as email and phone. It's the same for #2034.

@seg-leonelsanches nice to meet you! The changes look good. @joe-ayoub-segment can we merge this? Thank you!

[joe-ayoub-segment]

hi @seg-leonelsanches PR deployed. Please check it all looks good!