diff options
author | Michael Mokry <michael.mokry@att.com> | 2019-04-15 12:47:46 -0500 |
---|---|---|
committer | Joshua Reich <jreich@research.att.com> | 2019-04-15 14:20:11 -0700 |
commit | b8f82899e1d5777fc4db912d1dc4030aa7e6d99d (patch) | |
tree | 2a35c23cacbecb5324905e2127e7e223fb63b533 /applications/common | |
parent | 7a0772a0f09b4aeef7b91cb04bdefaf49e7bdc38 (diff) |
Fix a couple of sonar issues one blocker
1. fixed nullpointer blocker issue in CoordinationGuardTranslator.java
2. fixed "log or rethrow exception" critical issue
3. throw exceptions instead of returning null values
Change-Id: I2d567fe566c6f761ec2699016c4a868f203c9a01
Issue-ID: POLICY-1451
Signed-off-by: Michael Mokry <michael.mokry@att.com>
Signed-off-by: Joshua Reich <jreich@research.att.com>
Diffstat (limited to 'applications/common')
-rw-r--r-- | applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdXacmlApplicationServiceProvider.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdXacmlApplicationServiceProvider.java b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdXacmlApplicationServiceProvider.java index 2b8048c4..451ef7e1 100644 --- a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdXacmlApplicationServiceProvider.java +++ b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdXacmlApplicationServiceProvider.java @@ -217,7 +217,7 @@ public abstract class StdXacmlApplicationServiceProvider implements XacmlApplica XacmlPolicyUtils.storeXacmlProperties(newProperties, XacmlPolicyUtils.getPropertiesPath(this.getDataPath())); } catch (IOException e) { - LOGGER.error("Failed to save the properties to disk {}", newProperties); + LOGGER.error("Failed to save the properties to disk {}", newProperties, e); } // // Reload the engine |