diff options
author | 2018-09-04 15:02:23 +0530 | |
---|---|---|
committer | 2018-09-04 15:02:43 +0530 | |
commit | c399b663374b13918dacd58812eeb5b7c834dc2f (patch) | |
tree | 8d6641ce2188bbb5141c14b4e8033f979e8b912f | |
parent | 8b80c9a3005862032908478570f9c500bf0d9fb5 (diff) |
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>
-rw-r--r-- | src/main/java/org/onap/dmaap/dbcapi/resources/AuthorizationFilter.java | 6 |
1 files 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; } |