Skip to content

User Token is not taken for validation as Email is not in valid format. #4723

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

Merged
merged 5 commits into from
Aug 26, 2023

Conversation

Panandhan22
Copy link
Contributor

@Panandhan22 Panandhan22 commented May 12, 2023

Proposed changes

Email and User Token both fields are mapped in "Send custom behavioural event" mapping and sent over the event paylaod too. But the event is failing for delivery with "Payload validation error" as the email is not in valid format, why it is not taking the User token for validation? Event payload should have the email with valid format or empty string, so that the user token will take precedence and can be validated. An invalid email will not pass mapping validation. Hence the event delivery is failing.

Merge timing

Related issues (optional)

### Email and User Token both fields are mapped in "Send custom behavioural event" mapping and sent over the event paylaod too. But the event is failing for delivery with "Payload validation error" as the email is not in valid format, why it is not taking the User token for validation?
Event payload should have the email with valid format or empty string, so that the user token will take precedence and can be validated. An invalid email will not pass mapping validation. Hence the event delivery is failing.
@Panandhan22 Panandhan22 marked this pull request as ready for review August 22, 2023 01:02
@Panandhan22 Panandhan22 requested a review from a team as a code owner August 22, 2023 01:02
@Panandhan22 Panandhan22 requested review from rchinn1 and removed request for a team August 22, 2023 01:02
@rchinn1 rchinn1 merged commit 6bc0275 into develop Aug 26, 2023
@rchinn1 rchinn1 deleted the Panandhan22-patch-7 branch August 26, 2023 02:03
@github-actions
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

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

Successfully merging this pull request may close these issues.

2 participants