aboutsummaryrefslogtreecommitdiffstats
path: root/common-app-api/src/main/java/org/openecomp/sdc/common/config/EcompErrorLogUtil.java
diff options
context:
space:
mode:
Diffstat (limited to 'common-app-api/src/main/java/org/openecomp/sdc/common/config/EcompErrorLogUtil.java')
-rw-r--r--common-app-api/src/main/java/org/openecomp/sdc/common/config/EcompErrorLogUtil.java68
1 files changed, 18 insertions, 50 deletions
diff --git a/common-app-api/src/main/java/org/openecomp/sdc/common/config/EcompErrorLogUtil.java b/common-app-api/src/main/java/org/openecomp/sdc/common/config/EcompErrorLogUtil.java
index 950af691da..af53611104 100644
--- a/common-app-api/src/main/java/org/openecomp/sdc/common/config/EcompErrorLogUtil.java
+++ b/common-app-api/src/main/java/org/openecomp/sdc/common/config/EcompErrorLogUtil.java
@@ -20,22 +20,21 @@
package org.openecomp.sdc.common.config;
-import java.util.Formatter;
-import java.util.IllegalFormatException;
-import java.util.Locale;
-
+import fj.data.Either;
import org.apache.commons.lang3.StringUtils;
import org.openecomp.sdc.common.config.EcompErrorConfiguration.EcompErrorSeverity;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import org.openecomp.sdc.common.log.enums.EcompLoggerErrorCode;
+import org.openecomp.sdc.common.log.wrappers.Logger;
import org.slf4j.MDC;
-import fj.data.Either;
+import java.util.Formatter;
+import java.util.IllegalFormatException;
+import java.util.Locale;
public class EcompErrorLogUtil {
private static String ECOMP_ERROR_TMPL = "ETYPE = \"%s\" ENAME = \"%s\" ECODE = \"%s\" ECONTEXT = \"%s\" EDESC = \"%s\"";
- private static Logger log = LoggerFactory.getLogger(EcompErrorLogUtil.class.getName());
+ private static Logger log = Logger.getLogger(EcompErrorLogUtil.class.getName());
private static final String FATAL_ERROR_PREFIX = "FATAL ERROR!! ";
public static void logEcompError(EcompErrorName ecompErrorName, EcompErrorInfo ecompErrorInfo,
@@ -62,24 +61,9 @@ public class EcompErrorLogUtil {
// else it stays ERROR
formatter.format(ECOMP_ERROR_TMPL, ecompErrorInfo.getType(), ecompErrorName.name(),
ecompErrorInfo.getCode(), ecompErrorContext, description);
- switch (severity) {
- case INFO:
- log.info(formatter.toString());
- break;
- case WARN:
- log.warn(formatter.toString());
- break;
- case ERROR:
- log.error(formatter.toString());
- break;
- case FATAL:
- // same as ERROR for now, might be additional logic later..
- log.error(formatter.toString());
- break;
- default:
- break;
- }
- } finally {
+ log.error(severity, EcompLoggerErrorCode.getByValue(ecompErrorInfo.getCode()),
+ ecompErrorContext, ecompErrorContext, description);
+ } finally {
formatter.close();
MDC.remove("alarmSeverity");
}
@@ -96,7 +80,7 @@ public class EcompErrorLogUtil {
StringBuilder sb = new StringBuilder();
Formatter formatter = new Formatter(sb, Locale.US);
try {
- String description = ecompErrorEnum.getEcompErrorCode().getDescription();
+ String description;
Either<String, Boolean> setDescriptionParamsResult = setDescriptionParams(ecompErrorEnum,
ecompDescriptionParams);
@@ -104,13 +88,12 @@ public class EcompErrorLogUtil {
description = setDescriptionParamsResult.left().value();
} else {
EcompErrorEnum mismatchErrorEnum = EcompErrorEnum.EcompMismatchParam;
- if (logMissingParams == true) {
- logEcompError("logEcompError", mismatchErrorEnum, false, ecompErrorEnum.name().toString());
- return;
+ if (logMissingParams) {
+ logEcompError("logEcompError", mismatchErrorEnum, false, ecompErrorEnum.name());
} else {
log.info("Failed to log the error code {}", mismatchErrorEnum);
- return;
}
+ return;
}
EcompClassification classification = ecompErrorEnum.getClassification();
@@ -132,23 +115,9 @@ public class EcompErrorLogUtil {
// else it stays ERROR
formatter.format(ECOMP_ERROR_TMPL, ecompErrorEnum.geteType(), ecompErrorEnum.name(), eCode,
ecompErrorContext, description);
- switch (severity) {
- case INFO:
- log.info(formatter.toString());
- break;
- case WARN:
- log.warn(formatter.toString());
- break;
- case ERROR:
- log.error(formatter.toString());
- break;
- case FATAL:
- // same as ERROR for now, might be additional logic later..
- log.error(formatter.toString());
- break;
- default:
- break;
- }
+
+ log.error(severity, EcompLoggerErrorCode.getByValue(ecompErrorEnum.getEcompErrorCode().name()),
+ ecompErrorContext, ecompErrorContext, description);
} finally {
formatter.close();
MDC.remove("alarmSeverity");
@@ -161,9 +130,8 @@ public class EcompErrorLogUtil {
String ecompErrorCode = ecompErrorEnum.getEcompErrorCode().name();
String ecodeNumber = ecompErrorCode.substring(ecompErrorCode.indexOf("_") + 1);
- String eCode = "ASDC" + ecodeNumber + classification.getClassification();
- return eCode;
+ return "ASDC" + ecodeNumber + classification.getClassification();
}
private static Either<String, Boolean> setDescriptionParams(EcompErrorEnum ecompErrorEnum,