You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2020/06/21 22:31:31 UTC

[commons-validator] branch master updated: Best to check the domain separately

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-validator.git


The following commit(s) were added to refs/heads/master by this push:
     new d4d5e10  Best to check the domain separately
d4d5e10 is described below

commit d4d5e1066023479db4b652d271ea7c1982a8d400
Author: Sebb <se...@apache.org>
AuthorDate: Sun Jun 21 23:31:23 2020 +0100

    Best to check the domain separately
---
 src/main/java/org/apache/commons/validator/EmailValidator.java | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/main/java/org/apache/commons/validator/EmailValidator.java b/src/main/java/org/apache/commons/validator/EmailValidator.java
index 610306c..f82a4c3 100644
--- a/src/main/java/org/apache/commons/validator/EmailValidator.java
+++ b/src/main/java/org/apache/commons/validator/EmailValidator.java
@@ -192,8 +192,6 @@ public class EmailValidator {
             return false;
         }
         
-        // TODO: the tld should be checked against some sort of configurable 
-        // list
         String tld = domainSegment[len - 1];
         if (tld.length() > 1) {
             if (! TLD_PATTERN.matcher(tld).matches()) {