From c912e533f749061e57c9281fa8ee2f38b44e7c45 Mon Sep 17 00:00:00 2001 From: Tyler Ouyang Date: Thu, 1 Feb 2024 15:33:02 -0800 Subject: [PATCH 1/3] Add JsonProcessingExceptionMapper --- .../main/java/com/pinterest/teletraan/TeletraanService.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java b/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java index 0fda13f885..810d67a945 100644 --- a/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java +++ b/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java @@ -24,6 +24,7 @@ import io.dropwizard.configuration.SubstitutingSourceProvider; import io.dropwizard.health.conf.HealthConfiguration; import io.dropwizard.health.core.HealthCheckBundle; +import io.dropwizard.jersey.jackson.JsonProcessingExceptionMapper; import io.dropwizard.setup.Bootstrap; import io.dropwizard.setup.Environment; import io.swagger.jaxrs.config.BeanConfig; @@ -172,6 +173,8 @@ public void run(TeletraanServiceConfiguration configuration, Environment environ environment.healthChecks().register("generic", new GenericHealthCheck(context)); // Exception handler + // Jackson Json parsing exceptions, returns 4xx + environment.jersey().register(new JsonProcessingExceptionMapper()); environment.jersey().register(new GenericExceptionMapper(configuration.getSystemFactory().getClientError())); // Swagger API docs generation related From 5dc0b71e869a49f9b8f457c67b9b0366d31fea3f Mon Sep 17 00:00:00 2001 From: Tyler Ouyang Date: Thu, 1 Feb 2024 15:41:15 -0800 Subject: [PATCH 2/3] updated comment --- .../src/main/java/com/pinterest/teletraan/TeletraanService.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java b/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java index 810d67a945..d6d2d7090f 100644 --- a/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java +++ b/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java @@ -173,7 +173,7 @@ public void run(TeletraanServiceConfiguration configuration, Environment environ environment.healthChecks().register("generic", new GenericHealthCheck(context)); // Exception handler - // Jackson Json parsing exceptions, returns 4xx + // Jackson Json parsing exceptions environment.jersey().register(new JsonProcessingExceptionMapper()); environment.jersey().register(new GenericExceptionMapper(configuration.getSystemFactory().getClientError())); From 2e37465721d8403a2533cfe95c96b51971a18c9b Mon Sep 17 00:00:00 2001 From: Tyler Ouyang Date: Thu, 1 Feb 2024 15:43:56 -0800 Subject: [PATCH 3/3] show details --- .../src/main/java/com/pinterest/teletraan/TeletraanService.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java b/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java index d6d2d7090f..532780ed58 100644 --- a/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java +++ b/deploy-service/teletraanservice/src/main/java/com/pinterest/teletraan/TeletraanService.java @@ -174,7 +174,7 @@ public void run(TeletraanServiceConfiguration configuration, Environment environ // Exception handler // Jackson Json parsing exceptions - environment.jersey().register(new JsonProcessingExceptionMapper()); + environment.jersey().register(new JsonProcessingExceptionMapper(true)); environment.jersey().register(new GenericExceptionMapper(configuration.getSystemFactory().getClientError())); // Swagger API docs generation related