zudello-integration-sdk 1.0.33 → 1.0.34

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "zudello-integration-sdk",
-	"version": "1.0.33",
+	"version": "1.0.34",
 	"description": "Zudello Integrations SDK",
 	"main": "./src/index.js",
 	"repository": {

package/src/utils/apiInstance.js

@@ -11,13 +11,20 @@ class ApiInstance {
    */
   constructor () {
     this.bearerToken = null
-    this.retryStatuses = [502, 504]
+    this.retryStatuses = [502, 504, 408, 429, 500]
 
     this.responseHandler = new ResponseHandler()
 
     this.axiosInstance = axios.create({
-      httpsAgent: new https.Agent({ keepAlive: true })
+      httpsAgent: new https.Agent({ 
+        keepAlive: true,
+        timeout: 60000, // 60 seconds timeout
+        maxSockets: 100,
+        maxFreeSockets: 10,
+      }),
+      timeout: 30000, // 30 seconds timeout for the request itself
     })
+
     this.axiosInstance.interceptors.request.use((config) => {
       if (this.bearerToken) {
         config.headers.Authorization = `Bearer ${this.bearerToken}`
@@ -27,6 +34,35 @@ class ApiInstance {
     }, (error) => {
       return Promise.reject(error)
     })
+
+    // Add response interceptor for global error handling
+    this.axiosInstance.interceptors.response.use(
+      response => response,
+      error => {
+        // Check if the error is a network error or ECONNRESET
+        if (error.code === 'ECONNRESET' || error.code === 'ECONNABORTED' || error.message === 'socket hang up') {
+          error.shouldRetry = true
+        }
+
+        return Promise.reject(error)
+      }
+    )
+  }
+
+  /**
+   * Determines if a request should be retried based on the error
+   * @param {Error} error The error that occurred
+   * @returns {boolean} Whether to retry the request
+   */
+  shouldRetry(error) {
+    // Retry on network errors and specific HTTP status codes
+    return (
+      error.shouldRetry || // Set by our interceptor for network errors
+      this.retryStatuses.includes(error?.response?.status) ||
+      (error.code === 'ECONNRESET') ||
+      (error.message === 'socket hang up') ||
+      (error.code === 'ECONNABORTED')
+    )
   }
 
   /**
@@ -75,7 +111,7 @@ class ApiInstance {
       callback(CircularJSON.stringify(error, null, 2))
       callback(CircularJSON.stringify(error.response, null, 2))
 
-      if (this.retryStatuses.includes(error?.response?.status) && retryCount === 0) {
+      if (this.shouldRetry(error) && retryCount === 0) {
         await this.sleep(20000)
         return await this.get(url, params, headers, returnHeaders, callback, retryCount + 1)
       }
@@ -135,7 +171,7 @@ class ApiInstance {
       callback(CircularJSON.stringify(error, null, 2))
       callback(CircularJSON.stringify(error.response, null, 2))
 
-      if (this.retryStatuses.includes(error?.response?.status) && retryCount === 0) {
+      if (this.shouldRetry(error) && retryCount === 0) {
         await this.sleep(20000)
         return await this.post(url, data, headers, params, callback, retryCount + 1)
       }
@@ -180,7 +216,7 @@ class ApiInstance {
 
       callback(error?.response)
 
-      if (this.retryStatuses.includes(error?.response?.status) && retryCount === 0) {
+      if (this.shouldRetry(error) && retryCount === 0) {
         await this.sleep(20000)
         return await this.put(url, data, headers, params, callback, retryCount + 1)
       }
@@ -216,7 +252,7 @@ class ApiInstance {
 
       callback(error?.response)
 
-      if (this.retryStatuses.includes(error?.response?.status) && retryCount === 0) {
+      if (this.shouldRetry(error) && retryCount === 0) {
         await this.sleep(20000)
         return await this.delete(url, callback, retryCount + 1)
       }