Changeset 61507

Timestamp:

01/21/2026 05:23:44 PM (8 months 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 );
+        }
+} )();