@unito/integration-sdk 2.3.5 → 2.3.6

package/dist/src/index.cjs

@@ -1440,13 +1440,18 @@ class Provider {
      * @param options RequestOptions used to adjust the call made to the provider (use to override default headers).
      * @returns The {@link Response} extracted from the provider.
      */
-    async delete(endpoint, options) {
-        return this.fetchWrapper(endpoint, null, {
+    async delete(endpoint, options, body = null) {
+        const defaultHeaders = {
+            Accept: 'application/json',
+        };
+        // Only add Content-Type header when body is provided
+        if (body !== null) {
+            defaultHeaders['Content-Type'] = 'application/json';
+        }
+        return this.fetchWrapper(endpoint, body, {
             ...options,
             method: 'DELETE',
-            defaultHeaders: {
-                Accept: 'application/json',
-            },
+            defaultHeaders,
         });
     }
     generateAbsoluteUrl(providerUrl, endpoint, queryParams) {

package/dist/src/resources/provider.d.ts

@@ -207,7 +207,7 @@ export declare class Provider {
      * @param options RequestOptions used to adjust the call made to the provider (use to override default headers).
      * @returns The {@link Response} extracted from the provider.
      */
-    delete<T = undefined>(endpoint: string, options: RequestOptions): Promise<Response<T>>;
+    delete<T = undefined>(endpoint: string, options: RequestOptions, body?: RequestBody | null): Promise<Response<T>>;
     private generateAbsoluteUrl;
     private fetchWrapper;
     private handleError;

package/dist/src/resources/provider.js

@@ -241,13 +241,18 @@ export class Provider {
      * @param options RequestOptions used to adjust the call made to the provider (use to override default headers).
      * @returns The {@link Response} extracted from the provider.
      */
-    async delete(endpoint, options) {
-        return this.fetchWrapper(endpoint, null, {
+    async delete(endpoint, options, body = null) {
+        const defaultHeaders = {
+            Accept: 'application/json',
+        };
+        // Only add Content-Type header when body is provided
+        if (body !== null) {
+            defaultHeaders['Content-Type'] = 'application/json';
+        }
+        return this.fetchWrapper(endpoint, body, {
             ...options,
             method: 'DELETE',
-            defaultHeaders: {
-                Accept: 'application/json',
-            },
+            defaultHeaders,
         });
     }
     generateAbsoluteUrl(providerUrl, endpoint, queryParams) {

package/dist/test/resources/provider.test.js

@@ -404,6 +404,37 @@ describe('Provider', () => {
         ]);
         assert.deepEqual(actualResponse, { status: 204, headers: response.headers, body: undefined });
     });
+    it('deleteWithBody', async (context) => {
+        const response = new Response('{"success": true}', {
+            status: 200,
+            headers: { 'Content-Type': 'application/json' },
+        });
+        const fetchMock = context.mock.method(global, 'fetch', () => Promise.resolve(response));
+        const requestBody = { webhookIds: [1, 2, 3] };
+        const actualResponse = await provider.delete('/webhook', {
+            credentials: { apiKey: 'apikey#1111', unitoCredentialId: '123' },
+            logger: logger,
+            signal: new AbortController().signal,
+            additionnalheaders: { 'X-Additional-Header': 'value1' },
+        }, requestBody);
+        assert.equal(fetchMock.mock.calls.length, 1);
+        assert.deepEqual(fetchMock.mock.calls[0]?.arguments, [
+            'www.myApi.com/webhook',
+            {
+                method: 'DELETE',
+                body: JSON.stringify(requestBody),
+                signal: new AbortController().signal,
+                headers: {
+                    'Content-Type': 'application/json',
+                    Accept: 'application/json',
+                    'X-Custom-Provider-Header': 'value',
+                    'X-Provider-Credential-Header': 'apikey#1111',
+                    'X-Additional-Header': 'value1',
+                },
+            },
+        ]);
+        assert.deepEqual(actualResponse, { status: 200, headers: response.headers, body: { success: true } });
+    });
     it('uses rate limiter if provided', async (context) => {
         const mockRateLimiter = context.mock.fn((_context, request) => Promise.resolve(request()));
         const rateLimitedProvider = new Provider({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unito/integration-sdk",
-  "version": "2.3.5",
+  "version": "2.3.6",
   "description": "Integration SDK",
   "type": "module",
   "types": "dist/src/index.d.ts",

package/src/resources/provider.ts

@@ -336,13 +336,24 @@ export class Provider {
    * @param options RequestOptions used to adjust the call made to the provider (use to override default headers).
    * @returns The {@link Response} extracted from the provider.
    */
-  public async delete<T = undefined>(endpoint: string, options: RequestOptions): Promise<Response<T>> {
-    return this.fetchWrapper<T>(endpoint, null, {
+  public async delete<T = undefined>(
+    endpoint: string,
+    options: RequestOptions,
+    body: RequestBody | null = null,
+  ): Promise<Response<T>> {
+    const defaultHeaders: { Accept: string; 'Content-Type'?: string } = {
+      Accept: 'application/json',
+    };
+
+    // Only add Content-Type header when body is provided
+    if (body !== null) {
+      defaultHeaders['Content-Type'] = 'application/json';
+    }
+
+    return this.fetchWrapper<T>(endpoint, body, {
       ...options,
       method: 'DELETE',
-      defaultHeaders: {
-        Accept: 'application/json',
-      },
+      defaultHeaders,
     });
   }
 

package/test/resources/provider.test.ts

@@ -486,6 +486,45 @@ describe('Provider', () => {
     assert.deepEqual(actualResponse, { status: 204, headers: response.headers, body: undefined });
   });
 
+  it('deleteWithBody', async context => {
+    const response = new Response('{"success": true}', {
+      status: 200,
+      headers: { 'Content-Type': 'application/json' },
+    });
+
+    const fetchMock = context.mock.method(global, 'fetch', () => Promise.resolve(response));
+
+    const requestBody = { webhookIds: [1, 2, 3] };
+    const actualResponse = await provider.delete(
+      '/webhook',
+      {
+        credentials: { apiKey: 'apikey#1111', unitoCredentialId: '123' },
+        logger: logger,
+        signal: new AbortController().signal,
+        additionnalheaders: { 'X-Additional-Header': 'value1' },
+      },
+      requestBody,
+    );
+
+    assert.equal(fetchMock.mock.calls.length, 1);
+    assert.deepEqual(fetchMock.mock.calls[0]?.arguments, [
+      'www.myApi.com/webhook',
+      {
+        method: 'DELETE',
+        body: JSON.stringify(requestBody),
+        signal: new AbortController().signal,
+        headers: {
+          'Content-Type': 'application/json',
+          Accept: 'application/json',
+          'X-Custom-Provider-Header': 'value',
+          'X-Provider-Credential-Header': 'apikey#1111',
+          'X-Additional-Header': 'value1',
+        },
+      },
+    ]);
+    assert.deepEqual(actualResponse, { status: 200, headers: response.headers, body: { success: true } });
+  });
+
   it('uses rate limiter if provided', async context => {
     const mockRateLimiter = context.mock.fn((_context, request) => Promise.resolve(request()));