From: Guangrong Fu <fu.guangrong@zte.com.cn>
Date: Thu, 27 Oct 2022 02:52:55 +0000 (+0800)
Subject: bugfix - response message error
X-Git-Tag: 11.0.1~1
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=2fd6a2e5d048978e347835b3262643b224e5348b;p=holmes%2Frule-management.git

bugfix - response message error

Issue-ID: HOLMES-583
Signed-off-by: Guangrong Fu <fu.guangrong@zte.com.cn>
Change-Id: Ida28cec4c1ae5a8c0560b3d4db9ab1647216f021
---

diff --git a/rulemgt/src/main/java/org/onap/holmes/rulemgt/wrapper/RuleMgtWrapper.java b/rulemgt/src/main/java/org/onap/holmes/rulemgt/wrapper/RuleMgtWrapper.java
index ec6e9b3..0d5d110 100644
--- a/rulemgt/src/main/java/org/onap/holmes/rulemgt/wrapper/RuleMgtWrapper.java
+++ b/rulemgt/src/main/java/org/onap/holmes/rulemgt/wrapper/RuleMgtWrapper.java
@@ -16,6 +16,7 @@
 package org.onap.holmes.rulemgt.wrapper;
 
 import lombok.extern.slf4j.Slf4j;
+import org.apache.commons.lang3.StringUtils;
 import org.onap.holmes.common.api.entity.CorrelationRule;
 import org.onap.holmes.common.exception.CorrelationException;
 import org.onap.holmes.rulemgt.bean.request.*;
@@ -79,7 +80,9 @@ public class RuleMgtWrapper {
         try {
             result = correlationRuleService.saveRule(correlationRule);
         } catch (CorrelationException e) {
-            engineWarpper.deleteRuleFromEngine(packageName, ip);
+            if (StringUtils.isNotEmpty(packageName)) {
+                engineWarpper.deleteRuleFromEngine(packageName, ip);
+            }
             throw new CorrelationException(e.getMessage(), e);
         }
         RuleAddAndUpdateResponse ruleAddAndUpdateResponse = new RuleAddAndUpdateResponse();
diff --git a/rulemgt/src/main/resources/logback-spring.xml b/rulemgt/src/main/resources/logback-spring.xml
index eda9c6b..db95b38 100644
--- a/rulemgt/src/main/resources/logback-spring.xml
+++ b/rulemgt/src/main/resources/logback-spring.xml
@@ -9,7 +9,7 @@
             <charset>UTF-8</charset>
         </encoder>
         <filter class="ch.qos.logback.classic.filter.ThresholdFilter">
-            <level>debug</level>
+            <level>info</level>
         </filter>
     </appender>
 
@@ -46,8 +46,8 @@
         </encoder>
         <filter class="ch.qos.logback.classic.filter.LevelFilter">
             <level>error</level>
-            <onMatch>deny</onMatch>
-            <onMismatch>accept</onMismatch>
+            <onMatch>accept</onMatch>
+            <onMismatch>deny</onMismatch>
         </filter>
     </appender>