From 8dc035c90ba706efceb66ced5fa51408b4c494f9 Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Tue, 20 Jul 2021 13:09:07 +0200 Subject: [PATCH] Log full exception when we are missing errors Our current logging does not identify the source of the offending exception. Make sure we log the entire input exception, so that offenders may be fixed up. JIRA: NETCONF-789 Change-Id: I30a24a615ce221a838e1ad95f6b11263e2d6f8f5 Signed-off-by: Robert Varga --- .../errors/RestconfDocumentedExceptionMapper.java | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) 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 a4e9de5209..60aeaba843 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 @@ -92,10 +92,9 @@ public final class RestconfDocumentedExceptionMapper implements ExceptionMapper< if (responseStatus != Response.Status.FORBIDDEN && 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: {}", - exception.getMessage()); + // There should be at least one error entry for 4xx errors except 409 according to RFC8040, but we do not + // have it. Issue a warning with the call trace so we can fix whoever was the originator. + LOG.warn("Input exception has a family of 4xx but does not contain any descriptive errors", exception); } final ContainerNode errorsContainer = buildErrorsContainer(exception); @@ -368,4 +367,4 @@ public final class RestconfDocumentedExceptionMapper implements ExceptionMapper< void setHttpHeaders(final HttpHeaders httpHeaders) { this.headers = httpHeaders; } -} \ No newline at end of file +} -- 2.36.6