Changeset 61507

Timestamp:

01/21/2026 05:23:44 PM (8 days ago)

Author:

youknowriad

Message:

Build: Fix redirect and error handling in performance results logging.

• Replace https.request() with native fetch() in log-results.js.
• Drop www. from host name used to avoid redirects.

Props mcsf.
Fixes #64534.

Location:

trunk

Files:

• .github/workflows/reusable-performance-report-v2.yml (modified) (1 diff)
• .github/workflows/reusable-performance.yml (modified) (1 diff)
• tests/performance/log-results.js (modified) (2 diffs)

trunk/.github/workflows/reusable-performance-report-v2.yml

@@ -105 +105 @@
           BASE_SHA: ${{ steps.base-sha.outputs.result }}
           CODEVITALS_PROJECT_TOKEN: ${{ secrets.CODEVITALS_PROJECT_TOKEN }}
-          HOST_NAME: www.codevitals.run
+          HOST_NAME: codevitals.run
         run: |
           if [ -z "$CODEVITALS_PROJECT_TOKEN" ]; then

trunk/.github/workflows/reusable-performance.yml

@@ -348 +348 @@
           BASE_SHA: ${{ steps.base-sha.outputs.result }}
           CODEVITALS_PROJECT_TOKEN: ${{ secrets.CODEVITALS_PROJECT_TOKEN }}
-          HOST_NAME: "www.codevitals.run"
+          HOST_NAME: "codevitals.run"
         run: |
           if [ -z "$CODEVITALS_PROJECT_TOKEN" ]; then

trunk/tests/performance/log-results.js

@@ -11 +11 @@
  * External dependencies.
  */
-const https = require( 'https' );
 const [ token, branch, hash, baseHash, date, host ] =
     process.argv.slice( 2 );
@@ -83 +82 @@
 }
 
-const data = new TextEncoder().encode(
-    JSON.stringify( {
-        branch,
-        hash,
-        baseHash,
-        timestamp: date,
-        metrics: metrics,
-        baseMetrics: baseMetrics,
-    } )
-);
-
-const options = {
-    hostname: host,
-    port: 443,
-    path: '/api/log?token=' + token,
-    method: 'POST',
-    headers: {
-        'Content-Type': 'application/json',
-        'Content-Length': data.length,
-    },
-};
-
-const req = https.request( options, ( res ) => {
-    console.log( `statusCode: ${ res.statusCode }` );
-
-    res.on( 'data', ( d ) => {
-        process.stdout.write( d );
-    } );
+const data = JSON.stringify( {
+    branch,
+    hash,
+    baseHash,
+    timestamp: date,
+    metrics: metrics,
+    baseMetrics: baseMetrics,
 } );
 
-req.on( 'error', ( error ) => {
-    console.error( error );
-    process.exit( 1 );
-} );
+( async () => {
+    try {
+        const response = await fetch(
+            `https://${ host }/api/log?token=${ token }`,
+            {
+                method: 'POST',
+                headers: {
+                    'Content-Type': 'application/json',
+                },
+                body: data,
+            }
+        );
 
-req.write( data );
-req.end();
+        console.log( `statusCode: ${ response.status }` );
+
+        const responseText = await response.text();
+        if ( responseText ) {
+            console.log( responseText );
+        }
+
+        if ( ! response.ok ) {
+            process.exit( 1 );
+        }
+    } catch ( error ) {
+        console.error( error );
+        process.exit( 1 );
+    }
+} )();