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.


git-svn-id: https://develop.svn.wordpress.org/trunk@61507 602fd350-edb4-49c9-b593-d223f7449a82

Author: youknowriad

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

@@ -104,7 +104,7 @@ jobs:
         env:
           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
             echo "Performance results could not be published. 'CODEVITALS_PROJECT_TOKEN' is not set"

.github/workflows/reusable-performance.yml

@@ -347,7 +347,7 @@ jobs:
         env:
           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
             echo "Performance results could not be published. 'CODEVITALS_PROJECT_TOKEN' is not set"

tests/performance/log-results.js

@@ -10,7 +10,6 @@
 /**
  * External dependencies.
  */
-const https = require( 'https' );
 const [ token, branch, hash, baseHash, date, host ] =
 	process.argv.slice( 2 );
 const { median, parseFile, accumulateValues } = require( './utils' );
@@ -82,40 +81,40 @@ for ( const { title, results } of afterStats ) {
 	}
 }
 
-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 );
-} );
-
-req.write( data );
-req.end();
+( async () => {
+	try {
+		const response = await fetch(
+			`https://${ host }/api/log?token=${ token }`,
+			{
+				method: 'POST',
+				headers: {
+					'Content-Type': 'application/json',
+				},
+				body: data,
+			}
+		);
+
+		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 );
+	}
+} )();