Repository: struts Updated Branches: refs/heads/master 7621c2bc9 -> a0fdca138
Improves validation RegEx Project: http://git-wip-us.apache.org/repos/asf/struts/repo Commit: http://git-wip-us.apache.org/repos/asf/struts/commit/a0fdca13 Tree: http://git-wip-us.apache.org/repos/asf/struts/tree/a0fdca13 Diff: http://git-wip-us.apache.org/repos/asf/struts/diff/a0fdca13 Branch: refs/heads/master Commit: a0fdca138feec2c2e94eb75ca1f8b76678b4d152 Parents: 7621c2b Author: Lukasz Lenart <lukaszlen...@apache.org> Authored: Tue Jun 14 06:53:36 2016 +0200 Committer: Lukasz Lenart <lukaszlen...@apache.org> Committed: Tue Jun 14 07:02:09 2016 +0200 ---------------------------------------------------------------------- .../com/opensymphony/xwork2/validator/validators/URLValidator.java | 2 +- .../java/com/opensymphony/xwork2/validator/URLValidatorTest.java | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/struts/blob/a0fdca13/core/src/main/java/com/opensymphony/xwork2/validator/validators/URLValidator.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/validator/validators/URLValidator.java b/core/src/main/java/com/opensymphony/xwork2/validator/validators/URLValidator.java index f49d914..fd678a7 100644 --- a/core/src/main/java/com/opensymphony/xwork2/validator/validators/URLValidator.java +++ b/core/src/main/java/com/opensymphony/xwork2/validator/validators/URLValidator.java @@ -81,7 +81,7 @@ public class URLValidator extends FieldValidatorSupport { "|((\\d|[1-9]\\d|1\\d{2}|2[0-4][0-9]|25[0-5])\\.){3}" + "(\\d|[1-9]\\d|1\\d{2}|2[0-4][0-9]|25[0-5])" + ")(:\\d+)?" + - ")(((\\/+([a-z0-9$_\\.\\+!\\*\\'\\(\\),;:@&=-]|%[0-9a-f]{2})*)*" + + ")(((\\/{0,1}([a-z0-9$_\\.\\+!\\*\\'\\(\\),;:@&=-]|%[0-9a-f]{2})*)*" + "(\\?([a-z0-9$_\\.\\+!\\*\\'\\(\\),;:@&=-]|%[0-9a-f]{2})*)" + "?)?)?" + "(#([a-z0-9$_\\.\\+!\\*\\'\\(\\),;:@&=-]|%[0-9a-f]{2})*)?" + http://git-wip-us.apache.org/repos/asf/struts/blob/a0fdca13/core/src/test/java/com/opensymphony/xwork2/validator/URLValidatorTest.java ---------------------------------------------------------------------- diff --git a/core/src/test/java/com/opensymphony/xwork2/validator/URLValidatorTest.java b/core/src/test/java/com/opensymphony/xwork2/validator/URLValidatorTest.java index 2895d80..77223eb 100644 --- a/core/src/test/java/com/opensymphony/xwork2/validator/URLValidatorTest.java +++ b/core/src/test/java/com/opensymphony/xwork2/validator/URLValidatorTest.java @@ -138,6 +138,7 @@ public class URLValidatorTest extends XWorkTestCase { assertFalse(pattern.matcher("").matches()); assertFalse(pattern.matcher(" ").matches()); assertFalse(pattern.matcher("no url").matches()); + assertFalse(pattern.matcher("http://example.com////////////////////////////////////////////////////////////////////////////////////??";).matches()); assertTrue(pattern.matcher("http://www.opensymphony.com";).matches()); assertTrue(pattern.matcher("https://www.opensymphony.com";).matches());
