Fix ack alert API

opendistro-for-elasticsearch · Nov 16, 2020 · 32e202f · 32e202f
1 parent 71db812
commit 32e202f
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/public/pages/Dashboard/containers/Dashboard.js b/public/pages/Dashboard/containers/Dashboard.js
@@ -216,7 +216,9 @@ export default class Dashboard extends Component {
 
     const promises = Object.entries(monitorAlerts).map(([monitorId, alerts]) =>
       httpClient
-        .post(`../api/alerting/monitors/${monitorId}/_acknowledge/alerts`, { alerts })
+        .post(`../api/alerting/monitors/${monitorId}/_acknowledge/alerts`, {
+          body: JSON.stringify({ alerts }),
+        })
         .catch((error) => error)
     );
 

diff --git a/public/pages/Monitors/containers/Monitors/Monitors.js b/public/pages/Monitors/containers/Monitors/Monitors.js
@@ -236,7 +236,7 @@ export default class Monitors extends Component {
     const promises = Object.entries(monitorAlerts).map(([monitorId, alerts]) =>
       httpClient
         .post(`../api/alerting/monitors/${monitorId}/_acknowledge/alerts`, {
-          body: JSON.stringify(alerts),
+          body: JSON.stringify({ alerts }),
         })
         .catch((error) => error)
     );
@@ -305,9 +305,7 @@ export default class Monitors extends Component {
     const response = await httpClient.get('../api/alerting/alerts', { query: params });
 
     if (response.ok) {
-      const {
-        data: { alerts, totalAlerts },
-      } = response;
+      const { alerts, totalAlerts } = response;
       this.setState({
         alerts,
         totalAlerts,

diff --git a/server/routes/monitors.js b/server/routes/monitors.js
@@ -148,6 +148,7 @@ export default function (services, router) {
         params: schema.object({
           id: schema.string(),
         }),
+        body: schema.any(),
       },
     },
     monitorService.acknowledgeAlerts