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

the following shoud fail for idn-email: 'квіточка@поштаукр' #27

Open
lovas-ackerdemia opened this issue Jun 24, 2024 · 2 comments

Comments

@lovas-ackerdemia
Copy link

hello, it seems that the the format "idn-email" doesn't check, if the top-level domain is missing or should be separated by a dot

should pass: 'квіточка@пошта.укр'
should fail: 'квіточка@поштаукр'

@gene-hightower
Copy link

I have changed my address parser to provide basic domain validation, and a relaxed check for address length.
Up til now, addresses were parsed only according to the grammar in RFC-5321.
Upgrade to version 1.1.0 and domains will be verified to have at least two labels, and the top-level label to have at least two octets, no label exceeds 63 octets, and that the overall domain length (as encoded for DNS lookup) does not exceed 255 octets.

@Bessonov
Copy link

Bessonov commented Oct 8, 2024

It shouldn't. Both are pretty valid email addresses.

However, because of reasonable use cases, I think there should be an option for deniying local domains.

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

No branches or pull requests

3 participants