DefaultExceptionMapper

Signed-off-by: Maxim Nesen <maxim.nesen@oracle.com>

Author: senivam

core-common/src/main/java/org/glassfish/jersey/internal/DefaultExceptionMapper.java

@@ -0,0 +1,40 @@
+/*
+ * Copyright (c) 2021 Oracle and/or its affiliates. All rights reserved.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License v. 2.0, which is available at
+ * http://www.eclipse.org/legal/epl-2.0.
+ *
+ * This Source Code may also be made available under the following Secondary
+ * Licenses when the conditions for such availability set forth in the
+ * Eclipse Public License v. 2.0 are satisfied: GNU General Public License,
+ * version 2 with the GNU Classpath Exception, which is available at
+ * https://www.gnu.org/software/classpath/license.html.
+ *
+ * SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
+ */
+
+package org.glassfish.jersey.internal;
+
+import jakarta.ws.rs.WebApplicationException;
+import jakarta.ws.rs.core.Response;
+import jakarta.ws.rs.ext.ExceptionMapper;
+
+public class DefaultExceptionMapper implements ExceptionMapper<Throwable> {
+    @Override
+    public Response toResponse(Throwable exception) {
+        return (exception instanceof WebApplicationException)
+                ? processWebApplicationException((WebApplicationException) exception)
+                : processDefaultException(exception);
+    }
+
+    private static Response processWebApplicationException(WebApplicationException exception) {
+        return (exception.getResponse() == null)
+                ? processDefaultException(exception)
+                : exception.getResponse();
+    }
+
+    private static Response processDefaultException(Throwable exception) {
+        return Response.status(Response.Status.INTERNAL_SERVER_ERROR).entity(exception.getMessage()).build();
+    }
+}

core-common/src/main/java/org/glassfish/jersey/internal/ExceptionMapperFactory.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2010, 2020 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2010, 2021 Oracle and/or its affiliates. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0, which is available at
@@ -58,6 +58,8 @@ public class ExceptionMapperFactory implements ExceptionMappers {
 
     private static final Logger LOGGER = Logger.getLogger(ExceptionMapperFactory.class.getName());
 
+    private static final ExceptionMapper<Throwable> DEFAULT_EXCEPTION_MAPPER = new DefaultExceptionMapper();
+
     /**
      * Configurator which initializes and register {@link ExceptionMappers} instance into {@link InjectionManager} and
      * {@link BootstrapBag}.
@@ -127,7 +129,7 @@ private <T extends Throwable> ExceptionMapper<T> find(final Class<T> type, final
                 }
             }
         }
-        return mapper;
+        return mapper == null ? (ExceptionMapper<T>) DEFAULT_EXCEPTION_MAPPER : mapper;
     }
 
     /**

core-server/pom.xml

@@ -196,11 +196,6 @@
             <artifactId>jakarta.validation-api</artifactId>
         </dependency>
 
-        <dependency>
-            <groupId>jakarta.xml.bind</groupId>
-            <artifactId>jakarta.xml.bind-api</artifactId>
-        </dependency>
-
         <dependency>
             <groupId>com.sun.xml.bind</groupId>
             <artifactId>jaxb-osgi</artifactId>

core-server/src/main/java/org/glassfish/jersey/server/ServerRuntime.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2012, 2020 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2012, 2021 Oracle and/or its affiliates. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0, which is available at
@@ -54,6 +54,7 @@
 
 import jakarta.inject.Provider;
 
+import org.glassfish.jersey.internal.DefaultExceptionMapper;
 import org.glassfish.jersey.internal.guava.Preconditions;
 import org.glassfish.jersey.internal.inject.InjectionManager;
 import org.glassfish.jersey.internal.inject.Injections;
@@ -503,7 +504,7 @@ private Response mapException(final Throwable originalThrowable) throws Throwabl
                             && throwable.getCause() instanceof MessageBodyProviderNotFoundException) {
                         throw throwable;
                     }
-                    Response waeResponse = null;
+                    Response waeResponse;
 
                     if (throwable instanceof WebApplicationException) {
                         final WebApplicationException webApplicationException = (WebApplicationException) throwable;
@@ -512,10 +513,11 @@ private Response mapException(final Throwable originalThrowable) throws Throwabl
                         processingContext.routingContext().setMappedThrowable(throwable);
 
                         waeResponse = webApplicationException.getResponse();
-                        if (waeResponse.hasEntity()) {
-                            LOGGER.log(Level.FINE, LocalizationMessages
-                                    .EXCEPTION_MAPPING_WAE_ENTITY(waeResponse.getStatus()), throwable);
-                            return waeResponse;
+                        if (waeResponse != null) {
+                            LOGGER.log(Level.FINE, waeResponse.hasEntity()
+                                    ? LocalizationMessages.EXCEPTION_MAPPING_WAE_ENTITY(waeResponse.getStatus())
+                                    : LocalizationMessages.EXCEPTION_MAPPING_WAE_NO_ENTITY(waeResponse.getStatus()),
+                                    throwable);
                         }
                     }
 
@@ -527,7 +529,7 @@ private Response mapException(final Throwable originalThrowable) throws Throwabl
                         try {
                             final Response mappedResponse = mapper.toResponse(throwable);
 
-                            if (tracingLogger.isLogEnabled(ServerTraceEvent.EXCEPTION_MAPPING)) {
+                            if (isTracingLoggingEnabled(mapper, throwable, tracingLogger)) {
                                 tracingLogger.logDuration(ServerTraceEvent.EXCEPTION_MAPPING,
                                         timestamp, mapper, throwable, throwable.getLocalizedMessage(),
                                         mappedResponse != null ? mappedResponse.getStatusInfo() : "-no-response-");
@@ -561,13 +563,6 @@ private Response mapException(final Throwable originalThrowable) throws Throwabl
                             return Response.serverError().build();
                         }
                     }
-
-                    if (waeResponse != null) {
-                        LOGGER.log(Level.FINE, LocalizationMessages
-                                .EXCEPTION_MAPPING_WAE_NO_ENTITY(waeResponse.getStatus()), throwable);
-
-                        return waeResponse;
-                    }
                 }
                 // internal mapping
                 if (throwable instanceof HeaderValueException) {
@@ -740,6 +735,14 @@ private void release(final ContainerResponse responseContext) {
                 processingContext.triggerEvent(RequestEvent.Type.FINISHED);
             }
         }
+
+        private static boolean isTracingLoggingEnabled(ExceptionMapper mapper, Throwable throwable, TracingLogger tracingLogger) {
+            boolean defaultLoggingState = mapper instanceof DefaultExceptionMapper
+                    && throwable instanceof WebApplicationException;
+            return !defaultLoggingState
+                    && tracingLogger.isLogEnabled(ServerTraceEvent.EXCEPTION_MAPPING);
+
+        }
     }
 
     private static class AsyncResponder implements AsyncContext, ContainerResponseWriter.TimeoutHandler, CompletionCallback {

core-server/src/test/java/org/glassfish/jersey/server/ApplicationHandlerTest.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2010, 2020 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2010, 2021 Oracle and/or its affiliates. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0, which is available at
@@ -130,9 +130,13 @@ public String doPost() {
     public void testReturnBadRequestOnIllHeaderValue() throws Exception {
         ApplicationHandler app = createApplication(Resource.class);
 
-        assertEquals(400,
-                app.apply(RequestContextBuilder.from("/", "GET").header(HttpHeaders.CONTENT_LENGTH, "text").build())
-                        .get().getStatus());
+        final ContainerResponse response =
+                app.apply(RequestContextBuilder.from("/", "GET")
+                                .header(HttpHeaders.CONTENT_LENGTH, "text").build())
+                                .get();
+       assertEquals("Unable to parse \"Content-Length\" header value: \"text\"", response.getEntity());
+        assertEquals(500,
+                response.getStatus());
     }
 
     @Test
@@ -558,12 +562,14 @@ public void testMapResponseErrorForMbw() throws Exception {
     /**
      * Test that un-mapped response errors are tried to be processed only once (MBW).
      */
-    @Test(expected = ExecutionException.class)
+    @Test
     public void testMapCyclicResponseErrorForMbw() throws Exception {
         final ApplicationHandler handler = new ApplicationHandler(MapResponseErrorApplication.class);
 
         final ContainerRequest context = RequestContextBuilder.from("/foobar", "GET").build();
 
-        handler.apply(context).get();
+        final ContainerResponse containerResponse = handler.apply(context).get();
+        assertEquals(500, containerResponse.getStatus());
+        assertEquals("Cannot do that!", containerResponse.getEntity());
     }
 }

