fix policy mapper to replace "-" with "_", and clean up error logging

aserto-dev · Jun 21, 2024 · 8ee7436 · 8ee7436
1 parent 0b35c2a
commit 8ee7436
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/main/java/com/aserto/authroizer/AsertoAuthorizationManager.java b/src/main/java/com/aserto/authroizer/AsertoAuthorizationManager.java
@@ -118,7 +118,7 @@ public AuthorizationDecision check(HttpServletRequest httpRequest, IdentityMappe
             List<Decision> decisions = authzClient.is(identityCtx, policyCtx, resourceCtx);
             isAllowed = isAllowed(decisions);
         } catch (Exception e) {
-            log.error("Is call failed [{}]",  e.getMessage());
+            log.error("Is call failed",  e);
             return new AuthorizationDecision(false);
         }
 

diff --git a/src/main/java/com/aserto/authroizer/mapper/policy/HttpPathPolicyMapper.java b/src/main/java/com/aserto/authroizer/mapper/policy/HttpPathPolicyMapper.java
@@ -39,6 +39,7 @@ public String policyPath(HttpServletRequest request) {
             if (tokens[i].startsWith("{") && tokens[i].endsWith("}")) {
                 tokens[i] = "__" + tokens[i].substring(1, tokens[i].length() - 1);
             }
+            tokens[i] = tokens[i].replaceAll("-", "_");
         }
 
         return policyRoot + "." + method.toUpperCase()  + "." + String.join(".", tokens);