[ZEPPELIN-5436] NPE in ZeppelinClient

zjffdu · zjffdu · commit 100e66ae9f0b · 2021-07-07T16:06:45.000+08:00
diff --git a/zeppelin-client/src/main/java/org/apache/zeppelin/client/ZeppelinClient.java b/zeppelin-client/src/main/java/org/apache/zeppelin/client/ZeppelinClient.java
@@ -88,7 +88,9 @@ private void checkResponse(HttpResponse<JsonNode> response) throws Exception {
     }
     if (response.getStatus() != 200) {
       String message = response.getStatusText();
-      if (response.getBody().getObject().has("message")) {
+      if (response.getBody() != null &&
+              response.getBody().getObject() != null &&
+              response.getBody().getObject().has("message")) {
         message = response.getBody().getObject().getString("message");
       }
       throw new Exception(String.format("Unable to call rest api, status: %s, statusText: %s, message: %s",

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,9 @@ private void checkResponse(HttpResponse<JsonNode> response) throws Exception {`
`88`	`88`	`}`
`89`	`89`	`if (response.getStatus() != 200) {`
`90`	`90`	`String message = response.getStatusText();`
`91`		`- if (response.getBody().getObject().has("message")) {`
	`91`	`+ if (response.getBody() != null &&`
	`92`	`+ response.getBody().getObject() != null &&`
	`93`	`+ response.getBody().getObject().has("message")) {`
`92`	`94`	`message = response.getBody().getObject().getString("message");`
`93`	`95`	`}`
`94`	`96`	`throw new Exception(String.format("Unable to call rest api, status: %s, statusText: %s, message: %s",`