sitespeed.io 34.2.0 → 34.2.2

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # CHANGELOG - sitespeed.io  (we use [semantic versioning](https://semver.org))
 
+## 34.2.2 - 2024-06-18
+### Fixed
+* Make it possible to disable the API. In some cases where you extends another configuration file, you could end up in a API loop where the API calls the API [#4191](https://github.com/sitespeedio/sitespeed.io/pull/4191).
+
+## 34.2.1 - 2024-06-17
+### Fixed
+* API: safer parsing if something fail to get better error messages [#4188](https://github.com/sitespeedio/sitespeed.io/pull/4188).
+
 ## 34.2.0 - 2024-06-14
 
 ### Added

package/bin/sitespeed.js

@@ -132,7 +132,7 @@ async function start() {
 
   let options = parsed.options;
 
-  if (options.api && options.api.hostname) {
+  if (options.api && options.api.hostname && !options.disableAPI) {
     api(options);
   } else {
     try {

package/lib/api/send.js

@@ -24,13 +24,21 @@ export async function addTest(hostname, options) {
       let data = [];
       res.on('data', chunk => data.push(chunk));
       res.on('end', () => {
-        return res.statusCode === 200
-          ? resolve(Buffer.concat(data).toString('utf8'))
-          : reject(JSON.parse(Buffer.concat(data).toString('utf8')));
+        if (res.statusCode === 200) {
+          return resolve(Buffer.concat(data).toString('utf8'));
+        } else {
+          try {
+            return reject(JSON.parse(Buffer.concat(data).toString('utf8')));
+          } catch {
+            return reject({
+              message:
+                'Could parse result:' + Buffer.concat(data).toString('utf8')
+            });
+          }
+        }
       });
     });
     request.on('error', error => {
-      console.log('onERror');
       reject({ message: error.toString() });
     });
     request.write(JSON.stringify(postData));
@@ -69,9 +77,18 @@ export async function get(testId, hostname, options) {
         });
 
         res.on('end', () => {
-          return res.statusCode === 200
-            ? resolve(JSON.parse(Buffer.concat(data).toString('utf8')))
-            : reject(JSON.parse(Buffer.concat(data).toString('utf8')));
+          if (res.statusCode === 200) {
+            return resolve(JSON.parse(Buffer.concat(data).toString('utf8')));
+          } else {
+            try {
+              return reject(JSON.parse(Buffer.concat(data).toString('utf8')));
+            } catch {
+              return reject({
+                message:
+                  'Could parse result:' + Buffer.concat(data).toString('utf8')
+              });
+            }
+          }
         });
       })
       .on('error', () => {

package/lib/cli/cli.js

@@ -2034,9 +2034,14 @@ export async function parseCommandLine() {
       describe:
         'Give your test a slug. The slug is used when you send the metrics to your data storage to identify the test and the folder of the tests. The max length of the slug is 200 characters and it can only contain a-z A-Z 0-9 and -_ characters.'
     })
+    .option('disableAPI', {
+      default: false,
+      type: 'boolean'
+    })
     .help('h')
     .alias('help', 'h')
     .config(config)
+    .hide('disableAPI')
     .alias('version', 'V')
     .version(version)
     .coerce('budget', function (argument) {

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "sitespeed.io",
-  "version": "34.2.0",
+  "version": "34.2.2",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "sitespeed.io",
-      "version": "34.2.0",
+      "version": "34.2.2",
       "license": "MIT",
       "dependencies": {
         "@aws-sdk/client-s3": "3.564.0",

package/package.json

@@ -5,7 +5,7 @@
     "sitespeed.io": "./bin/sitespeed.js",
     "sitespeed.io-wpr": "./bin/browsertimeWebPageReplay.js"
   },
-  "version": "34.2.0",
+  "version": "34.2.2",
   "description": "sitespeed.io is an open-source tool for comprehensive web performance analysis, enabling you to test, monitor, and optimize your website’s speed using real browsers in various environments.",
   "keywords": [
     "performance",