Map ParsingException to HTTP Bad Request

trinodb · Nov 28, 2024 · 7d1b217 · 7d1b217
1 parent 826ae9d
commit 7d1b217
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/core/trino-main/src/main/java/io/trino/server/ThrowableMapper.java b/core/trino-main/src/main/java/io/trino/server/ThrowableMapper.java
@@ -15,6 +15,7 @@
 
 import com.google.common.base.Throwables;
 import com.google.inject.Inject;
+import io.airlift.jaxrs.ParsingException;
 import jakarta.ws.rs.BadRequestException;
 import jakarta.ws.rs.ForbiddenException;
 import jakarta.ws.rs.InternalServerErrorException;
@@ -83,6 +84,9 @@ public Response toResponse(Throwable throwable)
             case TimeoutException timeoutException -> plainTextError(Response.Status.REQUEST_TIMEOUT)
                     .entity("Error 408 Timeout: " + timeoutException.getMessage())
                     .build();
+            case ParsingException parsingException -> Response.status(Response.Status.BAD_REQUEST)
+                    .entity(Throwables.getStackTraceAsString(parsingException))
+                    .build();
             case WebApplicationException webApplicationException -> webApplicationException.getResponse();
             default -> {
                 ResponseBuilder responseBuilder = plainTextError(Response.Status.INTERNAL_SERVER_ERROR);