Author: sebb Date: Wed Dec 31 00:15:52 2014 New Revision: 1648633 URL: http://svn.apache.org/r1648633 Log: VALIDATOR-342 URLValidator returns false for http://example.rocks Add a note that the code checks the domain
Modified: commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/routines/UrlValidator.java Modified: commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/routines/UrlValidator.java URL: http://svn.apache.org/viewvc/commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/routines/UrlValidator.java?rev=1648633&r1=1648632&r2=1648633&view=diff ============================================================================== --- commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/routines/UrlValidator.java (original) +++ commons/proper/validator/trunk/src/main/java/org/apache/commons/validator/routines/UrlValidator.java Wed Dec 31 00:15:52 2014 @@ -273,6 +273,9 @@ public class UrlValidator implements Ser /** * <p>Checks if a field has a valid url address.</p> * + * Note that the method calls #isValidAuthority() + * which checks that the domain is valid. + * * @param value The value validation is being performed on. A <code>null</code> * value is considered invalid. * @return true if the url is valid. @@ -349,6 +352,7 @@ public class UrlValidator implements Ser /** * Returns true if the authority is properly formatted. An authority is the combination * of hostname and port. A <code>null</code> authority value is considered invalid. + * Note: this implementation validates the domain. * @param authority Authority value to validate. * @return true if authority (hostname and port) is valid. */