fix(notification): class cast exception would be thrown when sending …

…to feishu or wecom (#3118)
oceanbase · Aug 12, 2024 · f9b9608 · f9b9608
1 parent 898dca0
commit f9b9608
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/server/odc-service/src/main/java/com/oceanbase/odc/service/notification/FeishuSender.java b/server/odc-service/src/main/java/com/oceanbase/odc/service/notification/FeishuSender.java
@@ -78,8 +78,8 @@ protected String getBody(Message message) {
     }
 
     @Override
-    protected MessageSendResult checkResponse(Message message, ResponseEntity response) {
-        Map<String, Object> body = (Map) response.getBody();
+    protected MessageSendResult checkResponse(Message message, ResponseEntity<String> response) {
+        Map<String, Object> body = JsonUtils.fromJsonMap(response.getBody(), String.class, Object.class);
         if (Objects.isNull(body)) {
             return MessageSendResult.ofFail("empty response from Feishu");
         }

diff --git a/server/odc-service/src/main/java/com/oceanbase/odc/service/notification/WeComSender.java b/server/odc-service/src/main/java/com/oceanbase/odc/service/notification/WeComSender.java
@@ -63,8 +63,8 @@ protected String getBody(Message message) {
     }
 
     @Override
-    protected MessageSendResult checkResponse(Message message, ResponseEntity response) {
-        Map<String, Object> body = (Map) response.getBody();
+    protected MessageSendResult checkResponse(Message message, ResponseEntity<String> response) {
+        Map<String, Object> body = JsonUtils.fromJsonMap(response.getBody(), String.class, Object.class);
         if (Objects.isNull(body)) {
             return MessageSendResult.ofFail("empty response from WeCom");
         }