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

normaliseEmail yandex conversion #2440

Closed
AayushGH opened this issue Aug 20, 2024 · 1 comment
Closed

normaliseEmail yandex conversion #2440

AayushGH opened this issue Aug 20, 2024 · 1 comment
Labels

Comments

@AayushGH
Copy link
Contributor

AayushGH commented Aug 20, 2024

Describe the bug
Currently, all yandex domains like ua, com etc. are being converted to yandex.ru.
This behaviour should be optional with the default setting being true for the conversion.

Motivation

In micro-service communication, the services using this sanitizer shares the converted email, but other services expect the original, non converted email. This breaks the communication flow.

Examples
Screenshot 2024-08-20 at 3 45 52 PM
Screenshot 2024-08-20 at 3 46 34 PM

Additional context
Validator.js version: Latest
Node.js version: Latest
OS platform: [macOS]

@pano9000
Copy link
Contributor

closed by #2441

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

No branches or pull requests

2 participants