This is an automated email from the ASF dual-hosted git repository.

kirs pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler.git


The following commit(s) were added to refs/heads/dev by this push:
     new 2d71930  the update function should use post instead of get (#5703)
2d71930 is described below

commit 2d71930837092c52dc63a76941b19051ae38cc2e
Author: kyoty <[email protected]>
AuthorDate: Sun Jun 27 18:58:22 2021 +0800

    the update function should use post instead of get (#5703)
---
 .../dolphinscheduler/api/controller/AlertPluginInstanceController.java  | 2 +-
 dolphinscheduler-ui/src/js/conf/home/store/security/actions.js          | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/controller/AlertPluginInstanceController.java
 
b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/controller/AlertPluginInstanceController.java
index 42c5536..346e041 100644
--- 
a/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/controller/AlertPluginInstanceController.java
+++ 
b/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/controller/AlertPluginInstanceController.java
@@ -108,7 +108,7 @@ public class AlertPluginInstanceController extends 
BaseController {
             @ApiImplicitParam(name = "instanceName", value = 
"ALERT_PLUGIN_INSTANCE_NAME", required = true, dataType = "String", example = 
"DING TALK"),
             @ApiImplicitParam(name = "pluginInstanceParams", value = 
"ALERT_PLUGIN_INSTANCE_PARAMS", required = true, dataType = "String", example = 
"ALERT_PLUGIN_INSTANCE_PARAMS")
     })
-    @GetMapping(value = "/update")
+    @PostMapping(value = "/update")
     @ResponseStatus(HttpStatus.OK)
     @ApiException(UPDATE_ALERT_PLUGIN_INSTANCE_ERROR)
     @AccessLogAnnotation(ignoreRequestArgs = "loginUser")
diff --git a/dolphinscheduler-ui/src/js/conf/home/store/security/actions.js 
b/dolphinscheduler-ui/src/js/conf/home/store/security/actions.js
index 87a96dd..eda5d02 100644
--- a/dolphinscheduler-ui/src/js/conf/home/store/security/actions.js
+++ b/dolphinscheduler-ui/src/js/conf/home/store/security/actions.js
@@ -446,7 +446,7 @@ export default {
    */
   updateAlertPluginInstance ({ state }, payload) {
     return new Promise((resolve, reject) => {
-      io.get('alert-plugin-instance/update', payload, res => {
+      io.post('alert-plugin-instance/update', payload, res => {
         resolve(res)
       }).catch(e => {
         reject(e)

Reply via email to