This is an automated email from the ASF dual-hosted git repository.
mikexue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/eventmesh.git
The following commit(s) were added to refs/heads/master by this push:
new c5f3aba89 [ISSUE #3963] Method invocation 'getBytes' may produce
'NullPointerException'. (#3971)
c5f3aba89 is described below
commit c5f3aba895b3a877c30e3edc3ac5818e4c91fe18
Author: ZahaanMahajan <[email protected]>
AuthorDate: Wed May 31 07:43:27 2023 +0530
[ISSUE #3963] Method invocation 'getBytes' may produce
'NullPointerException'. (#3971)
* fixes [Enhancement] #3963
* Fix Updated
* Update QueryWebHookConfigByIdHandler.java
---------
Co-authored-by: mike_xwm <[email protected]>
---
.../runtime/admin/handler/QueryWebHookConfigByIdHandler.java | 4 ++--
.../admin/handler/QueryWebHookConfigByManufacturerHandler.java | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git
a/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/QueryWebHookConfigByIdHandler.java
b/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/QueryWebHookConfigByIdHandler.java
index 55e726e45..35ea9a671 100644
---
a/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/QueryWebHookConfigByIdHandler.java
+++
b/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/QueryWebHookConfigByIdHandler.java
@@ -30,7 +30,7 @@ import
org.apache.eventmesh.webhook.api.WebHookConfigOperation;
import java.io.IOException;
import java.io.OutputStream;
-
+import java.util.Objects;
import com.sun.net.httpserver.HttpExchange;
@@ -60,7 +60,7 @@ public class QueryWebHookConfigByIdHandler extends
AbstractHttpHandler {
try (OutputStream out = httpExchange.getResponseBody()) {
WebHookConfig result =
operation.queryWebHookConfigById(webHookConfig); // operating result
-
out.write(JsonUtils.toJSONString(result).getBytes(Constants.DEFAULT_CHARSET));
+
out.write(Objects.requireNonNull(JsonUtils.toJSONString(result)).getBytes(Constants.DEFAULT_CHARSET));
} catch (Exception e) {
log.error("get WebHookConfigOperation implementation Failed.", e);
}
diff --git
a/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/QueryWebHookConfigByManufacturerHandler.java
b/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/QueryWebHookConfigByManufacturerHandler.java
index f49e7942d..3a16cce36 100644
---
a/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/QueryWebHookConfigByManufacturerHandler.java
+++
b/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/QueryWebHookConfigByManufacturerHandler.java
@@ -73,7 +73,7 @@ public class QueryWebHookConfigByManufacturerHandler extends
AbstractHttpHandler
try (OutputStream out = httpExchange.getResponseBody()) {
List<WebHookConfig> result =
operation.queryWebHookConfigByManufacturer(webHookConfig, pageNum, pageSize);
// operating result
-
out.write(JsonUtils.toJSONString(result).getBytes(Constants.DEFAULT_CHARSET));
+
out.write(Objects.requireNonNull(JsonUtils.toJSONString(result)).getBytes(Constants.DEFAULT_CHARSET));
} catch (Exception e) {
log.error("get WebHookConfigOperation implementation Failed.", e);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
