accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From e..@apache.org
Subject svn commit: r1190280 - in /incubator/accumulo/branches/1.3/src/core/src: main/java/org/apache/accumulo/core/security/Authorizations.java test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java
Date Fri, 28 Oct 2011 12:49:12 GMT
Author: ecn
Date: Fri Oct 28 12:49:11 2011
New Revision: 1190280

URL: http://svn.apache.org/viewvc?rev=1190280&view=rev
Log:
ACCUMULO-90: allow colon as a legal character in a visibility term

Modified:
    incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/security/Authorizations.java
    incubator/accumulo/branches/1.3/src/core/src/test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java

Modified: incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/security/Authorizations.java
URL: http://svn.apache.org/viewvc/incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/security/Authorizations.java?rev=1190280&r1=1190279&r2=1190280&view=diff
==============================================================================
--- incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/security/Authorizations.java
(original)
+++ incubator/accumulo/branches/1.3/src/core/src/main/java/org/apache/accumulo/core/security/Authorizations.java
Fri Oct 28 12:49:11 2011
@@ -56,6 +56,7 @@ public class Authorizations implements I
     
     validAuthChars['_'] = true;
     validAuthChars['-'] = true;
+    validAuthChars[':'] = true;
   }
   
   static final boolean isValidAuthChar(byte b) {

Modified: incubator/accumulo/branches/1.3/src/core/src/test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java
URL: http://svn.apache.org/viewvc/incubator/accumulo/branches/1.3/src/core/src/test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java?rev=1190280&r1=1190279&r2=1190280&view=diff
==============================================================================
--- incubator/accumulo/branches/1.3/src/core/src/test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java
(original)
+++ incubator/accumulo/branches/1.3/src/core/src/test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java
Fri Oct 28 12:49:11 2011
@@ -1,8 +1,8 @@
 package org.apache.accumulo.core.security;
 
-import static org.junit.Assert.*;
+import static org.junit.Assert.assertArrayEquals;
+import static org.junit.Assert.fail;
 
-import org.apache.accumulo.core.security.ColumnVisibility;
 import org.junit.Test;
 
 public class ColumnVisibilityTest {
@@ -39,6 +39,7 @@ public class ColumnVisibilityTest {
   public void testCompound() {
     shouldNotThrow("a|b", "a&b", "ab&bc");
     shouldNotThrow("A&B&C&D&E", "A|B|C|D|E", "(A|B|C)", "(A)|B|(C)", "A&(B)&(C)",
"A&B&(L)");
+    shouldNotThrow("_&-&:");
   }
   
   @Test



Mime
View raw message