From eeffd2a3c065c4fdc15f0e28f7a072a2c8443a13 Mon Sep 17 00:00:00 2001 From: guangning Date: Tue, 6 Dec 2022 11:34:56 +0800 Subject: [PATCH 1/2] Update resposne body when request failed --- .../pulsar/client/admin/internal/http/AsyncHttpConnector.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/internal/http/AsyncHttpConnector.java b/pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/internal/http/AsyncHttpConnector.java index 318c404282a94..6ae18724166f6 100644 --- a/pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/internal/http/AsyncHttpConnector.java +++ b/pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/internal/http/AsyncHttpConnector.java @@ -241,6 +241,9 @@ public Status.Family getFamily() { @Override public String getReasonPhrase() { + if (response.hasResponseBody()) { + return response.getResponseBody(); + } return response.getStatusText(); } }); From 63e4b4860f30666777ac0bf8f6b6d867b6cb4c72 Mon Sep 17 00:00:00 2001 From: guangning Date: Wed, 7 Dec 2022 14:00:32 +0800 Subject: [PATCH 2/2] Fixed test --- .../java/org/apache/pulsar/broker/admin/AdminApiSchemaTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiSchemaTest.java b/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiSchemaTest.java index 37795438dbaf3..0795a9a6cc053 100644 --- a/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiSchemaTest.java +++ b/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiSchemaTest.java @@ -278,7 +278,7 @@ public void testInvalidSchemaDataException(ApiVersion version) { admin.schemas().createSchema(topicName, schemaInfo); } catch (PulsarAdminException e) { Assert.assertEquals(e.getStatusCode(), 422); - Assert.assertEquals(e.getMessage(), "HTTP 422 Invalid schema definition data for AVRO schema"); + Assert.assertTrue(e.getMessage().contains("Invalid schema definition data for AVRO schema")); } }