Change the SendTestMessage API to be a POST call (#506)

* Change the SendTestMessage API to be a POST call Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Include GET send test message path as a depricated option until next major version Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> * Update documentation comment for SendTestMessage replaced route Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com> Signed-off-by: Mohammad Qureshi <47198598+qreshi@users.noreply.github.com>
xluo-aws · Jan 9, 2023 · 2fa9f01 · 2fa9f01
1 parent 53ba269
commit 2fa9f01
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/public/services/NotificationService.ts b/public/services/NotificationService.ts
@@ -246,7 +246,7 @@ export default class NotificationService {
   sendTestMessage = async (
       configId: string
   ) => {
-    const response = await this.httpClient.get(
+    const response = await this.httpClient.post(
         `${NODE_API.SEND_TEST_MESSAGE}/${configId}`
     );
     if (response.status_list[0].delivery_status.status_code != 200) {

diff --git a/server/clusters/notificationsPlugin.ts b/server/clusters/notificationsPlugin.ts
@@ -89,7 +89,7 @@ export function NotificationsPlugin(Client: any, config: any, components: any) {
         },
       },
     },
-    method: 'GET',
+    method: 'POST',
   });
 
   notifications.getServerFeatures = clientAction({

diff --git a/server/routes/eventRoutes.ts b/server/routes/eventRoutes.ts
@@ -40,7 +40,7 @@ export function eventRoutes(router: IRouter) {
     }
   );
 
-  router.get(
+  router.post(
     {
       path: `${NODE_API.SEND_TEST_MESSAGE}/{configId}`,
       validate: {