From c399b663374b13918dacd58812eeb5b7c834dc2f Mon Sep 17 00:00:00 2001 From: Arundathi Patil Date: Tue, 4 Sep 2018 15:02:23 +0530 Subject: [PATCH] Authorization.java: Fixed sonar issue Fixed sonar code-smells/issues across this file. Issue-ID: DMAAP-712 Change-Id: I13683116ca5442042f6edc73a1ba1e29a5e49e01 Signed-off-by: Arundathi Patil --- .../java/org/onap/dmaap/dbcapi/resources/AuthorizationFilter.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/onap/dmaap/dbcapi/resources/AuthorizationFilter.java b/src/main/java/org/onap/dmaap/dbcapi/resources/AuthorizationFilter.java index ab0e262..9dafb55 100644 --- a/src/main/java/org/onap/dmaap/dbcapi/resources/AuthorizationFilter.java +++ b/src/main/java/org/onap/dmaap/dbcapi/resources/AuthorizationFilter.java @@ -25,15 +25,15 @@ import java.io.IOException; import javax.ws.rs.container.ContainerRequestContext; import javax.ws.rs.container.ContainerRequestFilter; +import org.apache.log4j.Logger; import org.onap.dmaap.dbcapi.authentication.AuthenticationErrorException; import org.onap.dmaap.dbcapi.service.ApiService; -import org.onap.dmaap.dbcapi.util.DmaapConfig; @Authorization public class AuthorizationFilter implements ContainerRequestFilter { - + private Logger logger = Logger.getLogger(AuthorizationFilter.class.getName()); @Override public void filter(ContainerRequestContext requestContext) @@ -48,9 +48,11 @@ public class AuthorizationFilter implements ContainerRequestFilter { try { apiResp.checkAuthorization(); } catch ( AuthenticationErrorException ae ) { + logger.error("Error", ae); requestContext.abortWith( apiResp.unauthorized( apiResp.getErr().getMessage() ) ); return ; } catch ( Exception e ) { + logger.error("Error", e); requestContext.abortWith( apiResp.unavailable() ); return; } -- 2.16.6