@testomatio/reporter 1.6.9 → 1.6.11-beta1.s3-fix

package/lib/client.js

@@ -161,34 +161,48 @@ class Client {
       suite_id,
       test_id,
       manuallyAttachedArtifacts,
-      meta,
     } = testData;
-    let { message = '' } = testData;
+    let { message = '', meta = {} } = testData;
 
     // stringify meta values and limit keys and values length to 255
-    if (meta) {
-      for (const key in meta) {
-        if (typeof meta[key] === 'object') {
-          meta[key] = JSON.stringify(meta[key]);
-        }
+    meta = Object.entries(meta)
+      .filter(([, value]) => value !== null && value !== undefined)
+      .map(([key, value]) => {
+        try {
+          if (typeof value === 'object') {
+            value = JSON.stringify(value);
+          } else if (typeof value !== 'string') {
+            try {
+              value = value.toString();
+            } catch (err) {
+              console.warn(APP_PREFIX, `Can't convert meta value to string`, err);
+            }
+          }
 
-        // cut values
-        if (meta[key]?.length > 255) {
-          meta[key] = meta[key].substring(0, 255);
-          debug(APP_PREFIX, `Meta info value "${meta[key]}" is too long, trimmed to 255 characters`);
-        }
+          if (value?.length > 255) {
+            value = value.substring(0, 255);
+            debug(APP_PREFIX, `Meta info value "${value}" is too long, trimmed to 255 characters`);
+          }
 
-        // cut keys
-        if (key?.length > 255) {
-          const newKey = key.substring(0, 255);
-          meta[newKey] = meta[key];
-          delete meta[key];
-          debug(APP_PREFIX, `Meta info key "${key}" is too long, trimmed to 255 characters`);
+          if (key?.length > 255) {
+            const newKey = key.substring(0, 255);
+            debug(APP_PREFIX, `Meta info key "${key}" is too long, trimmed to 255 characters`);
+            return [newKey, value];
+          }
+
+          return [key, value];
+        } catch (err) {
+          debug(APP_PREFIX, `Error while processing meta info key ${key}`, err);
+          return [null, null];
         }
-      }
-    }
+      })
+      .reduce((acc, [key, value]) => {
+        if (key) acc[key] = value;
+        return acc;
+      }, {});
 
     let errorFormatted = '';
+
     if (error) {
       errorFormatted += this.formatError(error) || '';
       message = error?.message;

package/lib/uploader.js

@@ -328,6 +328,11 @@ class S3Uploader {
       }
     }
 
+    // Normalize the URL
+    if (!s3Location.startsWith('http')) {
+      s3Location = `https://${s3Location}`;
+    }
+
     return s3Location;
   }
 
@@ -353,10 +358,13 @@ class S3Uploader {
       credentials: {
         accessKeyId: S3_ACCESS_KEY_ID,
         secretAccessKey: S3_SECRET_ACCESS_KEY,
-        s3ForcePathStyle: S3_FORCE_PATH_STYLE,
       },
     };
 
+    if (S3_FORCE_PATH_STYLE) {
+      cfg.forcePathStyle = !['false', '0'].includes(String(S3_FORCE_PATH_STYLE || '').toLowerCase());
+    }
+
     if (S3_SESSION_TOKEN) {
       cfg.credentials.sessionToken = S3_SESSION_TOKEN;
     }
@@ -367,6 +375,7 @@ class S3Uploader {
 
     return cfg;
   }
+
 }
 
 module.exports = S3Uploader;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "1.6.9",
+  "version": "1.6.11-beta1.s3-fix",
   "description": "Testomatio Reporter Client",
   "main": "./lib/reporter.js",
   "typings": "typings/index.d.ts",