[Actions] Adjust axios payload for APIs that do not want data sent (e…

…lastic#165756)
kibanamachine · Sep 6, 2023 · d6a2bd7 · d6a2bd7
1 parent b48dc84
commit d6a2bd7
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/x-pack/plugins/actions/server/lib/axios_utils.test.ts b/x-pack/plugins/actions/server/lib/axios_utils.test.ts
@@ -66,7 +66,6 @@ describe('request', () => {
 
     expect(axiosMock).toHaveBeenCalledWith('/test', {
       method: 'get',
-      data: {},
       httpAgent: undefined,
       httpsAgent: expect.any(HttpsAgent),
       proxy: false,
@@ -100,7 +99,6 @@ describe('request', () => {
 
     expect(axiosMock).toHaveBeenCalledWith(TestUrl, {
       method: 'get',
-      data: {},
       httpAgent,
       httpsAgent,
       proxy: false,
@@ -132,7 +130,6 @@ describe('request', () => {
 
     expect(axiosMock).toHaveBeenCalledWith('https://testProxy', {
       method: 'get',
-      data: {},
       httpAgent: undefined,
       httpsAgent: expect.any(HttpsAgent),
       proxy: false,

diff --git a/x-pack/plugins/actions/server/lib/axios_utils.ts b/x-pack/plugins/actions/server/lib/axios_utils.ts
@@ -51,7 +51,7 @@ export const request = async <T = unknown>({
     ...config,
     method,
     headers,
-    data: data ?? {},
+    ...(data ? { data } : {}),
     // use httpAgent and httpsAgent and set axios proxy: false, to be able to handle fail on invalid certs
     httpAgent,
     httpsAgent,