Merge pull request #8 from telicent-oss/unhandled-stack-traces

FallbackExceptionMapper logs stack traces (CORE-288)

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Change Log
 
+# 0.20.1
+
+- JAX-RS Base Server Improvements:
+    - `FallbackExceptionMapper` explicitly logs the stack trace for otherwise unhandled exceptions to aid in diagnosis 
+      of the underlying issue
+
 # 0.20.0
 
 - Build improvements:

jaxrs-base-server/src/main/java/io/telicent/smart/cache/server/jaxrs/errors/FallbackExceptionMapper.java

@@ -20,13 +20,17 @@
 import jakarta.ws.rs.core.Response;
 import jakarta.ws.rs.ext.ExceptionMapper;
 import jakarta.ws.rs.ext.Provider;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * Maps otherwise unhandled errors into RFC 7807 Problem responses
  */
 @Provider
 public class FallbackExceptionMapper implements ExceptionMapper<Exception> {
 
+    private static final Logger LOGGER = LoggerFactory.getLogger(FallbackExceptionMapper.class);
+
     private String buildDetail(Throwable e) {
         StringBuilder builder = new StringBuilder();
         while (e != null) {
@@ -55,6 +59,9 @@ public Response toResponse(Exception exception) {
             //@formatter:on
         }
 
+        // Explicitly log the error with its stack trace for diagnostic purposes
+        LOGGER.error("Unhandled exception, see stack trace for more detail:", exception);
+
         // For any other error just translate into a 500 Internal Server Error
         //@formatter:off
         return new Problem("InternalServerError",