Merge pull request #62 from contentstack/fix/DX-1492

Fix: getData to receive params and headers both in data

Author: nadeem-cs

CHANGELOG.md

@@ -1,3 +1,8 @@
+## Change log
+### Version: 1.1.3
+#### Date: Oct-22-2024
+ - Fix: getData to receive params and headers both in data
+
 ## Change log
 ### Version: 1.1.2
 #### Date: Oct-22-2024

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentstack/core",
-  "version": "1.1.2",
+  "version": "1.1.3",
   "type": "commonjs",
   "main": "./dist/cjs/src/index.js",
   "types": "./dist/cjs/src/index.d.ts",

src/lib/request.ts

@@ -23,7 +23,7 @@ export async function getData(instance: AxiosInstance, url: string, data?: any)
         }
       }
     }
-    const response = await instance.get(url, { params: data });
+    const response = await instance.get(url, data);
 
     if (response && response.data) {
       return response.data;

src/lib/retryPolicy/delivery-sdk-handlers.ts

@@ -50,9 +50,10 @@ export const retryResponseErrorHandler = (error: any, config: any, axiosInstance
         if (error.response && error.response.data) {
           return Promise.reject(error.response.data);
         }
+
         return Promise.reject(error);
       }
-        error.config.retryCount = retryCount;
+      error.config.retryCount = retryCount;
 
       return axiosInstance(error.config);
     }