Merge pull request #4009 from jamezp/RESTEASY-3443

[RESTEASY-3443] In the JsonProcessingExceptionMapper currently return…
resteasy · Feb 6, 2024 · 3f64acb · 3f64acb
2 parents 32d1a82 + 73446c4
commit 3f64acb
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/...main/java/org/jboss/resteasy/plugins/providers/jackson/JsonProcessingExceptionMapper.java b/...main/java/org/jboss/resteasy/plugins/providers/jackson/JsonProcessingExceptionMapper.java
@@ -35,6 +35,6 @@ public class JsonProcessingExceptionMapper implements ExceptionMapper<JsonProces
     @Override
     public Response toResponse(final JsonProcessingException exception) {
         JacksonLogger.LOGGER.logCannotDeserialize(exception);
-        return Response.status(Response.Status.INTERNAL_SERVER_ERROR).entity(JacksonLogger.LOGGER.cannotDeserialize()).build();
+        return Response.status(Response.Status.BAD_REQUEST).entity(JacksonLogger.LOGGER.cannotDeserialize()).build();
     }
 }
diff --git a/...est/providers/jackson2/whitelist/WhiteListPolymorphicTypeValidatorManualOverrideTest.java b/...est/providers/jackson2/whitelist/WhiteListPolymorphicTypeValidatorManualOverrideTest.java
@@ -91,7 +91,7 @@ public void testAircraftFailure() throws Exception {
         String response = sendPost(new TestPolymorphicType(new Aircraft()));
         logger.info("response: " + response);
         Assert.assertNotNull(response);
-        Assert.assertTrue(response.contains("Response code: " + HttpResponseCodes.SC_INTERNAL_SERVER_ERROR));
+        Assert.assertTrue(response.contains("Response code: " + HttpResponseCodes.SC_BAD_REQUEST));
         Assert.assertTrue("Expected response to contain \"Not able to deserialize data provided\" but was \"" + response + "\"",
                 response.contains("Not able to deserialize data provided"));
     }
@@ -101,7 +101,7 @@ public void testAutomobileFailure() throws Exception {
         String response = sendPost(new TestPolymorphicType(new Automobile()));
         logger.info("response: " + response);
         Assert.assertNotNull(response);
-        Assert.assertTrue(response.contains("Response code: " + HttpResponseCodes.SC_INTERNAL_SERVER_ERROR));
+        Assert.assertTrue(response.contains("Response code: " + HttpResponseCodes.SC_BAD_REQUEST));
         Assert.assertTrue("Expected response to contain \"Not able to deserialize data provided\" but was \"" + response + "\"",
                 response.contains("Not able to deserialize data provided"));
     }

diff --git a/...oss/resteasy/test/providers/jackson2/whitelist/WhiteListPolymorphicTypeValidatorTest.java b/...oss/resteasy/test/providers/jackson2/whitelist/WhiteListPolymorphicTypeValidatorTest.java
@@ -109,7 +109,7 @@ public void testBad() throws Exception {
         String response = sendPost(new TestPolymorphicType(new Aircraft()));
         logger.info("response: " + response);
         Assert.assertNotNull(response);
-        Assert.assertTrue(response.contains("Response code: " + HttpResponseCodes.SC_INTERNAL_SERVER_ERROR));
+        Assert.assertTrue(response.contains("Response code: " + HttpResponseCodes.SC_BAD_REQUEST));
         Assert.assertTrue("Expected response to contain \"Not able to deserialize data provided\" but was \"" + response + "\"",
                 response.contains("Not able to deserialize data provided"));
     }