Code Review
/
policy
/
engine.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Fixes for sonar critical issues
[policy/engine.git]
/
BRMSGateway
/
src
/
main
/
java
/
org
/
onap
/
policy
/
brmsInterface
/
BRMSHandler.java
diff --git
a/BRMSGateway/src/main/java/org/onap/policy/brmsInterface/BRMSHandler.java
b/BRMSGateway/src/main/java/org/onap/policy/brmsInterface/BRMSHandler.java
index
305f64d
..
4303186
100644
(file)
--- a/
BRMSGateway/src/main/java/org/onap/policy/brmsInterface/BRMSHandler.java
+++ b/
BRMSGateway/src/main/java/org/onap/policy/brmsInterface/BRMSHandler.java
@@
-91,7
+91,7
@@
public class BRMSHandler implements BackUpHandler{
bRMSPush.removeRule(removedPolicy.getPolicyName());
removed = true;
}catch(Exception e){
bRMSPush.removeRule(removedPolicy.getPolicyName());
removed = true;
}catch(Exception e){
- logger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW+"Rertriving policy failed " + e.getMessage());
+ logger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW+"Rertriving policy failed " + e.getMessage()
, e
);
}
}
}
}
}
}
@@
-106,6
+106,7
@@
public class BRMSHandler implements BackUpHandler{
//Upon Notification failure
failureFlag = true;
bRMSPush.rotateURLs();
//Upon Notification failure
failureFlag = true;
bRMSPush.rotateURLs();
+ logger.error("Failure during Push Operation " , e);
}
}
i++;
}
}
i++;
@@
-138,6
+139,7
@@
public class BRMSHandler implements BackUpHandler{
//Upon Notification failure
successFlag = false;
bRMSPush.rotateURLs();
//Upon Notification failure
successFlag = false;
bRMSPush.rotateURLs();
+ logger.error("Failure during Push Operation " , e);
}
}
}
}
}
}
@@
-171,7
+173,7
@@
public class BRMSHandler implements BackUpHandler{
}
}
}catch(Exception e){
}
}
}catch(Exception e){
- logger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW+"Rertriving policy failed " + e.getMessage());
+ logger.error(XACMLErrorConstants.ERROR_PROCESS_FLOW+"Rertriving policy failed " + e.getMessage()
, e
);
}
}
}
}
}
}