You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@river.apache.org by pe...@apache.org on 2012/07/11 09:50:14 UTC

svn commit: r1360043 - /river/jtsk/trunk/src/org/apache/river/api/security/DefaultPolicyScanner.java

Author: peter_firmstone
Date: Wed Jul 11 07:50:13 2012
New Revision: 1360043

URL: http://svn.apache.org/viewvc?rev=1360043&view=rev
Log:
Backed out last change for URI codebase grants which causes numerous test failures.  Windows platforms have issues with file name paths, illegal characters need to be escaped, this requires further work, not as simple as I originally thought.

Modified:
    river/jtsk/trunk/src/org/apache/river/api/security/DefaultPolicyScanner.java

Modified: river/jtsk/trunk/src/org/apache/river/api/security/DefaultPolicyScanner.java
URL: http://svn.apache.org/viewvc/river/jtsk/trunk/src/org/apache/river/api/security/DefaultPolicyScanner.java?rev=1360043&r1=1360042&r2=1360043&view=diff
==============================================================================
--- river/jtsk/trunk/src/org/apache/river/api/security/DefaultPolicyScanner.java (original)
+++ river/jtsk/trunk/src/org/apache/river/api/security/DefaultPolicyScanner.java Wed Jul 11 07:50:13 2012
@@ -496,8 +496,7 @@ class DefaultPolicyScanner {
                     Collection<PrincipalEntry> pe,
                     Collection<PermissionEntry> perms){
             this.signers = signers;
-            // Spaces are an illegal character in URI, so get rid of them now.
-            this.codebase = codebase.replace(" ", "%20");
+            this.codebase = codebase;
             this.principals = pe;
             this.permissions = perms;
         }