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 2015/11/05 01:48:29 UTC

svn commit: r1712701 - in /commons/proper/validator/trunk/src: changes/changes.xml test/java/org/apache/commons/validator/routines/UrlValidatorTest.java

Author: sebb
Date: Thu Nov  5 00:48:29 2015
New Revision: 1712701

URL: http://svn.apache.org/viewvc?rev=1712701&view=rev
Log:
VALIDATOR-361 UrlValidator rejects new gTLDs with more than 4 characters

Modified:
    commons/proper/validator/trunk/src/changes/changes.xml
    commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java

Modified: commons/proper/validator/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/validator/trunk/src/changes/changes.xml?rev=1712701&r1=1712700&r2=1712701&view=diff
==============================================================================
--- commons/proper/validator/trunk/src/changes/changes.xml (original)
+++ commons/proper/validator/trunk/src/changes/changes.xml Thu Nov  5 00:48:29 2015
@@ -43,6 +43,10 @@ The <action> type attribute can be add,u
   <body>
 
   <release version="1.5.0" date="tba" description="tba">
+    <action issue="VALIDATOR-361" type="add" dev="sebb">
+    UrlValidator rejects new gTLDs with more than 4 characters
+    Added unit test to show that this has been fixed
+    </action>
     <action issue="VALIDATOR-341" type="add" dev="sebb">
     Make TLD list configurable;
     both generic and country-code now support addition and removal

Modified: commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java
URL: http://svn.apache.org/viewvc/commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java?rev=1712701&r1=1712700&r2=1712701&view=diff
==============================================================================
--- commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java (original)
+++ commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java Thu Nov  5 00:48:29 2015
@@ -409,6 +409,11 @@ public class UrlValidatorTest extends Te
 //        assertTrue(validator.isValid("http://test.xn--hlcj6aya9esc7a")); // Tamil
     }
 
+   public void testValidator361() {
+       UrlValidator validator = new UrlValidator();
+       assertTrue(validator.isValid("http://hello.tokyo/"));
+    }
+
    
    /**
     * Only used to debug the unit tests.