@unito/integration-sdk 1.0.16 → 1.0.19

package/dist/src/handler.js

@@ -1,6 +1,6 @@
 import { Router } from 'express';
 import { InvalidHandler } from './errors.js';
-import { UnauthorizedError, BadRequestError } from './httpErrors.js';
+import { HttpError, UnauthorizedError, BadRequestError } from './httpErrors.js';
 import busboy from 'busboy';
 function assertValidPath(path) {
     if (!path.startsWith('/')) {
@@ -136,21 +136,34 @@ export class Handler {
                  */
                 const bb = busboy({ headers: req.headers });
                 bb.on('file', async (_name, file, info) => {
-                    const createdBlob = await handler({
-                        credentials: res.locals.credentials,
-                        secrets: res.locals.secrets,
-                        body: {
-                            file: file,
-                            mimeType: info.mimeType,
-                            encoding: info.encoding,
-                            filename: info.filename,
-                        },
-                        logger: res.locals.logger,
-                        signal: res.locals.signal,
-                        params: req.params,
-                        query: req.query,
-                    });
-                    res.status(201).send(createdBlob);
+                    try {
+                        const createdBlob = await handler({
+                            credentials: res.locals.credentials,
+                            secrets: res.locals.secrets,
+                            body: {
+                                file: file,
+                                mimeType: info.mimeType,
+                                encoding: info.encoding,
+                                filename: info.filename,
+                            },
+                            logger: res.locals.logger,
+                            signal: res.locals.signal,
+                            params: req.params,
+                            query: req.query,
+                        });
+                        res.status(201).send(createdBlob);
+                    }
+                    catch (error) {
+                        if (error instanceof HttpError) {
+                            res.status(error.status).send(error);
+                        }
+                        else {
+                            res.status(500).send({ message: `Error creating the blob: ${error}` });
+                        }
+                    }
+                });
+                bb.on('error', error => {
+                    res.status(500).send({ message: `Error parsing the form data: ${error}` });
                 });
                 req.pipe(bb);
             });

package/dist/src/httpErrors.d.ts

@@ -46,7 +46,7 @@ export declare class UnprocessableEntityError extends HttpError {
     constructor(message?: string);
 }
 /**
- * Used to generate a 429 Unprocessable Entity. Usually used when an operation triggers or would trigger a rate limit
+ * Used to generate a 429 Rate Limit Exceeded. Usually used when an operation triggers or would trigger a rate limit
  * error on the provider's side.
  */
 export declare class RateLimitExceededError extends HttpError {

package/dist/src/httpErrors.js

@@ -62,7 +62,7 @@ export class UnprocessableEntityError extends HttpError {
     }
 }
 /**
- * Used to generate a 429 Unprocessable Entity. Usually used when an operation triggers or would trigger a rate limit
+ * Used to generate a 429 Rate Limit Exceeded. Usually used when an operation triggers or would trigger a rate limit
  * error on the provider's side.
  */
 export class RateLimitExceededError extends HttpError {

package/dist/src/index.cjs

@@ -295,7 +295,7 @@ class UnprocessableEntityError extends HttpError {
     }
 }
 /**
- * Used to generate a 429 Unprocessable Entity. Usually used when an operation triggers or would trigger a rate limit
+ * Used to generate a 429 Rate Limit Exceeded. Usually used when an operation triggers or would trigger a rate limit
  * error on the provider's side.
  */
 class RateLimitExceededError extends HttpError {
@@ -683,21 +683,34 @@ class Handler {
                  */
                 const bb = busboy({ headers: req.headers });
                 bb.on('file', async (_name, file, info) => {
-                    const createdBlob = await handler({
-                        credentials: res.locals.credentials,
-                        secrets: res.locals.secrets,
-                        body: {
-                            file: file,
-                            mimeType: info.mimeType,
-                            encoding: info.encoding,
-                            filename: info.filename,
-                        },
-                        logger: res.locals.logger,
-                        signal: res.locals.signal,
-                        params: req.params,
-                        query: req.query,
-                    });
-                    res.status(201).send(createdBlob);
+                    try {
+                        const createdBlob = await handler({
+                            credentials: res.locals.credentials,
+                            secrets: res.locals.secrets,
+                            body: {
+                                file: file,
+                                mimeType: info.mimeType,
+                                encoding: info.encoding,
+                                filename: info.filename,
+                            },
+                            logger: res.locals.logger,
+                            signal: res.locals.signal,
+                            params: req.params,
+                            query: req.query,
+                        });
+                        res.status(201).send(createdBlob);
+                    }
+                    catch (error) {
+                        if (error instanceof HttpError) {
+                            res.status(error.status).send(error);
+                        }
+                        else {
+                            res.status(500).send({ message: `Error creating the blob: ${error}` });
+                        }
+                    }
+                });
+                bb.on('error', error => {
+                    res.status(500).send({ message: `Error parsing the form data: ${error}` });
                 });
                 req.pipe(bb);
             });
@@ -1142,7 +1155,10 @@ class Provider {
                         response.on('end', () => {
                             try {
                                 const body = JSON.parse(responseBody);
-                                resolve({ status: 201, headers: response.headers, body });
+                                if (body.error) {
+                                    reject(this.handleError(400, body.error.message));
+                                }
+                                resolve({ status: 201, headers: response.headers, body: body });
                             }
                             catch (error) {
                                 reject(this.handleError(500, `Failed to parse response body: "${error}"`));
@@ -1283,7 +1299,9 @@ class Provider {
                 // (Provider's response Content-Type might be more specific, e.g. application/json;charset=utf-8)
                 // Default to application/json if no Content-Type header is provided
                 if (responseContentType && !responseContentType.includes('application/json')) {
-                    throw this.handleError(500, `Unsupported content-type. Expected 'application/json', got '${responseContentType}'`);
+                    const textResult = await response.text();
+                    throw this.handleError(500, `Unsupported content-type, expected 'application/json' but got '${responseContentType}'.
+             Original response (${response.status}): ${textResult}`);
                 }
                 try {
                     body = response.body ? await response.json() : undefined;

package/dist/src/resources/provider.js

@@ -136,7 +136,10 @@ export class Provider {
                         response.on('end', () => {
                             try {
                                 const body = JSON.parse(responseBody);
-                                resolve({ status: 201, headers: response.headers, body });
+                                if (body.error) {
+                                    reject(this.handleError(400, body.error.message));
+                                }
+                                resolve({ status: 201, headers: response.headers, body: body });
                             }
                             catch (error) {
                                 reject(this.handleError(500, `Failed to parse response body: "${error}"`));
@@ -277,7 +280,9 @@ export class Provider {
                 // (Provider's response Content-Type might be more specific, e.g. application/json;charset=utf-8)
                 // Default to application/json if no Content-Type header is provided
                 if (responseContentType && !responseContentType.includes('application/json')) {
-                    throw this.handleError(500, `Unsupported content-type. Expected 'application/json', got '${responseContentType}'`);
+                    const textResult = await response.text();
+                    throw this.handleError(500, `Unsupported content-type, expected 'application/json' but got '${responseContentType}'.
+             Original response (${response.status}): ${textResult}`);
                 }
                 try {
                     body = response.body ? await response.json() : undefined;

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

@@ -401,7 +401,7 @@ describe('Provider', () => {
             error = e;
         }
         assert.ok(error instanceof HttpErrors.HttpError);
-        assert.equal(error.message, `Unsupported content-type. Expected 'application/json', got 'application/text'`);
+        assert.equal(error.status, 500);
     });
     it('throws on status 400', async (context) => {
         const response = new Response('response body', {

package/package.json

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

package/src/handler.ts

@@ -1,7 +1,7 @@
 import { Router } from 'express';
 import * as API from '@unito/integration-api';
 import { InvalidHandler } from './errors.js';
-import { UnauthorizedError, BadRequestError } from './httpErrors.js';
+import { HttpError, UnauthorizedError, BadRequestError } from './httpErrors.js';
 import {
   GetBlobContext,
   GetItemContext,
@@ -351,21 +351,33 @@ export class Handler {
          */
         const bb = busboy({ headers: req.headers });
         bb.on('file', async (_name, file, info: FileInfo) => {
-          const createdBlob = await handler({
-            credentials: res.locals.credentials,
-            secrets: res.locals.secrets,
-            body: {
-              file: file,
-              mimeType: info.mimeType,
-              encoding: info.encoding,
-              filename: info.filename,
-            },
-            logger: res.locals.logger,
-            signal: res.locals.signal,
-            params: req.params,
-            query: req.query,
-          });
-          res.status(201).send(createdBlob);
+          try {
+            const createdBlob = await handler({
+              credentials: res.locals.credentials,
+              secrets: res.locals.secrets,
+              body: {
+                file: file,
+                mimeType: info.mimeType,
+                encoding: info.encoding,
+                filename: info.filename,
+              },
+              logger: res.locals.logger,
+              signal: res.locals.signal,
+              params: req.params,
+              query: req.query,
+            });
+            res.status(201).send(createdBlob);
+          } catch (error) {
+            if (error instanceof HttpError) {
+              res.status((error as HttpError).status).send(error);
+            } else {
+              res.status(500).send({ message: `Error creating the blob: ${error}` });
+            }
+          }
+        });
+
+        bb.on('error', error => {
+          res.status(500).send({ message: `Error parsing the form data: ${error}` });
         });
 
         req.pipe(bb);

package/src/httpErrors.ts

@@ -70,7 +70,7 @@ export class UnprocessableEntityError extends HttpError {
 }
 
 /**
- * Used to generate a 429 Unprocessable Entity. Usually used when an operation triggers or would trigger a rate limit
+ * Used to generate a 429 Rate Limit Exceeded. Usually used when an operation triggers or would trigger a rate limit
  * error on the provider's side.
  */
 export class RateLimitExceededError extends HttpError {

package/src/resources/provider.ts

@@ -209,8 +209,11 @@ export class Provider {
 
             response.on('end', () => {
               try {
-                const body = JSON.parse(responseBody) as T;
-                resolve({ status: 201, headers: response.headers as unknown as Headers, body });
+                const body = JSON.parse(responseBody);
+                if (body.error) {
+                  reject(this.handleError(400, body.error.message));
+                }
+                resolve({ status: 201, headers: response.headers as unknown as Headers, body: body as T });
               } catch (error) {
                 reject(this.handleError(500, `Failed to parse response body: "${error}"`));
               }
@@ -372,9 +375,11 @@ export class Provider {
         // (Provider's response Content-Type might be more specific, e.g. application/json;charset=utf-8)
         // Default to application/json if no Content-Type header is provided
         if (responseContentType && !responseContentType.includes('application/json')) {
+          const textResult = await response.text();
           throw this.handleError(
             500,
-            `Unsupported content-type. Expected 'application/json', got '${responseContentType}'`,
+            `Unsupported content-type, expected 'application/json' but got '${responseContentType}'.
+             Original response (${response.status}): ${textResult}`,
           );
         }
 

package/test/resources/provider.test.ts

@@ -479,7 +479,7 @@ describe('Provider', () => {
     }
 
     assert.ok(error instanceof HttpErrors.HttpError);
-    assert.equal(error.message, `Unsupported content-type. Expected 'application/json', got 'application/text'`);
+    assert.equal(error.status, 500);
   });
 
   it('throws on status 400', async context => {