From d320e47f93c3a657debab801da9b437c45f56e9f Mon Sep 17 00:00:00 2001 From: shashikanth Date: Thu, 21 Sep 2017 15:09:59 +0530 Subject: Fix Blocker/Critical sonar issues Fix Major sonar issues in aai/traversal module https://sonar.onap.org/component_issues?id=org.onap.aai.traversal%3Atraversal#resolved=false|severities=BLOCKER Fixed NullPointerException might be thrown as 'uri' is nullable here Issue-Id:AAI-212 Change-Id: Ie4f47a47c3d54a4356aad787dcc10acbfcdd87ea Signed-off-by: shashikanth.vh --- .../java/org/openecomp/aai/interceptors/AAILogJAXRSInInterceptor.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/aai-traversal/src/main/java/org/openecomp/aai/interceptors/AAILogJAXRSInInterceptor.java b/aai-traversal/src/main/java/org/openecomp/aai/interceptors/AAILogJAXRSInInterceptor.java index a0ddeff..dd718de 100644 --- a/aai-traversal/src/main/java/org/openecomp/aai/interceptors/AAILogJAXRSInInterceptor.java +++ b/aai-traversal/src/main/java/org/openecomp/aai/interceptors/AAILogJAXRSInInterceptor.java @@ -94,7 +94,7 @@ public class AAILogJAXRSInInterceptor extends JAXRSInInterceptor { ErrorLogHelper.logException(e1); } - if (uri.contains(EchoResponse.echoPath)) { + if ((uri != null) && (uri.contains(EchoResponse.echoPath))) { // if it's a health check, we don't want to log ANYTHING if it's a lightweight one if (query == null) { if (message.getExchange().containsKey("AAI_LOGGING_HBASE_ENABLED")) { @@ -106,7 +106,7 @@ public class AAILogJAXRSInInterceptor extends JAXRSInInterceptor { go = false; } } - else if (uri.contains("/translog/")) { + else if ((uri != null) && (uri.contains("/translog/"))) { // if it's a translog query, we don't want to log the responses if (message.getExchange().containsKey("AAI_LOGGING_HBASE_LOGRESPONSE")) { message.getExchange().remove("AAI_LOGGING_HBASE_LOGRESPONSE"); -- cgit 1.2.3-korg