@unito/integration-sdk 1.4.3 → 1.4.4

package/dist/src/index.cjs

@@ -1374,6 +1374,10 @@ class Provider {
                 const textResult = await response.text();
                 throw this.handleError(response.status, textResult, options);
             }
+            else if (response.status === 204 || response.body === null) {
+                // No content: return without inspecting the body
+                return { status: response.status, headers: response.headers, body: undefined };
+            }
             const responseContentType = response.headers.get('content-type');
             let body;
             if (options.rawBody || headers.Accept === 'application/octet-stream') {
@@ -1387,7 +1391,7 @@ class Provider {
                 if (responseContentType && !responseContentType.includes('application/json')) {
                     const textResult = await response.text();
                     throw this.handleError(500, `Unsupported content-type, expected 'application/json' but got '${responseContentType}'.
-             Original response (${response.status}): ${textResult}`, options);
+             Original response (${response.status}): "${textResult}"`, options);
                 }
                 try {
                     body = response.body ? await response.json() : undefined;

package/dist/src/resources/provider.js

@@ -275,6 +275,10 @@ export class Provider {
                 const textResult = await response.text();
                 throw this.handleError(response.status, textResult, options);
             }
+            else if (response.status === 204 || response.body === null) {
+                // No content: return without inspecting the body
+                return { status: response.status, headers: response.headers, body: undefined };
+            }
             const responseContentType = response.headers.get('content-type');
             let body;
             if (options.rawBody || headers.Accept === 'application/octet-stream') {
@@ -288,7 +292,7 @@ export class Provider {
                 if (responseContentType && !responseContentType.includes('application/json')) {
                     const textResult = await response.text();
                     throw this.handleError(500, `Unsupported content-type, expected 'application/json' but got '${responseContentType}'.
-             Original response (${response.status}): ${textResult}`, options);
+             Original response (${response.status}): "${textResult}"`, options);
                 }
                 try {
                     body = response.body ? await response.json() : undefined;

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

@@ -452,6 +452,22 @@ describe('Provider', () => {
         assert.ok(providerResponse);
         assert.ok(providerResponse.body instanceof ReadableStream);
     });
+    it('returns successfully on unexpected content-type response with no body', async (context) => {
+        const response = new Response(null, {
+            status: 201,
+            headers: { 'Content-Type': 'html/text' },
+        });
+        context.mock.method(global, 'fetch', () => Promise.resolve(response));
+        const providerResponse = await provider.post('/endpoint/123', {}, {
+            credentials: { apiKey: 'apikey#1111' },
+            logger: logger,
+            signal: new AbortController().signal,
+        });
+        assert.ok(providerResponse);
+        assert.strictEqual(providerResponse.status, response.status);
+        assert.strictEqual(providerResponse.headers, response.headers);
+        assert.strictEqual(providerResponse.body, undefined);
+    });
     it('throws on invalid json response', async (context) => {
         const response = new Response('{invalidJSON}', {
             status: 200,

package/package.json

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

package/src/resources/provider.ts

@@ -390,6 +390,9 @@ export class Provider {
       if (response.status >= 400) {
         const textResult = await response.text();
         throw this.handleError(response.status, textResult, options);
+      } else if (response.status === 204 || response.body === null) {
+        // No content: return without inspecting the body
+        return { status: response.status, headers: response.headers, body: undefined as unknown as T };
       }
 
       const responseContentType = response.headers.get('content-type');
@@ -407,7 +410,7 @@ export class Provider {
           throw this.handleError(
             500,
             `Unsupported content-type, expected 'application/json' but got '${responseContentType}'.
-             Original response (${response.status}): ${textResult}`,
+             Original response (${response.status}): "${textResult}"`,
             options,
           );
         }

package/test/resources/provider.test.ts

@@ -539,6 +539,30 @@ describe('Provider', () => {
     assert.ok(providerResponse.body instanceof ReadableStream);
   });
 
+  it('returns successfully on unexpected content-type response with no body', async context => {
+    const response = new Response(null, {
+      status: 201,
+      headers: { 'Content-Type': 'html/text' },
+    });
+
+    context.mock.method(global, 'fetch', () => Promise.resolve(response));
+
+    const providerResponse = await provider.post(
+      '/endpoint/123',
+      {},
+      {
+        credentials: { apiKey: 'apikey#1111' },
+        logger: logger,
+        signal: new AbortController().signal,
+      },
+    );
+
+    assert.ok(providerResponse);
+    assert.strictEqual(providerResponse.status, response.status);
+    assert.strictEqual(providerResponse.headers, response.headers);
+    assert.strictEqual(providerResponse.body, undefined);
+  });
+
   it('throws on invalid json response', async context => {
     const response = new Response('{invalidJSON}', {
       status: 200,