attlaz-client 1.17.5 → 1.17.6

package/dist/Http/Data/QueryString.js

@@ -8,6 +8,10 @@ export class QueryString {
         this.command = command;
     }
     set(parameter, value) {
+        const existingParam = this.parameters.find((param) => param.parameter === parameter);
+        if (existingParam !== undefined) {
+            throw new Error('Param already defined');
+        }
         this.parameters.push({ parameter, value });
     }
     addPagination(pagination) {

package/dist/Http/HttpClient.js

@@ -1,4 +1,4 @@
-import axios from 'axios';
+import axios, { AxiosError } from 'axios';
 import { HttpClientResponse } from './HttpClientResponse.js';
 import { ClientError } from './ClientError.js';
 export class HttpClient {
@@ -13,7 +13,6 @@ export class HttpClient {
     static HTTP_UNAVAILABLE = 503;
     static HTTP_TIMEOUT = 504;
     static async request(request) {
-        // TODo: should we catch axios error sand make them into ClientErrors?
         const response = await HttpClient.axiosRequest(request);
         const error = ClientError.byStatus(response.status, response.statusText);
         if (error !== null && error !== undefined) {
@@ -29,18 +28,37 @@ export class HttpClient {
             url: request.getFullUrl(),
             method: request.method,
             headers: request.headers,
-            data: request.body,
         };
+        if (request.body !== null && request.body !== undefined) {
+            requestConfig.data = request.body;
+        }
         // TODO: should we share this instance between requests?
         const axiosInstance = axios.create();
-        const rawResponse = await axiosInstance.request(requestConfig);
-        const httpResponse = new HttpClientResponse(rawResponse.status, rawResponse.statusText);
-        Object.keys(rawResponse.headers).forEach((header) => {
-            const headerValue = rawResponse.headers[header];
-            httpResponse.headers[header] = headerValue === null ? '' : headerValue;
-        });
-        httpResponse.body = await rawResponse.data;
-        return httpResponse;
+        // TODo: should we catch axios error sand make them into ClientErrors?
+        try {
+            const rawResponse = await axiosInstance.request(requestConfig);
+            const httpResponse = new HttpClientResponse(rawResponse.status, rawResponse.statusText);
+            Object.keys(rawResponse.headers).forEach((header) => {
+                const headerValue = rawResponse.headers[header];
+                httpResponse.headers[header] = headerValue === null ? '' : headerValue;
+            });
+            httpResponse.body = await rawResponse.data;
+            return httpResponse;
+        }
+        catch (e) {
+            if (e instanceof AxiosError) {
+                const status = e.status === undefined ? 500 : e.status;
+                const statusCode = e.code === undefined ? 'Error' : e.code;
+                const response = new HttpClientResponse(status, statusCode);
+                if (e.response !== undefined) {
+                    response.body = e.response.data;
+                }
+                return response;
+            }
+            console.error('Unable to perform request', { error: e });
+            const response = new HttpClientResponse(500, '');
+            return response;
+        }
     }
     static isJson(response) {
         return response.getContentType() === 'application/json';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "attlaz-client",
-  "version": "1.17.5",
+  "version": "1.17.6",
   "description": "Javascript Client to access Attlaz API",
   "types": "./dist/index.d.ts",
   "main": "./dist/index.js",