diff options
| author |   shashikanth <shashikanth.vh@huawei.com> | 2017-09-21 15:09:59 +0530 |
|---|---|---|
| committer | shashikanth <shashikanth.vh@huawei.com> | 2017-09-21 15:10:00 +0530 |
| commit | d320e47f93c3a657debab801da9b437c45f56e9f (patch) | |
| tree | ad456c40d8ca36cd0db4e98bd3ddba3bfcad461f | |
| parent | 70ac9a27664224d40cee2e08be878a5cecb925cc (diff) | |
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 <shashikanth.vh@huawei.com>
| -rw-r--r-- | aai-traversal/src/main/java/org/openecomp/aai/interceptors/AAILogJAXRSInInterceptor.java | 4 |
1 files 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"); |