From: Arundathi Patil <arundpil@in.ibm.com>
Date: Tue, 4 Sep 2018 09:32:23 +0000 (+0530)
Subject: Authorization.java: Fixed sonar issue
X-Git-Tag: 1.0.16~24
X-Git-Url: https://gerrit.onap.org/r/gitweb?p=dmaap%2Fdbcapi.git;a=commitdiff_plain;h=c399b663374b13918dacd58812eeb5b7c834dc2f

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 <arundpil@in.ibm.com>
---

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;
 		}