testlens-playwright-reporter 0.1.2 → 0.1.5

package/index.js

@@ -56,12 +56,12 @@ class TestLensReporter {
         'Content-Type': 'application/json',
         ...(this.config.apiKey && { 'X-API-Key': this.config.apiKey }),
       },
-      // Enhanced SSL handling
+      // Enhanced SSL handling with flexible TLS configuration
       httpsAgent: new https.Agent({
         rejectUnauthorized: rejectUnauthorized,
-        // Additional SSL options for better compatibility
-        secureProtocol: 'TLSv1_2_method',
-        ciphers: 'ECDHE-RSA-AES128-GCM-SHA256:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-RSA-AES128-SHA256:ECDHE-RSA-AES256-SHA384'
+        // Allow any TLS version for better compatibility
+        minVersion: 'TLSv1.2',
+        maxVersion: 'TLSv1.3'
       })
     });
 
@@ -589,51 +589,108 @@ class TestLensReporter {
 
   async uploadArtifactToS3(filePath, testId, fileName) {
     try {
-      // Use the public S3 upload API endpoint
+      // Check file size first
+      const fileSize = this.getFileSize(filePath);
+      const fileSizeMB = (fileSize / (1024 * 1024)).toFixed(2);
+      
+      console.log(`📤 Uploading ${fileName} (${fileSizeMB}MB) directly to S3...`);
+
       const baseUrl = this.config.apiEndpoint.replace('/api/v1/webhook/playwright', '');
-      const uploadUrl = `${baseUrl}/api/v1/artifacts/public/upload`;
-
-      // Prepare form data for multipart upload
-      const form = new FormData();
-
-      // Add required fields
-      form.append('apiKey', this.config.apiKey);
-      form.append('testRunId', this.runId);
-      form.append('testId', testId);
-      form.append('artifactType', this.getArtifactType(fileName));
-      form.append('file', fs.createReadStream(filePath), {
-        filename: fileName,
-        contentType: this.getContentType(fileName)
+      
+      // Step 1: Request pre-signed URL from server
+      const presignedUrlEndpoint = `${baseUrl}/api/v1/artifacts/public/presigned-url`;
+      const presignedResponse = await this.axiosInstance.post(presignedUrlEndpoint, {
+        apiKey: this.config.apiKey,
+        testRunId: this.runId,
+        testId: testId,
+        fileName: fileName,
+        fileType: this.getContentType(fileName),
+        fileSize: fileSize,
+        artifactType: this.getArtifactType(fileName)
+      }, {
+        timeout: 10000 // Quick timeout for metadata request
       });
 
-      console.log(`📤 Uploading ${fileName} to TestLens S3 via API...`);
+      if (!presignedResponse.data.success) {
+        throw new Error(`Failed to get presigned URL: ${presignedResponse.data.error || 'Unknown error'}`);
+      }
 
-      // Make the upload request
-      const response = await this.axiosInstance.post(uploadUrl, form, {
+      const { uploadUrl, s3Key, metadata } = presignedResponse.data;
+      
+      // Step 2: Upload directly to S3 using presigned URL
+      console.log(`⬆️  Uploading ${fileName} directly to S3 (bypass server)...`);
+      
+      const fileBuffer = fs.readFileSync(filePath);
+      
+      // IMPORTANT: When using presigned URLs, we MUST include exactly the headers that were signed
+      // The backend signs with ServerSideEncryption:'AES256', so we must send that header
+      // AWS presigned URLs are very strict about header matching
+      const uploadResponse = await axios.put(uploadUrl, fileBuffer, {
         headers: {
-          ...form.getHeaders(),
-          // Override content-type to let form-data set it
-          'Content-Type': `multipart/form-data; boundary=${form.getBoundary()}`
+          'x-amz-server-side-encryption': 'AES256'
         },
-        timeout: 60000 // 60 second timeout for uploads
+        maxContentLength: Infinity,
+        maxBodyLength: Infinity,
+        timeout: Math.max(300000, Math.ceil(fileSize / (1024 * 1024)) * 5000), // 5s per MB, min 5 minutes
+        // Don't use custom HTTPS agent for S3 uploads
+        httpsAgent: undefined
       });
 
-      if (response.status === 201 && response.data.success) {
-        const artifact = response.data.artifact;
-        console.log(`✅ S3 upload completed for ${fileName}`);
+      if (uploadResponse.status !== 200) {
+        throw new Error(`S3 upload failed with status ${uploadResponse.status}`);
+      }
+
+      console.log(`✅ S3 direct upload completed for ${fileName}`);
+
+      // Step 3: Confirm upload with server to save metadata
+      const confirmEndpoint = `${baseUrl}/api/v1/artifacts/public/confirm-upload`;
+      const confirmResponse = await this.axiosInstance.post(confirmEndpoint, {
+        apiKey: this.config.apiKey,
+        testRunId: this.runId,
+        testId: testId,
+        s3Key: s3Key,
+        fileName: fileName,
+        fileType: this.getContentType(fileName),
+        fileSize: fileSize,
+        artifactType: this.getArtifactType(fileName)
+      }, {
+        timeout: 10000
+      });
+
+      if (confirmResponse.status === 201 && confirmResponse.data.success) {
+        const artifact = confirmResponse.data.artifact;
+        console.log(`✅ Upload confirmed and saved to database`);
         return {
-          key: artifact.s3Key,
+          key: s3Key,
           url: artifact.s3Url,
           presignedUrl: artifact.presignedUrl,
           fileSize: artifact.fileSize,
           contentType: artifact.contentType
         };
       } else {
-        throw new Error(`Upload failed: ${response.data.error || 'Unknown error'}`);
+        throw new Error(`Upload confirmation failed: ${confirmResponse.data.error || 'Unknown error'}`);
       }
     } catch (error) {
-      console.error(`❌ Failed to upload ${fileName} to TestLens S3 API:`, error.message);
-      throw error; // Re-throw to prevent fallback to local storage
+      // Better error messages for common issues
+      let errorMsg = error.message;
+      
+      if (error.code === 'ECONNABORTED' && error.message.includes('timeout')) {
+        errorMsg = `Upload timeout - file may be too large or connection is slow`;
+      } else if (error.response?.status === 413) {
+        errorMsg = `File too large (413) - server rejected the upload`;
+      } else if (error.response?.status === 400) {
+        errorMsg = `Bad request (400) - ${error.response.data?.error || 'check file format'}`;
+      } else if (error.response?.status === 403) {
+        errorMsg = `Access denied (403) - presigned URL may have expired`;
+      }
+      
+      console.error(`❌ Failed to upload ${fileName} to S3:`, errorMsg);
+      if (error.response?.data) {
+        console.error('Error details:', error.response.data);
+      }
+      
+      // Don't throw, just return null to continue with other artifacts
+      return null;
     }
   }
 

package/index.ts

@@ -185,12 +185,12 @@ export class TestLensReporter implements Reporter {
         'Content-Type': 'application/json',
         ...(this.config.apiKey && { 'X-API-Key': this.config.apiKey }),
       },
-      // Enhanced SSL handling
+      // Enhanced SSL handling with flexible TLS configuration
       httpsAgent: new https.Agent({
         rejectUnauthorized: rejectUnauthorized,
-        // Additional SSL options for better compatibility
-        secureProtocol: 'TLSv1_2_method',
-        ciphers: 'ECDHE-RSA-AES128-GCM-SHA256:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-RSA-AES128-SHA256:ECDHE-RSA-AES256-SHA384'
+        // Allow any TLS version for better compatibility
+        minVersion: 'TLSv1.2',
+        maxVersion: 'TLSv1.3'
       })
     });
 
@@ -472,6 +472,12 @@ export class TestLensReporter implements Reporter {
           // Upload to S3 first
           const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
           
+          // Skip if upload failed or file was too large
+          if (!s3Data) {
+            console.log(`⏭️  Skipping artifact ${attachment.name} - upload failed or file too large`);
+            continue;
+          }
+          
           const artifactData = {
             testId,
             type: this.getArtifactType(attachment.name),
@@ -668,44 +674,79 @@ export class TestLensReporter implements Reporter {
   }
 
 
-  private async uploadArtifactToS3(filePath: string, testId: string, fileName: string): Promise<{ key: string; url: string; presignedUrl: string; fileSize: number; contentType: string }> {
+  private async uploadArtifactToS3(filePath: string, testId: string, fileName: string): Promise<{ key: string; url: string; presignedUrl: string; fileSize: number; contentType: string } | null> {
     try {
-      // Use the new public S3 upload API endpoint instead of direct AWS SDK calls
+      // Check file size first
+      const fileSize = this.getFileSize(filePath);
+      const fileSizeMB = (fileSize / (1024 * 1024)).toFixed(2);
+      
+      console.log(`📤 Uploading ${fileName} (${fileSizeMB}MB) directly to S3...`);
+
       const baseUrl = this.config.apiEndpoint.replace('/api/v1/webhook/playwright', '');
-      const uploadUrl = `${baseUrl}/api/v1/artifacts/public/upload`;
-
-      // Prepare form data for multipart upload
-      const form = new FormData();
-
-      // Add required fields
-      form.append('apiKey', this.config.apiKey);
-      form.append('testRunId', this.runId);
-      form.append('testId', testId);
-      form.append('artifactType', this.getArtifactType(fileName));
-      form.append('file', fs.createReadStream(filePath), {
-        filename: fileName,
-        contentType: this.getContentType(fileName)
+      
+      // Step 1: Request pre-signed URL from server
+      const presignedUrlEndpoint = `${baseUrl}/api/v1/artifacts/public/presigned-url`;
+      const presignedResponse = await this.axiosInstance.post(presignedUrlEndpoint, {
+        apiKey: this.config.apiKey,
+        testRunId: this.runId,
+        testId: testId,
+        fileName: fileName,
+        fileType: this.getContentType(fileName),
+        fileSize: fileSize,
+        artifactType: this.getArtifactType(fileName)
+      }, {
+        timeout: 10000 // Quick timeout for metadata request
       });
 
-      console.log(`📤 Uploading ${fileName} to TestLens S3 via API...`);
+      if (!presignedResponse.data.success) {
+        throw new Error(`Failed to get presigned URL: ${presignedResponse.data.error || 'Unknown error'}`);
+      }
 
-      // Make the upload request
-      const response = await this.axiosInstance.post(uploadUrl, form, {
+      const { uploadUrl, s3Key, metadata } = presignedResponse.data;
+      
+      // Step 2: Upload directly to S3 using presigned URL
+      console.log(`⬆️  Uploading ${fileName} directly to S3 (bypass server)...`);
+      
+      const fileBuffer = fs.readFileSync(filePath);
+      
+      // IMPORTANT: When using presigned URLs, we MUST include exactly the headers that were signed
+      // The backend signs with ServerSideEncryption:'AES256', so we must send that header
+      // AWS presigned URLs are very strict about header matching
+      const uploadResponse = await axios.put(uploadUrl, fileBuffer, {
         headers: {
-          ...form.getHeaders(),
-          // Override content-type to let form-data set it
-          'Content-Type': `multipart/form-data; boundary=${form.getBoundary()}`
+          'x-amz-server-side-encryption': 'AES256'
         },
-        timeout: 60000 // 60 second timeout for uploads
+        maxContentLength: Infinity,
+        maxBodyLength: Infinity,
+        timeout: Math.max(300000, Math.ceil(fileSize / (1024 * 1024)) * 5000), // 5s per MB, min 5 minutes
+        // Don't use custom HTTPS agent for S3 uploads
+        httpsAgent: undefined
       });
 
-      if (response.status === 201 && response.data.success) {
-        const artifact = response.data.artifact;
-        console.log(`✅ S3 upload completed for ${fileName}`);
+      if (uploadResponse.status !== 200) {
+        throw new Error(`S3 upload failed with status ${uploadResponse.status}`);
+      }
 
-        // Extract S3 key from URL if not provided
-        const s3Key = artifact.s3Key || artifact.s3Url.split('/').slice(-1)[0];
+      console.log(`✅ S3 direct upload completed for ${fileName}`);
+
+      // Step 3: Confirm upload with server to save metadata
+      const confirmEndpoint = `${baseUrl}/api/v1/artifacts/public/confirm-upload`;
+      const confirmResponse = await this.axiosInstance.post(confirmEndpoint, {
+        apiKey: this.config.apiKey,
+        testRunId: this.runId,
+        testId: testId,
+        s3Key: s3Key,
+        fileName: fileName,
+        fileType: this.getContentType(fileName),
+        fileSize: fileSize,
+        artifactType: this.getArtifactType(fileName)
+      }, {
+        timeout: 10000
+      });
 
+      if (confirmResponse.status === 201 && confirmResponse.data.success) {
+        const artifact = confirmResponse.data.artifact;
+        console.log(`✅ Upload confirmed and saved to database`);
         return {
           key: s3Key,
           url: artifact.s3Url,
@@ -714,11 +755,29 @@ export class TestLensReporter implements Reporter {
           contentType: artifact.contentType
         };
       } else {
-        throw new Error(`Upload failed: ${response.data.error || 'Unknown error'}`);
+        throw new Error(`Upload confirmation failed: ${confirmResponse.data.error || 'Unknown error'}`);
       }
-    } catch (error) {
-      console.error(`❌ Failed to upload ${fileName} to TestLens S3 API:`, (error as Error).message);
-      throw error; // Re-throw to prevent fallback to local storage
+    } catch (error: any) {
+      // Better error messages for common issues
+      let errorMsg = error.message;
+      
+      if (error.code === 'ECONNABORTED' && error.message.includes('timeout')) {
+        errorMsg = `Upload timeout - file may be too large or connection is slow`;
+      } else if (error.response?.status === 413) {
+        errorMsg = `File too large (413) - server rejected the upload`;
+      } else if (error.response?.status === 400) {
+        errorMsg = `Bad request (400) - ${error.response.data?.error || 'check file format'}`;
+      } else if (error.response?.status === 403) {
+        errorMsg = `Access denied (403) - presigned URL may have expired`;
+      }
+      
+      console.error(`❌ Failed to upload ${fileName} to S3:`, errorMsg);
+      if (error.response?.data) {
+        console.error('Error details:', error.response.data);
+      }
+      
+      // Don't throw, just return null to continue with other artifacts
+      return null;
     }
   }
 

package/lib/index.js

@@ -45,7 +45,6 @@ const https = __importStar(require("https"));
 const axios_1 = __importDefault(require("axios"));
 const child_process_1 = require("child_process");
 const mime = __importStar(require("mime"));
-const form_data_1 = __importDefault(require("form-data"));
 class TestLensReporter {
     constructor(options) {
         this.config = {
@@ -93,12 +92,12 @@ class TestLensReporter {
                 'Content-Type': 'application/json',
                 ...(this.config.apiKey && { 'X-API-Key': this.config.apiKey }),
             },
-            // Enhanced SSL handling
+            // Enhanced SSL handling with flexible TLS configuration
             httpsAgent: new https.Agent({
                 rejectUnauthorized: rejectUnauthorized,
-                // Additional SSL options for better compatibility
-                secureProtocol: 'TLSv1_2_method',
-                ciphers: 'ECDHE-RSA-AES128-GCM-SHA256:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-RSA-AES128-SHA256:ECDHE-RSA-AES256-SHA384'
+                // Allow any TLS version for better compatibility
+                minVersion: 'TLSv1.2',
+                maxVersion: 'TLSv1.3'
             })
         });
         // Add retry interceptor
@@ -342,6 +341,11 @@ class TestLensReporter {
                 try {
                     // Upload to S3 first
                     const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
+                    // Skip if upload failed or file was too large
+                    if (!s3Data) {
+                        console.log(`⏭️  Skipping artifact ${attachment.name} - upload failed or file too large`);
+                        continue;
+                    }
                     const artifactData = {
                         testId,
                         type: this.getArtifactType(attachment.name),
@@ -521,35 +525,65 @@ class TestLensReporter {
     }
     async uploadArtifactToS3(filePath, testId, fileName) {
         try {
-            // Use the new public S3 upload API endpoint instead of direct AWS SDK calls
+            // Check file size first
+            const fileSize = this.getFileSize(filePath);
+            const fileSizeMB = (fileSize / (1024 * 1024)).toFixed(2);
+            console.log(`📤 Uploading ${fileName} (${fileSizeMB}MB) directly to S3...`);
             const baseUrl = this.config.apiEndpoint.replace('/api/v1/webhook/playwright', '');
-            const uploadUrl = `${baseUrl}/api/v1/artifacts/public/upload`;
-            // Prepare form data for multipart upload
-            const form = new form_data_1.default();
-            // Add required fields
-            form.append('apiKey', this.config.apiKey);
-            form.append('testRunId', this.runId);
-            form.append('testId', testId);
-            form.append('artifactType', this.getArtifactType(fileName));
-            form.append('file', fs.createReadStream(filePath), {
-                filename: fileName,
-                contentType: this.getContentType(fileName)
+            // Step 1: Request pre-signed URL from server
+            const presignedUrlEndpoint = `${baseUrl}/api/v1/artifacts/public/presigned-url`;
+            const presignedResponse = await this.axiosInstance.post(presignedUrlEndpoint, {
+                apiKey: this.config.apiKey,
+                testRunId: this.runId,
+                testId: testId,
+                fileName: fileName,
+                fileType: this.getContentType(fileName),
+                fileSize: fileSize,
+                artifactType: this.getArtifactType(fileName)
+            }, {
+                timeout: 10000 // Quick timeout for metadata request
             });
-            console.log(`📤 Uploading ${fileName} to TestLens S3 via API...`);
-            // Make the upload request
-            const response = await this.axiosInstance.post(uploadUrl, form, {
+            if (!presignedResponse.data.success) {
+                throw new Error(`Failed to get presigned URL: ${presignedResponse.data.error || 'Unknown error'}`);
+            }
+            const { uploadUrl, s3Key, metadata } = presignedResponse.data;
+            // Step 2: Upload directly to S3 using presigned URL
+            console.log(`⬆️  Uploading ${fileName} directly to S3 (bypass server)...`);
+            const fileBuffer = fs.readFileSync(filePath);
+            // IMPORTANT: When using presigned URLs, we MUST include exactly the headers that were signed
+            // The backend signs with ServerSideEncryption:'AES256', so we must send that header
+            // AWS presigned URLs are very strict about header matching
+            const uploadResponse = await axios_1.default.put(uploadUrl, fileBuffer, {
                 headers: {
-                    ...form.getHeaders(),
-                    // Override content-type to let form-data set it
-                    'Content-Type': `multipart/form-data; boundary=${form.getBoundary()}`
+                    'x-amz-server-side-encryption': 'AES256'
                 },
-                timeout: 60000 // 60 second timeout for uploads
+                maxContentLength: Infinity,
+                maxBodyLength: Infinity,
+                timeout: Math.max(300000, Math.ceil(fileSize / (1024 * 1024)) * 5000), // 5s per MB, min 5 minutes
+                // Don't use custom HTTPS agent for S3 uploads
+                httpsAgent: undefined
             });
-            if (response.status === 201 && response.data.success) {
-                const artifact = response.data.artifact;
-                console.log(`✅ S3 upload completed for ${fileName}`);
-                // Extract S3 key from URL if not provided
-                const s3Key = artifact.s3Key || artifact.s3Url.split('/').slice(-1)[0];
+            if (uploadResponse.status !== 200) {
+                throw new Error(`S3 upload failed with status ${uploadResponse.status}`);
+            }
+            console.log(`✅ S3 direct upload completed for ${fileName}`);
+            // Step 3: Confirm upload with server to save metadata
+            const confirmEndpoint = `${baseUrl}/api/v1/artifacts/public/confirm-upload`;
+            const confirmResponse = await this.axiosInstance.post(confirmEndpoint, {
+                apiKey: this.config.apiKey,
+                testRunId: this.runId,
+                testId: testId,
+                s3Key: s3Key,
+                fileName: fileName,
+                fileType: this.getContentType(fileName),
+                fileSize: fileSize,
+                artifactType: this.getArtifactType(fileName)
+            }, {
+                timeout: 10000
+            });
+            if (confirmResponse.status === 201 && confirmResponse.data.success) {
+                const artifact = confirmResponse.data.artifact;
+                console.log(`✅ Upload confirmed and saved to database`);
                 return {
                     key: s3Key,
                     url: artifact.s3Url,
@@ -559,12 +593,30 @@ class TestLensReporter {
                 };
             }
             else {
-                throw new Error(`Upload failed: ${response.data.error || 'Unknown error'}`);
+                throw new Error(`Upload confirmation failed: ${confirmResponse.data.error || 'Unknown error'}`);
             }
         }
         catch (error) {
-            console.error(`❌ Failed to upload ${fileName} to TestLens S3 API:`, error.message);
-            throw error; // Re-throw to prevent fallback to local storage
+            // Better error messages for common issues
+            let errorMsg = error.message;
+            if (error.code === 'ECONNABORTED' && error.message.includes('timeout')) {
+                errorMsg = `Upload timeout - file may be too large or connection is slow`;
+            }
+            else if (error.response?.status === 413) {
+                errorMsg = `File too large (413) - server rejected the upload`;
+            }
+            else if (error.response?.status === 400) {
+                errorMsg = `Bad request (400) - ${error.response.data?.error || 'check file format'}`;
+            }
+            else if (error.response?.status === 403) {
+                errorMsg = `Access denied (403) - presigned URL may have expired`;
+            }
+            console.error(`❌ Failed to upload ${fileName} to S3:`, errorMsg);
+            if (error.response?.data) {
+                console.error('Error details:', error.response.data);
+            }
+            // Don't throw, just return null to continue with other artifacts
+            return null;
         }
     }
     getContentType(fileName) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testlens-playwright-reporter",
-  "version": "0.1.2",
+  "version": "0.1.5",
   "description": "Universal Playwright reporter for TestLens - works with both TypeScript and JavaScript projects",
   "main": "lib/index.js",
   "types": "index.d.ts",