replacing all request api call by axios

Author: Shahnawaz-Sk

index.js

@@ -1,4 +1,4 @@
 var lambdaAutomationClient = require("./lib/automation_client");
 module.exports = {
     AutomationClient: lambdaAutomationClient.AutomationClient,
-};
+};

lib/api_client.js

@@ -1,13 +1,8 @@
+const axios = require('axios');
 const { version, appVersion, baseUrl, baseUrlApp } = require("./config");
+const packageVersion = require("../package.json").version;
+let logger;
 
-var request = require("request"),
-    logger,
-    packageVersion = require("../package.json").version;
-
-/**
- * ApiClient is a function based Class.
- * @param {!AutomationApiClient} settings An object that for lambda user credentials
- */
 var ApiClient = function (settings) {
     settings = settings || {};
     /**
@@ -28,58 +23,40 @@ var ApiClient = function (settings) {
         logger = require("./logger")(settings.logFile);
     }
     logger.info("Imported ApiClient and User Credenetials", settings);
-    ApiClient.authHeader =
-        "Basic " + Buffer.from(settings.username + ":" + settings.accessKey).toString("base64");
-    ApiClient.settings = settings;
+    ApiClient.basicAuth = "Basic " + Buffer.from(settings.username + ":" + settings.accessKey).toString("base64");
 
-    if (settings.isApp) {
-        ApiClient.baseUrl = baseUrlApp + appVersion.latestVersion;
-    }
-};
+    ApiClient.apiBaseUrl = settings.isApp ? baseUrlApp + appVersion.latestVersion : baseUrl + version.latestVersion;
 
-/**
- * Extends ApiClient Class with static method for all Api Request.
- * @param {!ApiClient} options An object that determine the Request payload
- * @param {Function} fnCallback is callback function.
- * @return {Function|Error} return response to Callable method or throw Error.
- */
-ApiClient.request = function (options, fnCallback) {
-    /** Initialize Callback function is not there */
-    fnCallback = fnCallback || function () {};
-    /** Check baseUrl is on ApiClient */
-    if (ApiClient.baseUrl === undefined) {
-        ApiClient.baseUrl = baseUrl + version.latestVersion;
-        ApiClient.request(options, fnCallback);
-    } else {
-        /** Update Options with User Credential If It is object */
-        if (typeof options === "object") {
-            options.headers = {
-                Authorization: ApiClient.authHeader,
-                "Content-Type": "application/json",
-                Accept: "application/json",
-                client: "npm-rest-client",
-                version: packageVersion,
-            };
-            options.url = ApiClient.baseUrl + options.url;
+    this.makeRequest = async function(method, path, body = null, params = {}) {
+        // Validate path
+        if (!path) {
+            throw new Error('Path is required');
         }
-        /** For Debbuging purpose log Request Payload */
+
+        // Use instance-specific baseUrl
+        const url = `${ApiClient.apiBaseUrl}${path}`;
+        
+        // Prepare request options
+        const options = {
+            method: method,
+            headers: { Authorization: ApiClient.basicAuth },
+            url: url,
+        };
+        
+
+        if (body) options.data = JSON.stringify(body);
+        if (Object.keys(params).length) options.params = params;
+
         logger.info("Api request options ", options);
-        request(options, function (e, response, body) {
-            if (e) {
-                logger.error("Error while Api call of ", e);
-            } else if (response.statusCode === 200) {
-                // use try-catch Error possible due to json parse of non-parseable
-                try {
-                    body = JSON.parse(body);
-                    logger.info("Api response json : ", body);
-                    return fnCallback(e, body);
-                } catch (e) {
-                    logger.error("Error while parse to json of output response ", e);
-                }
-            }
-            return fnCallback(new Error(e || body), null);
-        });
-    }
-};
+        // Axios request
+        try {
+            const response = await axios(options);
+            return response.data;
+        } catch (error) {
+            throw error;
+        }
+    };
 
+};
 module.exports = ApiClient;
+