@unito/integration-sdk 1.1.3 → 1.2.0

package/dist/src/index.cjs

@@ -1209,7 +1209,7 @@ class Provider {
         });
     }
     async postForm(endpoint, form, options) {
-        const { url: providerUrl, headers: providerHeaders } = this.prepareRequest(options);
+        const { url: providerUrl, headers: providerHeaders } = await this.prepareRequest(options);
         const absoluteUrl = this.generateAbsoluteUrl(providerUrl, endpoint, options.queryParams);
         const headers = { ...form.getHeaders(), ...providerHeaders, ...options.additionnalheaders };
         const reqOptions = {
@@ -1335,7 +1335,7 @@ class Provider {
         return absoluteUrl;
     }
     async fetchWrapper(endpoint, body, options) {
-        const { url: providerUrl, headers: providerHeaders } = this.prepareRequest(options);
+        const { url: providerUrl, headers: providerHeaders } = await this.prepareRequest(options);
         const absoluteUrl = this.generateAbsoluteUrl(providerUrl, endpoint, options.queryParams);
         const headers = { ...options.defaultHeaders, ...providerHeaders, ...options.additionnalheaders };
         let stringifiedBody = null;
@@ -1395,6 +1395,10 @@ class Provider {
                 // When we expect octet-stream, we accept any Content-Type the provider sends us, we just want to stream it.
                 body = response.body;
             }
+            else if (headers.Accept?.includes('text/html')) {
+                // Accept text based content types
+                body = (await response.text());
+            }
             else {
                 throw this.handleError(500, 'Unsupported Content-Type');
             }

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

@@ -54,6 +54,10 @@ export type Response<T> = {
     status: number;
     headers: Headers;
 };
+export type PreparedRequest = {
+    url: string;
+    headers: Record<string, string>;
+};
 /**
  * The Provider class is a wrapper around the fetch function to call a provider's HTTP API.
  *
@@ -82,10 +86,7 @@ export declare class Provider {
     protected prepareRequest: (options: {
         credentials: Credentials;
         logger: Logger;
-    }) => {
-        url: string;
-        headers: Record<string, string>;
-    };
+    }) => PreparedRequest | Promise<PreparedRequest>;
     /**
      * (Optional) Custom error handler to handle specific errors returned by the provider.
      *

package/dist/src/resources/provider.js

@@ -110,7 +110,7 @@ export class Provider {
         });
     }
     async postForm(endpoint, form, options) {
-        const { url: providerUrl, headers: providerHeaders } = this.prepareRequest(options);
+        const { url: providerUrl, headers: providerHeaders } = await this.prepareRequest(options);
         const absoluteUrl = this.generateAbsoluteUrl(providerUrl, endpoint, options.queryParams);
         const headers = { ...form.getHeaders(), ...providerHeaders, ...options.additionnalheaders };
         const reqOptions = {
@@ -236,7 +236,7 @@ export class Provider {
         return absoluteUrl;
     }
     async fetchWrapper(endpoint, body, options) {
-        const { url: providerUrl, headers: providerHeaders } = this.prepareRequest(options);
+        const { url: providerUrl, headers: providerHeaders } = await this.prepareRequest(options);
         const absoluteUrl = this.generateAbsoluteUrl(providerUrl, endpoint, options.queryParams);
         const headers = { ...options.defaultHeaders, ...providerHeaders, ...options.additionnalheaders };
         let stringifiedBody = null;
@@ -296,6 +296,10 @@ export class Provider {
                 // When we expect octet-stream, we accept any Content-Type the provider sends us, we just want to stream it.
                 body = response.body;
             }
+            else if (headers.Accept?.includes('text/html')) {
+                // Accept text based content types
+                body = (await response.text());
+            }
             else {
                 throw this.handleError(500, 'Unsupported Content-Type');
             }

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

@@ -50,6 +50,35 @@ describe('Provider', () => {
         ]);
         assert.deepEqual(actualResponse, { status: 200, headers: response.headers, body: { data: 'value' } });
     });
+    it('should accept text/html type response', async (context) => {
+        const response = new Response('', {
+            status: 200,
+            headers: { 'Content-Type': 'text/html; charset=UTF-8' },
+        });
+        const fetchMock = context.mock.method(global, 'fetch', () => Promise.resolve(response));
+        const actualResponse = await provider.get('/endpoint', {
+            credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
+            signal: new AbortController().signal,
+            additionnalheaders: { 'X-Additional-Header': 'value1', Accept: 'text/html; charset=UTF-8' },
+        });
+        assert.equal(fetchMock.mock.calls.length, 1);
+        assert.deepEqual(fetchMock.mock.calls[0]?.arguments, [
+            'www.myApi.com/endpoint',
+            {
+                method: 'GET',
+                body: null,
+                signal: new AbortController().signal,
+                headers: {
+                    Accept: 'text/html; charset=UTF-8',
+                    'X-Custom-Provider-Header': 'value',
+                    'X-Provider-Credential-Header': 'apikey#1111',
+                    'X-Additional-Header': 'value1',
+                },
+            },
+        ]);
+        assert.deepEqual(actualResponse, { status: 200, headers: response.headers, body: '' });
+    });
     it('gets an endpoint which is an absolute url', async (context) => {
         const response = new Response('{"data": "value"}', {
             status: 200,

package/package.json

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

package/src/resources/provider.ts

@@ -57,6 +57,11 @@ export type Response<T> = {
   headers: Headers;
 };
 
+export type PreparedRequest = {
+  url: string;
+  headers: Record<string, string>;
+};
+
 /**
  * The Provider class is a wrapper around the fetch function to call a provider's HTTP API.
  *
@@ -82,10 +87,10 @@ export class Provider {
    * This is applied at large to all requests made to the provider. If you need to add specific headers to a single request,
    * pass it through the RequestOptions object when calling the Provider's methods.
    */
-  protected prepareRequest: (options: { credentials: Credentials; logger: Logger }) => {
-    url: string;
-    headers: Record<string, string>;
-  };
+  protected prepareRequest: (options: {
+    credentials: Credentials;
+    logger: Logger;
+  }) => PreparedRequest | Promise<PreparedRequest>;
   /**
    * (Optional) Custom error handler to handle specific errors returned by the provider.
    *
@@ -181,7 +186,7 @@ export class Provider {
   }
 
   public async postForm<T>(endpoint: string, form: FormData, options: RequestOptions): Promise<Response<T>> {
-    const { url: providerUrl, headers: providerHeaders } = this.prepareRequest(options);
+    const { url: providerUrl, headers: providerHeaders } = await this.prepareRequest(options);
     const absoluteUrl = this.generateAbsoluteUrl(providerUrl, endpoint, options.queryParams);
     const headers = { ...form.getHeaders(), ...providerHeaders, ...options.additionnalheaders };
 
@@ -325,7 +330,7 @@ export class Provider {
     body: Record<string, unknown> | null,
     options: RequestOptions & { defaultHeaders: { 'Content-Type'?: string; Accept?: string }; method: string },
   ): Promise<Response<T>> {
-    const { url: providerUrl, headers: providerHeaders } = this.prepareRequest(options);
+    const { url: providerUrl, headers: providerHeaders } = await this.prepareRequest(options);
     const absoluteUrl = this.generateAbsoluteUrl(providerUrl, endpoint, options.queryParams);
     const headers = { ...options.defaultHeaders, ...providerHeaders, ...options.additionnalheaders };
 
@@ -395,6 +400,9 @@ export class Provider {
       } else if (headers.Accept == 'application/octet-stream') {
         // When we expect octet-stream, we accept any Content-Type the provider sends us, we just want to stream it.
         body = response.body as T;
+      } else if (headers.Accept?.includes('text/html')) {
+        // Accept text based content types
+        body = (await response.text()) as T;
       } else {
         throw this.handleError(500, 'Unsupported Content-Type');
       }

package/test/resources/provider.test.ts

@@ -59,6 +59,41 @@ describe('Provider', () => {
     assert.deepEqual(actualResponse, { status: 200, headers: response.headers, body: { data: 'value' } });
   });
 
+  it('should accept text/html type response', async context => {
+    const response = new Response('', {
+      status: 200,
+      headers: { 'Content-Type': 'text/html; charset=UTF-8' },
+    });
+
+    const fetchMock = context.mock.method(global, 'fetch', () => Promise.resolve(response));
+
+    const actualResponse = await provider.get('/endpoint', {
+      credentials: { apiKey: 'apikey#1111' },
+      logger: logger,
+      signal: new AbortController().signal,
+      additionnalheaders: { 'X-Additional-Header': 'value1', Accept: 'text/html; charset=UTF-8' },
+    });
+
+    assert.equal(fetchMock.mock.calls.length, 1);
+
+    assert.deepEqual(fetchMock.mock.calls[0]?.arguments, [
+      'www.myApi.com/endpoint',
+      {
+        method: 'GET',
+        body: null,
+        signal: new AbortController().signal,
+        headers: {
+          Accept: 'text/html; charset=UTF-8',
+          'X-Custom-Provider-Header': 'value',
+          'X-Provider-Credential-Header': 'apikey#1111',
+          'X-Additional-Header': 'value1',
+        },
+      },
+    ]);
+
+    assert.deepEqual(actualResponse, { status: 200, headers: response.headers, body: '' });
+  });
+
   it('gets an endpoint which is an absolute url', async context => {
     const response = new Response('{"data": "value"}', {
       status: 200,