remove slash in rego rule not mapper

Author: adwk67

src/main/java/tech/stackable/hadoop/StackableGroupMapper.java

@@ -14,7 +14,6 @@
 import java.util.Objects;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.security.GroupMappingServiceProvider;
-import org.apache.hadoop.util.Lists;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -114,32 +113,17 @@ public List<String> getGroups(String user) throws IOException {
 
     String responseBody = response.body();
     LOG.debug("Response body [{}]", responseBody);
-    List<String> groups = Lists.newArrayList();
 
     @SuppressWarnings("unchecked")
     Map<String, Object> result =
         (Map<String, Object>) json.readValue(responseBody, HashMap.class).get(OPA_RESULT_FIELD);
-    List<String> rawGroups = (List<String>) result.get(this.mappingGroupName);
-
-    for (String rawGroup : rawGroups) {
-      groups.add(stripSlashes(rawGroup));
-    }
+    List<String> groups = (List<String>) result.get(this.mappingGroupName);
 
     LOG.info("Groups for [{}]: [{}]", user, groups);
 
     return groups;
   }
 
-  private static String stripSlashes(String s) {
-    if (s.startsWith("/")) {
-      s = s.substring(1);
-    }
-    if (s.endsWith("/")) {
-      s = s.substring(0, s.length() - 1);
-    }
-    return s;
-  }
-
   /** Caches groups, no need to do that for this provider */
   @Override
   public void cacheGroupsRefresh() {

src/test/java/tech/stackable/hadoop/StackableGroupMapperTest.java

@@ -19,8 +19,8 @@ public void testJsonMapper() throws JsonProcessingException {
         "{\n"
             + "    \"result\": {\n"
             + "          \"groups\": [\n"
-            + "              \"/admin\",\n"
-            + "              \"/superuser\"\n"
+            + "              \"admin\",\n"
+            + "              \"superuser\"\n"
             + "          ]\n"
             + "        ,\n"
             + "        \"users_by_name\": {\n"
@@ -61,7 +61,7 @@ public void testJsonMapper() throws JsonProcessingException {
     Map<String, List<String>> result =
         (Map<String, List<String>>) json.readValue(input, HashMap.class).get("result");
     List<String> groups = result.get("groups");
-    Assert.assertEquals("/admin", groups.get(0));
-    Assert.assertEquals("/superuser", groups.get(1));
+    Assert.assertEquals("admin", groups.get(0));
+    Assert.assertEquals("superuser", groups.get(1));
   }
 }

test/stack/05-opa.yaml

@@ -12,11 +12,14 @@ data:
     # this will return the group data in this form:
     #    "result": {
     #        "groups": [
-    #            "/admin",
-    #            "/superuser"
+    #            "admin",
+    #            "superuser"
     #        ]
     #        ...
-    groups := users_by_name[input.username].groups
+    groups := {g |
+      raw = users_by_name[input.username].groups[_]
+      g := trim(raw, "/")
+    }
 
     # returning data in the form presented by the UIF
     users_by_name := {