summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGaurav Agrawal <gaurav.agrawal@huawei.com>2017-09-08 19:58:55 +0530
committerGaurav Agrawal <gaurav.agrawal@huawei.com>2017-09-08 21:08:12 +0530
commitc78bab01fa4667e840aad907ee37fa67b4fe8239 (patch)
treec4fd539536559c0da9dcf26e44918b36e452cfa6
parentea0aabe54bc7b5d609b587e87ce85765f73fecbc (diff)
Fix few Blocker sonar issues
Fix few Blocker sonar issues in CCSDK SLI Northbound https://sonar.onap.org/component_issues/index?id=org.onap.ccsdk.sli.northbound%3Accsdk-sli-northbound#resolved=false|severities=BLOCKER Change-Id: I7229c7e2a0dabc0ff789b91ae413b9a15ef3cbad Issue-Id: CCSDK-67 Signed-off-by: Gaurav Agrawal <gaurav.agrawal@huawei.com>
-rw-r--r--ueb-listener/src/main/java/org/onap/ccsdk/sli/northbound/uebclient/SdncUebCallback.java18
1 files changed, 4 insertions, 14 deletions
diff --git a/ueb-listener/src/main/java/org/onap/ccsdk/sli/northbound/uebclient/SdncUebCallback.java b/ueb-listener/src/main/java/org/onap/ccsdk/sli/northbound/uebclient/SdncUebCallback.java
index 73e8fe55b..e7a42bcb8 100644
--- a/ueb-listener/src/main/java/org/onap/ccsdk/sli/northbound/uebclient/SdncUebCallback.java
+++ b/ueb-listener/src/main/java/org/onap/ccsdk/sli/northbound/uebclient/SdncUebCallback.java
@@ -208,12 +208,8 @@ public class SdncUebCallback implements INotificationCallback {
Properties props = new Properties();
props.load(new FileInputStream(propFile));
- try {
- jdbcDataSource = new DBResourceManager(props);
- } catch(Throwable exc) {
- LOG.error("", exc);
- }
-
+ jdbcDataSource = new DBResourceManager(props);
+
if(((DBResourceManager)jdbcDataSource).isActive()){
LOG.warn( "DBLIB: JDBC DataSource has been initialized.");
} else {
@@ -405,8 +401,7 @@ public class SdncUebCallback implements INotificationCallback {
boolean writeSucceeded = false;
- try {
- FileWriter spoolFileWriter = new FileWriter(spoolFile);
+ try (FileWriter spoolFileWriter = new FileWriter(spoolFile)) {
spoolFileWriter.write(payload);
spoolFileWriter.close();
writeSucceeded = true;
@@ -916,11 +911,8 @@ public class SdncUebCallback implements INotificationCallback {
msgBuffer.append("<artifact-version>"+artifact.getArtifactVersion()+"</artifact-version>\n");
- try {
- BufferedReader rdr = new BufferedReader(new FileReader(artifact.getFile()));
-
+ try (BufferedReader rdr = new BufferedReader(new FileReader(artifact.getFile()))){
String curLine = rdr.readLine();
-
while (curLine != null) {
if (!curLine.startsWith("<?")) {
@@ -928,8 +920,6 @@ public class SdncUebCallback implements INotificationCallback {
}
curLine = rdr.readLine();
}
- rdr.close();
-
} catch (Exception e) {
LOG.error("Could not process spool file "+artifact.getFile().getName(), e);
return(DistributionStatusEnum.DEPLOY_ERROR);