accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubb...@apache.org
Subject [accumulo-testing] branch master updated: Fixes issue #114, incorrect auths used by scanners and walkers (#115)
Date Mon, 07 Oct 2019 23:44:16 GMT
This is an automated email from the ASF dual-hosted git repository.

ctubbsii pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/accumulo-testing.git


The following commit(s) were added to refs/heads/master by this push:
     new ee472db  Fixes issue #114, incorrect auths used by scanners and walkers (#115)
ee472db is described below

commit ee472db0f47dbaaabd89605e3463f5983ef965fd
Author: Mandar Inamdar <mandarinamdar@users.noreply.github.com>
AuthorDate: Mon Oct 7 16:44:11 2019 -0700

    Fixes issue #114, incorrect auths used by scanners and walkers (#115)
    
    Fixed authValue.split("|") call in ContinuousEnv.java.
    It was not considering | as a metacharacter in regex used by split
    added escape character \\ in the split function call as authValue.split("\\|")
---
 src/main/java/org/apache/accumulo/testing/continuous/ContinuousEnv.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/accumulo/testing/continuous/ContinuousEnv.java b/src/main/java/org/apache/accumulo/testing/continuous/ContinuousEnv.java
index 0912475..309379a 100644
--- a/src/main/java/org/apache/accumulo/testing/continuous/ContinuousEnv.java
+++ b/src/main/java/org/apache/accumulo/testing/continuous/ContinuousEnv.java
@@ -27,7 +27,7 @@ public class ContinuousEnv extends TestEnv {
         authList = Collections.singletonList(Authorizations.EMPTY);
       } else {
         authList = new ArrayList<>();
-        for (String a : authValue.split("|")) {
+        for (String a : authValue.split("\\|")) {
           authList.add(new Authorizations(a.split(",")));
         }
       }


Mime
View raw message