feat: http request timeout option (#271)

README.md

@@ -407,6 +407,21 @@ async function test() {
 test();
 ```
 
+### Http request timeout
+
+By default request timeout will vary on http client implementation and/or environment (e.g. `fetch` uses timeout configured by the browser).  
+But there is an option to set constant value:
+
+```javascript
+const crowdin = require('@crowdin/crowdin-api-client');
+
+const credentials = { token: 'token' };
+
+const httpRequestTimeout = 60 * 1000; // 60 seconds
+
+const client = new crowdin.default(credentials, { httpRequestTimeout });
+```
+
 ## Over-The-Air Content Delivery
 
 :dizzy: Recommended for translations delivery to your website or mobile application.

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@crowdin/crowdin-api-client",
-    "version": "1.23.0",
+    "version": "1.23.1",
     "description": "JavaScript library for Crowdin API",
     "main": "out/index.js",
     "types": "out/index.d.ts",

src/core/index.ts

@@ -51,6 +51,9 @@ export interface ClientConfig {
 
     /** Retry strategy configuration */
     retryConfig?: RetryConfig;
+
+    /** Http request timeout in ms */
+    httpRequestTimeout?: number;
 }
 
 export interface ResponseList<T> {
@@ -228,6 +231,11 @@ export abstract class CrowdinApi {
         }
         this.retryService = new RetryService(retryConfig);
 
+        if (config?.httpRequestTimeout) {
+            CrowdinApi.FETCH_INSTANCE.withTimeout(config?.httpRequestTimeout);
+            CrowdinApi.AXIOS_INSTANCE.defaults.timeout = config?.httpRequestTimeout;
+        }
+
         this.config = config;
     }
 
@@ -286,8 +294,7 @@ export abstract class CrowdinApi {
         return CrowdinApi.AXIOS_INSTANCE;
     }
 
-    // eslint-disable-next-line @typescript-eslint/explicit-function-return-type
-    public withFetchAll(maxLimit?: number) {
+    public withFetchAll(maxLimit?: number): this {
         this.fetchAllFlag = true;
         this.maxLimit = maxLimit;
         return this;

src/core/internal/fetch/fetchClient.ts

@@ -14,6 +14,13 @@ export class FetchClient implements HttpClient {
     private requestIntervalMs = 10;
     private pendingRequests = 0;
 
+    private timeout: number | undefined;
+
+    withTimeout(timeout?: number): this {
+        this.timeout = timeout;
+        return this;
+    }
+
     get<T>(url: string, config?: { headers: Record<string, string> }): Promise<T> {
         return this.request(url, 'GET', config);
     }
@@ -52,11 +59,21 @@ export class FetchClient implements HttpClient {
         }
         await this.waitInQueue();
 
-        return fetch(url, {
-            method: method,
-            headers: config ? config.headers : {},
-            body: body,
-        })
+        let request;
+        const headers = config ? config.headers : {};
+
+        if (this.timeout) {
+            const controller = new AbortController();
+            const timeoutId = setTimeout(() => controller.abort(), this.timeout);
+            request = fetch(url, { method, headers, body, signal: controller.signal }).then(res => {
+                clearTimeout(timeoutId);
+                return res;
+            });
+        } else {
+            request = fetch(url, { method, headers, body });
+        }
+
+        return request
             .then(async res => {
                 if (res.status === 204) {
                     return {};