[jboss-cvs] JBossAS SVN: r103068 - projects/specs/trunk/jboss-jacc-api_1.4_spec/src/main/java/javax/security/jacc.

[jboss-cvs-commits at lists.jboss.org]

Author: sguilhen at redhat.com
Date: 2010-03-26 18:26:09 -0400 (Fri, 26 Mar 2010)
New Revision: 103068

Modified:
   projects/specs/trunk/jboss-jacc-api_1.4_spec/src/main/java/javax/security/jacc/URLPatternSpec.java
   projects/specs/trunk/jboss-jacc-api_1.4_spec/src/main/java/javax/security/jacc/WebResourcePermission.java
Log:
JBEE-33: Fixed URLPatternSpec constructor to correct a wrong interpretation of the specification.

Modified: projects/specs/trunk/jboss-jacc-api_1.4_spec/src/main/java/javax/security/jacc/URLPatternSpec.java
===================================================================
--- projects/specs/trunk/jboss-jacc-api_1.4_spec/src/main/java/javax/security/jacc/URLPatternSpec.java	2010-03-26 21:55:46 UTC (rev 103067)
+++ projects/specs/trunk/jboss-jacc-api_1.4_spec/src/main/java/javax/security/jacc/URLPatternSpec.java	2010-03-26 22:26:09 UTC (rev 103068)
@@ -80,9 +80,9 @@
    {
       if (spec == null)
          this.urlPattern = new URLPattern("/");
-      else if (spec.indexOf(WebResourcePermission.ENCODED_COLON) > 0)
+      else if (spec.indexOf(":") > 0)
       {
-         String[] patterns = spec.split(WebResourcePermission.ENCODED_COLON);
+         String[] patterns = spec.split(":");
          this.urlPatternList = new HashSet<URLPattern>();
          for (String pattern : patterns)
          {

Modified: projects/specs/trunk/jboss-jacc-api_1.4_spec/src/main/java/javax/security/jacc/WebResourcePermission.java
===================================================================
--- projects/specs/trunk/jboss-jacc-api_1.4_spec/src/main/java/javax/security/jacc/WebResourcePermission.java	2010-03-26 21:55:46 UTC (rev 103067)
+++ projects/specs/trunk/jboss-jacc-api_1.4_spec/src/main/java/javax/security/jacc/WebResourcePermission.java	2010-03-26 22:26:09 UTC (rev 103068)
@@ -386,7 +386,7 @@
          uri = "";
       }
       
-      // according to the JACC specification, all colons must be escaped.
+      // according to the JACC specification, all colons within the request URI must be escaped.
       if (uri.indexOf(':') > 0)
          uri = uri.replaceAll(":", ENCODED_COLON);
       return uri;