Authorization.java: Fixed sonar issue 77/64377/1
authorArundathi Patil <arundpil@in.ibm.com>
Tue, 4 Sep 2018 09:32:23 +0000 (15:02 +0530)
committerIBM602-PC0F1E3C\Arundathi <arundpil@in.ibm.com>
Tue, 4 Sep 2018 09:32:43 +0000 (15:02 +0530)
Fixed sonar code-smells/issues across this file.

Issue-ID: DMAAP-712
Change-Id: I13683116ca5442042f6edc73a1ba1e29a5e49e01
Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
src/main/java/org/onap/dmaap/dbcapi/resources/AuthorizationFilter.java

index ab0e262..9dafb55 100644 (file)
@@ -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;
                }