diff --git a/axzo-log-api/src/main/java/cn/axzo/log/platform/client/model/req/LogAddReq.java b/axzo-log-api/src/main/java/cn/axzo/log/platform/client/model/req/LogAddReq.java index e9b41ef..f58f0a0 100644 --- a/axzo-log-api/src/main/java/cn/axzo/log/platform/client/model/req/LogAddReq.java +++ b/axzo-log-api/src/main/java/cn/axzo/log/platform/client/model/req/LogAddReq.java @@ -41,6 +41,5 @@ public class LogAddReq { /** * 日志内容,需要是json格式 */ - @NotBlank(message = "message is required") private String message; } diff --git a/axzo-log-server/src/main/java/cn/axzo/log/platform/server/config/ExceptionAdviceHandler.java b/axzo-log-server/src/main/java/cn/axzo/log/platform/server/config/ExceptionAdviceHandler.java index 1874f90..f9fb685 100644 --- a/axzo-log-server/src/main/java/cn/axzo/log/platform/server/config/ExceptionAdviceHandler.java +++ b/axzo-log-server/src/main/java/cn/axzo/log/platform/server/config/ExceptionAdviceHandler.java @@ -5,6 +5,7 @@ import cn.axzo.framework.domain.web.result.ApiResult; import lombok.extern.slf4j.Slf4j; import org.springframework.core.annotation.Order; +import org.springframework.web.bind.MethodArgumentNotValidException; import org.springframework.web.bind.annotation.ExceptionHandler; import org.springframework.web.bind.annotation.RestControllerAdvice; @@ -29,4 +30,10 @@ public class ExceptionAdviceHandler { log.warn("业务异常", e); return ApiResult.err(e.getMessage()); } + + @ExceptionHandler({MethodArgumentNotValidException.class}) + public ApiResult exceptionHandler(MethodArgumentNotValidException e) { + log.error("参数异常", e); + return ApiResult.err(e.getMessage()); + } } diff --git a/axzo-log-server/src/main/java/cn/axzo/log/platform/server/service/impl/LogServiceImpl.java b/axzo-log-server/src/main/java/cn/axzo/log/platform/server/service/impl/LogServiceImpl.java index 4c4bf11..63d1365 100644 --- a/axzo-log-server/src/main/java/cn/axzo/log/platform/server/service/impl/LogServiceImpl.java +++ b/axzo-log-server/src/main/java/cn/axzo/log/platform/server/service/impl/LogServiceImpl.java @@ -78,6 +78,9 @@ public class LogServiceImpl implements LogService { */ @Override public String addLog(LogAddReq req) { + if (StringUtils.isBlank(req.getMessage())) { + return ""; + } LogEntity logEntity = buildToLogEntity(req); logMapper.save(logEntity); return logEntity.getId(); @@ -145,8 +148,12 @@ public class LogServiceImpl implements LogService { */ @Override public List batchAddLogs(LogBatchAddReq req) { + if (req.getLogs().stream().allMatch(l -> StringUtils.isBlank(l.getMessage()))) { + return Collections.emptyList(); + } List logEntities = req.getLogs() .stream() + .filter(l -> StringUtils.isNotBlank(l.getMessage())) .map(this::buildToLogEntity) .collect(Collectors.toList()); logMapper.saveAll(logEntities);