[jira] Created: (VALIDATOR-273) EmailValidator does not support mailboxes at TLDs

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (VALIDATOR-273) EmailValidator does not support mailboxes at TLDs

JIRA jira@apache.org
EmailValidator does not support mailboxes at TLDs
-------------------------------------------------

                 Key: VALIDATOR-273
                 URL: https://issues.apache.org/jira/browse/VALIDATOR-273
             Project: Commons Validator
          Issue Type: Bug
          Components: Framework
    Affects Versions: 1.3.1 Release
            Reporter: Marcos Sanz


According to RFC 5321 and 5322, mailboxes at top level domain names are permitted. Thus, the syntax localpart@tld is valid. Explicit text to this topic can be found in RFC 5321:

2.3.5.  Domain Names

   A domain name (or often just a "domain") consists of one or more
   components, separated by dots if more than one appears.  In the case
   of a top-level domain used by itself in an email address, a single
   string is used without any dots.

Accordingly, the code

                EmailValidator ev = EmailValidator.getInstance();
                System.out.println(ev.isValid("m@de"));

should output "true".

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (VALIDATOR-273) EmailValidator does not support mailboxes at TLDs

JIRA jira@apache.org

     [ https://issues.apache.org/jira/browse/VALIDATOR-273?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chris Lee updated VALIDATOR-273:
--------------------------------

    Attachment: commons-validator_allow-tld-domains.patch

Added an additional check in EmailValidator to also pass if domain-part is a valid TLD, not just if it's a valid domain as judged by DomainValidator.

This could also be accomplished by modifying DomainValidator to say raw TLDs are valid (change a + to a * in the regex), and all unit tests pass when I did that too, but I figured this was less likely to have unintended consequences.

> EmailValidator does not support mailboxes at TLDs
> -------------------------------------------------
>
>                 Key: VALIDATOR-273
>                 URL: https://issues.apache.org/jira/browse/VALIDATOR-273
>             Project: Commons Validator
>          Issue Type: Bug
>          Components: Framework
>    Affects Versions: 1.3.1 Release
>            Reporter: Marcos Sanz
>         Attachments: commons-validator_allow-tld-domains.patch
>
>
> According to RFC 5321 and 5322, mailboxes at top level domain names are permitted. Thus, the syntax localpart@tld is valid. Explicit text to this topic can be found in RFC 5321:
> 2.3.5.  Domain Names
>    A domain name (or often just a "domain") consists of one or more
>    components, separated by dots if more than one appears.  In the case
>    of a top-level domain used by itself in an email address, a single
>    string is used without any dots.
> Accordingly, the code
> EmailValidator ev = EmailValidator.getInstance();
> System.out.println(ev.isValid("m@de"));
> should output "true".

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.