@alternative-path/testlens-playwright-reporter 0.3.8 → 0.3.9

package/index.d.ts

@@ -157,6 +157,11 @@ export interface SpecData {
 export declare class TestLensReporter implements Reporter {
     private config;
     private axiosInstance;
+    /**
+     * Automatically detect and load system CA certificates
+     * Uses Node.js built-in root certificates (works across all platforms)
+     */
+    private static getSystemCaCertificates;
     private runId;
     private runMetadata;
     private specMap;

package/index.js

@@ -7,6 +7,7 @@ const os = tslib_1.__importStar(require("os"));
 const path = tslib_1.__importStar(require("path"));
 const fs = tslib_1.__importStar(require("fs"));
 const https = tslib_1.__importStar(require("https"));
+const tls = tslib_1.__importStar(require("tls"));
 const axios_1 = tslib_1.__importDefault(require("axios"));
 const child_process_1 = require("child_process");
 // Lazy-load mime module to support ESM
@@ -22,23 +23,33 @@ async function getMime() {
 class TestLensReporter {
     /**
      * Automatically detect and load system CA certificates
-     * Tries multiple common system certificate locations
+     * Uses Node.js built-in root certificates (works across all platforms)
      */
     static getSystemCaCertificates() {
-        const certificates = [];
+        // Use Node.js's built-in root certificates (available in Node.js 12.3.0+)
+        // This includes all common root CAs and works reliably across Windows, macOS, and Linux
+        try {
+            if (tls.rootCertificates && Array.isArray(tls.rootCertificates) && tls.rootCertificates.length > 0) {
+                // Convert string certificates to Buffer
+                const rootCerts = tls.rootCertificates.map(cert => Buffer.from(cert));
+                if (process.env.DEBUG) {
+                    console.log(`✓ Using Node.js built-in root certificates (${rootCerts.length} certificates)`);
+                }
+                return rootCerts;
+            }
+        }
+        catch (error) {
+            // Fall through to file-based detection if tls.rootCertificates is not available
+            if (process.env.DEBUG) {
+                console.log('⚠️  Node.js built-in root certificates not available, trying file-based detection');
+            }
+        }
+        // Fallback: Try to load from file system (for older Node.js versions or special cases)
         const platform = os.platform();
-        // Common certificate store locations by platform
+        const certificates = [];
+        // Fallback: Try to load certificates from common file system locations
         const certPaths = [];
-        if (platform === 'win32') {
-            // Windows: Node.js uses Windows certificate store automatically
-            // But we can also check common locations
-            const winPaths = [
-                process.env.CERT_PATH,
-                'C:\\Windows\\System32\\certmgr.msc', // Not a file, but indicates cert store
-            ];
-            certPaths.push(...winPaths.filter(Boolean));
-        }
-        else if (platform === 'darwin') {
+        if (platform === 'darwin') {
             // macOS: Common certificate locations
             certPaths.push('/etc/ssl/cert.pem', '/usr/local/etc/openssl/cert.pem', '/opt/homebrew/etc/openssl/cert.pem', '/System/Library/OpenSSL/certs/cert.pem');
         }
@@ -63,7 +74,9 @@ class TestLensReporter {
                 // This is expected as not all paths will exist on every system
             }
         }
-        return certificates;
+        // Return undefined if no certificates found (let Node.js use defaults)
+        // Return certificates array if we found any
+        return certificates.length > 0 ? certificates : undefined;
     }
     /**
      * Parse custom metadata from environment variables
@@ -151,27 +164,34 @@ class TestLensReporter {
             try {
                 if (fs.existsSync(this.config.caCertificate)) {
                     const customCert = fs.readFileSync(this.config.caCertificate);
-                    // Combine system certs with custom cert
-                    ca = systemCerts.length > 0 ? [...systemCerts, customCert] : customCert;
+                    // Combine system certs with custom cert if system certs are available
+                    if (systemCerts && Array.isArray(systemCerts) && systemCerts.length > 0) {
+                        ca = [...systemCerts, customCert];
+                    }
+                    else {
+                        ca = customCert;
+                    }
                     console.log(`✓ Using custom CA certificate: ${this.config.caCertificate}`);
                 }
                 else {
                     console.warn(`⚠️  CA certificate file not found: ${this.config.caCertificate}`);
                     // Fall back to system certs if custom cert not found
-                    ca = systemCerts.length > 0 ? systemCerts : undefined;
+                    ca = systemCerts || undefined;
                 }
             }
             catch (error) {
                 console.warn(`⚠️  Failed to read CA certificate file: ${this.config.caCertificate}`, error.message);
                 // Fall back to system certs if custom cert read fails
-                ca = systemCerts.length > 0 ? systemCerts : undefined;
+                ca = systemCerts || undefined;
             }
         }
         else {
             // Use automatically detected system certificates
-            ca = systemCerts.length > 0 ? systemCerts : undefined;
-            // Node.js will use system default CA store if we don't specify 'ca'
-            // But providing system certs explicitly can help in some environments
+            // On Windows, systemCerts will be undefined to let Node.js use Windows certificate store
+            // On Unix systems, systemCerts will contain certificates or be undefined
+            ca = systemCerts;
+            // If ca is undefined, Node.js will use its default certificate store
+            // This is the correct behavior, especially on Windows
         }
         // Check various ways SSL validation can be disabled (in order of precedence)
         if (this.config.ignoreSslErrors) {
@@ -196,10 +216,17 @@ class TestLensReporter {
             minVersion: 'TLSv1.2',
             maxVersion: 'TLSv1.3'
         };
-        // Add CA certificates if available (Node.js will use system defaults if not specified)
-        // Explicitly setting 'ca' can help in environments where Node.js doesn't find certs automatically
-        if (ca && ca.length > 0) {
-            httpsAgentOptions.ca = ca;
+        // Add CA certificates if available
+        // On Windows, ca will be undefined to let Node.js use Windows certificate store automatically
+        // On Unix systems, ca will contain certificates if found, or undefined to use Node.js defaults
+        if (ca !== undefined) {
+            if (Array.isArray(ca) && ca.length > 0) {
+                httpsAgentOptions.ca = ca;
+            }
+            else if (typeof ca === 'string' || Buffer.isBuffer(ca)) {
+                httpsAgentOptions.ca = ca;
+            }
+            // If ca is undefined, we don't set it, allowing Node.js to use its default certificate store
         }
         this.axiosInstance = axios_1.default.create({
             baseURL: this.config.apiEndpoint,

package/index.ts

@@ -3,6 +3,7 @@ import * as os from 'os';
 import * as path from 'path';
 import * as fs from 'fs';
 import * as https from 'https';
+import * as tls from 'tls';
 import axios, { AxiosInstance } from 'axios';
 import type { Reporter, TestCase, TestResult, FullConfig, Suite } from '@playwright/test/reporter';
 import { execSync } from 'child_process';
@@ -185,24 +186,35 @@ export class TestLensReporter implements Reporter {
 
   /**
    * Automatically detect and load system CA certificates
-   * Tries multiple common system certificate locations
+   * Uses Node.js built-in root certificates (works across all platforms)
    */
-  private static getSystemCaCertificates(): Buffer[] {
-    const certificates: Buffer[] = [];
+  private static getSystemCaCertificates(): Buffer[] | undefined {
+    // Use Node.js's built-in root certificates (available in Node.js 12.3.0+)
+    // This includes all common root CAs and works reliably across Windows, macOS, and Linux
+    try {
+      if (tls.rootCertificates && Array.isArray(tls.rootCertificates) && tls.rootCertificates.length > 0) {
+        // Convert string certificates to Buffer
+        const rootCerts = tls.rootCertificates.map(cert => Buffer.from(cert));
+        if (process.env.DEBUG) {
+          console.log(`✓ Using Node.js built-in root certificates (${rootCerts.length} certificates)`);
+        }
+        return rootCerts;
+      }
+    } catch (error) {
+      // Fall through to file-based detection if tls.rootCertificates is not available
+      if (process.env.DEBUG) {
+        console.log('⚠️  Node.js built-in root certificates not available, trying file-based detection');
+      }
+    }
+
+    // Fallback: Try to load from file system (for older Node.js versions or special cases)
     const platform = os.platform();
+    const certificates: Buffer[] = [];
 
-    // Common certificate store locations by platform
+    // Fallback: Try to load certificates from common file system locations
     const certPaths: string[] = [];
 
-    if (platform === 'win32') {
-      // Windows: Node.js uses Windows certificate store automatically
-      // But we can also check common locations
-      const winPaths = [
-        process.env.CERT_PATH,
-        'C:\\Windows\\System32\\certmgr.msc', // Not a file, but indicates cert store
-      ];
-      certPaths.push(...winPaths.filter(Boolean) as string[]);
-    } else if (platform === 'darwin') {
+    if (platform === 'darwin') {
       // macOS: Common certificate locations
       certPaths.push(
         '/etc/ssl/cert.pem',
@@ -240,7 +252,9 @@ export class TestLensReporter implements Reporter {
       }
     }
 
-    return certificates;
+    // Return undefined if no certificates found (let Node.js use defaults)
+    // Return certificates array if we found any
+    return certificates.length > 0 ? certificates : undefined;
   }
   private runId: string;
   private runMetadata: RunMetadata;
@@ -343,24 +357,30 @@ export class TestLensReporter implements Reporter {
       try {
         if (fs.existsSync(this.config.caCertificate)) {
           const customCert = fs.readFileSync(this.config.caCertificate);
-          // Combine system certs with custom cert
-          ca = systemCerts.length > 0 ? [...systemCerts, customCert] : customCert;
+          // Combine system certs with custom cert if system certs are available
+          if (systemCerts && Array.isArray(systemCerts) && systemCerts.length > 0) {
+            ca = [...systemCerts, customCert];
+          } else {
+            ca = customCert;
+          }
           console.log(`✓ Using custom CA certificate: ${this.config.caCertificate}`);
         } else {
           console.warn(`⚠️  CA certificate file not found: ${this.config.caCertificate}`);
           // Fall back to system certs if custom cert not found
-          ca = systemCerts.length > 0 ? systemCerts : undefined;
+          ca = systemCerts || undefined;
         }
       } catch (error) {
         console.warn(`⚠️  Failed to read CA certificate file: ${this.config.caCertificate}`, (error as Error).message);
         // Fall back to system certs if custom cert read fails
-        ca = systemCerts.length > 0 ? systemCerts : undefined;
+        ca = systemCerts || undefined;
       }
     } else {
       // Use automatically detected system certificates
-      ca = systemCerts.length > 0 ? systemCerts : undefined;
-      // Node.js will use system default CA store if we don't specify 'ca'
-      // But providing system certs explicitly can help in some environments
+      // On Windows, systemCerts will be undefined to let Node.js use Windows certificate store
+      // On Unix systems, systemCerts will contain certificates or be undefined
+      ca = systemCerts;
+      // If ca is undefined, Node.js will use its default certificate store
+      // This is the correct behavior, especially on Windows
     }
 
     // Check various ways SSL validation can be disabled (in order of precedence)
@@ -386,10 +406,16 @@ export class TestLensReporter implements Reporter {
       maxVersion: 'TLSv1.3'
     };
 
-    // Add CA certificates if available (Node.js will use system defaults if not specified)
-    // Explicitly setting 'ca' can help in environments where Node.js doesn't find certs automatically
-    if (ca && ca.length > 0) {
-      httpsAgentOptions.ca = ca;
+    // Add CA certificates if available
+    // On Windows, ca will be undefined to let Node.js use Windows certificate store automatically
+    // On Unix systems, ca will contain certificates if found, or undefined to use Node.js defaults
+    if (ca !== undefined) {
+      if (Array.isArray(ca) && ca.length > 0) {
+        httpsAgentOptions.ca = ca;
+      } else if (typeof ca === 'string' || Buffer.isBuffer(ca)) {
+        httpsAgentOptions.ca = ca;
+      }
+      // If ca is undefined, we don't set it, allowing Node.js to use its default certificate store
     }
 
     this.axiosInstance = axios.create({

package/package.json

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