aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPamela Dragosh <pdragosh@research.att.com>2020-03-27 11:25:29 -0400
committerPamela Dragosh <pdragosh@research.att.com>2020-03-27 11:25:34 -0400
commite6a1c6818c7b5040e6661dc9185396173b8f7991 (patch)
tree8598c22fb2e9d9b81d3d8a81f49d28649d06b34e
parent67d13b7e11677e3f2678e137471923222ec9b329 (diff)
Cleanup sonar bugs and security hotspot
Just some trivial sonar bugs for missing argument and a couple sonar overrides to clear security hotspots. Issue-ID: POLICY-2165 Change-Id: I392e5fee458465c66313a16a8912e23a3a119848 Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
-rw-r--r--models-interactions/model-actors/actor.cds/src/main/java/org/onap/policy/controlloop/actor/cds/CdsActorServiceProvider.java2
-rw-r--r--models-interactions/model-impl/aai/src/main/java/org/onap/policy/aai/AaiManager.java2
-rw-r--r--models-sim/policy-models-sim-pdp/src/main/java/org/onap/policy/models/sim/pdp/PdpSimulatorMain.java6
-rw-r--r--models-sim/policy-models-simulators/src/main/java/org/onap/policy/models/simulators/Main.java4
4 files changed, 7 insertions, 7 deletions
diff --git a/models-interactions/model-actors/actor.cds/src/main/java/org/onap/policy/controlloop/actor/cds/CdsActorServiceProvider.java b/models-interactions/model-actors/actor.cds/src/main/java/org/onap/policy/controlloop/actor/cds/CdsActorServiceProvider.java
index 0b0ae6bfc..e3b91f2c4 100644
--- a/models-interactions/model-actors/actor.cds/src/main/java/org/onap/policy/controlloop/actor/cds/CdsActorServiceProvider.java
+++ b/models-interactions/model-actors/actor.cds/src/main/java/org/onap/policy/controlloop/actor/cds/CdsActorServiceProvider.java
@@ -231,7 +231,7 @@ public class CdsActorServiceProvider extends ActorImpl {
public void onError(final Throwable throwable) {
Status status = Status.fromThrowable(throwable);
cdsStatus.compareAndSet(null, CdsActorConstants.ERROR);
- LOGGER.error("Failed processing blueprint {} {}", status, throwable);
+ LOGGER.error("Failed processing blueprint {}", status, throwable);
}
/**
diff --git a/models-interactions/model-impl/aai/src/main/java/org/onap/policy/aai/AaiManager.java b/models-interactions/model-impl/aai/src/main/java/org/onap/policy/aai/AaiManager.java
index 41f4d3b90..24dcd8bbd 100644
--- a/models-interactions/model-impl/aai/src/main/java/org/onap/policy/aai/AaiManager.java
+++ b/models-interactions/model-impl/aai/src/main/java/org/onap/policy/aai/AaiManager.java
@@ -253,7 +253,7 @@ public final class AaiManager {
urlGet = url + PNF_URL;
pnfName = URLEncoder.encode(pnfName, StandardCharsets.UTF_8.toString()) + AAI_DEPTH_SUFFIX;
} catch (UnsupportedEncodingException e) {
- logger.error("Failed to encode the pnfName: {} using UTF-8 encoding. {}", pnfName, e);
+ logger.error("Failed to encode the pnfName: {} using UTF-8", pnfName, e);
return null;
}
String responseGet = getStringQuery(urlGet, username, password, requestId, pnfName);
diff --git a/models-sim/policy-models-sim-pdp/src/main/java/org/onap/policy/models/sim/pdp/PdpSimulatorMain.java b/models-sim/policy-models-sim-pdp/src/main/java/org/onap/policy/models/sim/pdp/PdpSimulatorMain.java
index e367cca41..99bf20e48 100644
--- a/models-sim/policy-models-sim-pdp/src/main/java/org/onap/policy/models/sim/pdp/PdpSimulatorMain.java
+++ b/models-sim/policy-models-sim-pdp/src/main/java/org/onap/policy/models/sim/pdp/PdpSimulatorMain.java
@@ -1,7 +1,7 @@
/*-
* ============LICENSE_START=======================================================
* Copyright (C) 2019 Nordix Foundation.
- * Modifications Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -143,12 +143,12 @@ public class PdpSimulatorMain {
}
/**
- * The main method.
+ * The main method. Arguments are validated in the constructor thus adding the NOSONAR.
*
* @param args the arguments
*
*/
- public static void main(final String[] args) {
+ public static void main(final String[] args) { // NOSONAR
new PdpSimulatorMain(args);
}
}
diff --git a/models-sim/policy-models-simulators/src/main/java/org/onap/policy/models/simulators/Main.java b/models-sim/policy-models-simulators/src/main/java/org/onap/policy/models/simulators/Main.java
index a0b165564..d161e545a 100644
--- a/models-sim/policy-models-simulators/src/main/java/org/onap/policy/models/simulators/Main.java
+++ b/models-sim/policy-models-simulators/src/main/java/org/onap/policy/models/simulators/Main.java
@@ -127,11 +127,11 @@ public class Main extends ServiceManagerContainer {
}
/**
- * The main method.
+ * The main method. The arguments are validated, thus adding the NOSONAR.
*
* @param args the arguments, the first of which is the name of the parameter file
*/
- public static void main(final String[] args) {
+ public static void main(final String[] args) { // NOSONAR
try {
if (args.length != 1) {
throw new IllegalArgumentException("arg(s): parameter-file-name");