Sonar critical issues 65/42665/1
authorSurendra Reddy Katam <SK00524980@techmahindra.com>
Fri, 13 Apr 2018 06:20:46 +0000 (11:50 +0530)
committerSurendra Reddy Katam <SK00524980@techmahindra.com>
Fri, 13 Apr 2018 06:20:46 +0000 (11:50 +0530)
 Either log or rethrow this exception
Sonar Link:
https://sonar.onap.org/project/issues?assignees=surendraReddy&id=org.onap.dmaap.messagerouter.msgrtr%3Amsgrtr&open=AV4-Vahb32hFUzlqc5uD&resolved=false&severities=CRITICAL
Location:
src/main/java/com/att/nsa/cambria/resources/streamReaders/CambriaJsonStreamReader.java
Line No-111

Change-Id: I98b083ebc2410e3da5a15d52d4917311b33d10f7
Issue-ID: DMAAP-408
Signed-off-by: Surendra Reddy Katam <SK00524980@techmahindra.com>
src/main/java/com/att/nsa/cambria/resources/streamReaders/CambriaJsonStreamReader.java

index 9d727ad..54a9bc6 100644 (file)
@@ -22,6 +22,7 @@
 package com.att.nsa.cambria.resources.streamReaders;
 
 import java.io.InputStream;
+import java.util.logging.Logger;
 
 import javax.servlet.http.HttpServletResponse;
 
@@ -34,14 +35,17 @@ import com.att.nsa.cambria.backends.Publisher.message;
 import com.att.nsa.cambria.beans.LogDetails;
 import com.att.nsa.cambria.resources.CambriaEventSet.reader;
 
+import jline.internal.Log;
+
 /**
  * 
  * @author author
  *
  */
 public class CambriaJsonStreamReader implements reader {
+       private static final Logger LOG = Logger.getLogger(CambriaJsonStreamReader.class.toString());
        private final JSONTokener fTokens;
-       private final boolean fIsList;
+       private final boolean fIsList;          
        private long fCount;
        private final String fDefPart;
        public static final String kKeyField = "cambria.partition";
@@ -109,6 +113,7 @@ public class CambriaJsonStreamReader implements reader {
                        fCount++;
                        return new msg(o);
                } catch (JSONException e) {
+                       Log.error(e.toString());
                        throw new CambriaApiException(HttpServletResponse.SC_BAD_REQUEST, e.getMessage());
 
                }