core-server/src/test/java/org/glassfish/jersey/server/AsyncCallbackServerTest.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2012, 2020 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2012, 2021 Oracle and/or its affiliates. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0, which is available at
@@ -35,6 +35,7 @@
 
 import jakarta.inject.Singleton;
 
+import org.junit.Ignore;
 import org.junit.Test;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
@@ -71,6 +72,7 @@ public void testCompletionCallback() throws ExecutionException, InterruptedExcep
     }
 
     @Test
+    @Ignore  //since JAXRS 3.1 failure is not processed by callbacks
     public void testCompletionFail() throws ExecutionException, InterruptedException {
         final Flags flags = new Flags();
 

core-server/src/test/java/org/glassfish/jersey/server/filter/ApplicationFilterTest.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011, 2020 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2011, 2021 Oracle and/or its affiliates. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0, which is available at
@@ -318,11 +318,7 @@ public Response apply(final ContainerRequestContext request) {
         });
         resourceConfig.registerResources(rb.build());
         final ApplicationHandler application = new ApplicationHandler(resourceConfig);
-        try {
-            application.apply(RequestContextBuilder.from("/test", "GET").build()).get().getStatus();
-            Assert.fail("should throw an exception");
-        } catch (final Exception e) {
-            // ok
-        }
+        int status = application.apply(RequestContextBuilder.from("/test", "GET").build()).get().getStatus();
+        Assert.assertEquals(500, status);
     }
 }

ext/entity-filtering/pom.xml

@@ -52,12 +52,6 @@
             <scope>provided</scope>
         </dependency>
 
-        <dependency>
-            <groupId>jakarta.xml.bind</groupId>
-            <artifactId>jakarta.xml.bind-api</artifactId>
-            <scope>provided</scope>
-        </dependency>
-
         <dependency>
             <groupId>org.glassfish.jersey.test-framework</groupId>
             <artifactId>jersey-test-framework-core</artifactId>

ext/wadl-doclet/pom.xml

@@ -207,10 +207,6 @@
             <artifactId>jersey-server</artifactId>
             <version>${project.version}</version>
         </dependency>
-        <dependency>
-            <groupId>jakarta.xml.bind</groupId>
-            <artifactId>jakarta.xml.bind-api</artifactId>
-        </dependency>
     </dependencies>
 
     <build>

incubator/declarative-linking/pom.xml

@@ -52,11 +52,6 @@
             <artifactId>jakarta.xml.bind-api</artifactId>
             <scope>provided</scope>
         </dependency>
-        <dependency>
-            <groupId>jakarta.xml.bind</groupId>
-            <artifactId>jakarta.xml.bind-api</artifactId>
-            <scope>provided</scope>
-        </dependency>
         <dependency>
             <groupId>jakarta.el</groupId>
             <artifactId>jakarta.el-api</artifactId>