Merge pull request #444 from Nick-1234531/DOT-6966

create separate function for upload logs in caps, reduced debug logs

Author: sushobhit-lt

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lambdatest/smartui-cli",
-  "version": "4.1.47",
+  "version": "4.1.48",
   "description": "A command line interface (CLI) to run SmartUI tests on LambdaTest",
   "files": [
     "dist/**/*"

src/lib/httpClient.ts

@@ -112,10 +112,10 @@ export default class httpClient {
 
     async request(config: AxiosRequestConfig, log: Logger): Promise<Record<string, any>> {
         log.debug(`http request: ${config.method} ${config.url}`);
-        if (config && config.data && !config.data.skipLogging && !config.data.name && !config.data.snapshot) {
+        if (config && config.data && !config.data.skipLogging && !config.data.name && !config.data.snapshot && config.method!=='PUT') {
             log.debug(config.data);
         }
-        if (config && config.data && !config.data.skipLogging && config.data.snapshotUuid) {
+        if (config && config.data && !config.data.skipLogging && config.data.snapshotUuid && config.method!=='PUT') {
             log.debug(config.data);
         }
         return this.axiosInstance.request(config)
@@ -615,7 +615,24 @@ export default class httpClient {
     }
 
     uploadLogs(ctx: Context, uploadURL: string) {
-        const logContent = fs.readFileSync(constants.LOG_FILE_PATH);
+        const fileStream = fs.createReadStream(constants.LOG_FILE_PATH);
+        const { size } = fs.statSync(constants.LOG_FILE_PATH);
+
+        return this.request({
+            url: uploadURL,
+            method: 'PUT',
+            headers: {
+                'Content-Type': 'text/plain',
+                'Content-Length': size,
+            },
+            data: fileStream,
+            maxBodyLength: Infinity, // prevent axios from limiting the body size
+            maxContentLength: Infinity, // prevent axios from limiting the content size
+        }, ctx.log)
+    }
+
+    uploadLogsForCaps(ctx: Context, uploadURL: string) {
+        const logContent = fs.readFileSync(constants.LOG_FILE_PATH)
         const { size } = fs.statSync(constants.LOG_FILE_PATH);
 
         return this.request({

src/tasks/finalizeBuild.ts

@@ -64,7 +64,7 @@ export default (ctx: Context): ListrTask<Context, ListrRendererFactory, ListrRen
                             if (!uploadCLILogsToS3) {
                                 ctx.log.debug(`Log file to be uploaded`)
                                 let resp = await ctx.client.getS3PreSignedURLForCaps(ctx, buildId, projectToken);
-                                await ctx.client.uploadLogs(ctx, resp.data.url);
+                                await ctx.client.uploadLogsForCaps(ctx, resp.data.url);
                             } else {
                                 ctx.log.debug(`Log file to be uploaded via LSRS`)
                                 ctx.client.sendCliLogsToLSRSForCaps(ctx, buildId, projectToken);