diff options
| author |   Jorge Hernandez <jh1730@att.com> | 2018-09-19 14:22:28 -0500 |
|---|---|---|
| committer | Jorge Hernandez <jh1730@att.com> | 2018-09-19 14:22:28 -0500 |
| commit | ea37c0eb6f4f93596135035b0e34f29599735c32 (patch) | |
| tree | f070970bd9ee02c44c474cdcee224cf1bf5ebf44 /policy-endpoints | |
| parent | 2f53e137769ec9e658bc7e16a7893c5c35482032 (diff) | |
tweaks from testing in auth classes
Change-Id: Idb1346ca645ffcc55808e25c8957e01d27a48a75
Signed-off-by: Jorge Hernandez <jh1730@att.com>
Issue-ID: POLICY-1043
Diffstat (limited to 'policy-endpoints')
3 files changed, 3 insertions, 4 deletions
diff --git a/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/http/server/AuthorizationFilter.java b/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/http/server/AuthorizationFilter.java index cb58371f..d884b869 100644 --- a/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/http/server/AuthorizationFilter.java +++ b/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/http/server/AuthorizationFilter.java @@ -54,7 +54,7 @@ public abstract class AuthorizationFilter implements Filter { boolean authorized = request.isUserInRole(role); logger.info("user {} in role {} is {}authorized to {}", - request.getUserPrincipal().getName(), role, ((authorized) ? "" : "NOT "), request.getMethod()); + request.getUserPrincipal(), role, ((authorized) ? "" : "NOT "), request.getMethod()); if (!authorized) { response.setStatus(HttpServletResponse.SC_FORBIDDEN); diff --git a/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/http/server/aaf/AafGranularAuthFilter.java b/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/http/server/aaf/AafGranularAuthFilter.java index 42df2484..27b15a9c 100644 --- a/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/http/server/aaf/AafGranularAuthFilter.java +++ b/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/http/server/aaf/AafGranularAuthFilter.java @@ -36,8 +36,7 @@ public abstract class AafGranularAuthFilter extends AafAuthFilter { @Override protected String getPermissionType(HttpServletRequest request) { - return getPermissionTypeRoot() + "." + - request.getRequestURI().replace('/', '.'); + return getPermissionTypeRoot() + request.getRequestURI().replace('/', '.'); } @Override diff --git a/policy-endpoints/src/test/java/org/onap/policy/common/endpoints/http/server/test/TestAafGranularAuthFilter.java b/policy-endpoints/src/test/java/org/onap/policy/common/endpoints/http/server/test/TestAafGranularAuthFilter.java index cad6fb5c..183e5ae7 100644 --- a/policy-endpoints/src/test/java/org/onap/policy/common/endpoints/http/server/test/TestAafGranularAuthFilter.java +++ b/policy-endpoints/src/test/java/org/onap/policy/common/endpoints/http/server/test/TestAafGranularAuthFilter.java @@ -28,7 +28,7 @@ public class TestAafGranularAuthFilter extends AafGranularAuthFilter { @Override protected String getRole(HttpServletRequest request) { - String expectedPerm = this.getPermissionTypeRoot() + "." + String expectedPerm = this.getPermissionTypeRoot() + request.getRequestURI().replace('/', '.') + "|" + NetworkUtil.getHostname() + "|" + request.getMethod().toLowerCase(); |