From 8ee74362f39c2425d2c00890c290965194f6eee8 Mon Sep 17 00:00:00 2001 From: Omri Gazitt Date: Fri, 21 Jun 2024 16:53:17 -0700 Subject: [PATCH] fix policy mapper to replace "-" with "_", and clean up error logging --- .../java/com/aserto/authroizer/AsertoAuthorizationManager.java | 2 +- .../aserto/authroizer/mapper/policy/HttpPathPolicyMapper.java | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/java/com/aserto/authroizer/AsertoAuthorizationManager.java b/src/main/java/com/aserto/authroizer/AsertoAuthorizationManager.java index 0bfeecf..3496e3f 100644 --- 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 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 index e872d18..aea8f85 100644 --- 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);