From 1e13d9808a9d300da116119a915f26c16dc10be7 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Jaroslav=20T=C3=B3th?= Date: Fri, 22 May 2020 13:42:22 +0200 Subject: [PATCH] Fixing errors check in RestconfDocumentedExceptionMapper MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit - Added check for number of errors that should not equal to 0; otherwise warning logs are generated. Change-Id: I5e173d6389c0b17b278aae8abb0d585cb45e21d1 Signed-off-by: Jaroslav Tóth --- .../providers/errors/RestconfDocumentedExceptionMapper.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/jersey/providers/errors/RestconfDocumentedExceptionMapper.java b/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/jersey/providers/errors/RestconfDocumentedExceptionMapper.java index b5d713f5ec..2090bdac73 100644 --- a/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/jersey/providers/errors/RestconfDocumentedExceptionMapper.java +++ b/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/jersey/providers/errors/RestconfDocumentedExceptionMapper.java @@ -90,7 +90,8 @@ public final class RestconfDocumentedExceptionMapper implements ExceptionMapper< LOG.debug("Starting to map received exception to error response: {}", exception.getMessage()); final Status responseStatus = getResponseStatusCode(exception); if (responseStatus != Response.Status.FORBIDDEN - && responseStatus.getFamily() == Response.Status.Family.CLIENT_ERROR) { + && responseStatus.getFamily() == Response.Status.Family.CLIENT_ERROR + && exception.getErrors().isEmpty()) { // there should be at least one error entry for 4xx errors except 409 according to the RFC 8040 // - creation of WARN log that something went wrong way on the server side LOG.warn("Input exception has a family of 4xx but doesn't contain any descriptive errors: {}", -- 2.36.6