sitespeed.io 21.1.0 → 21.2.0

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # CHANGELOG - sitespeed.io  (we use [semantic versioning](https://semver.org))
 
+## 21.2.0 - 2021-12-22
+### Fixed
+* The catching of errors in the queue was broken and reported the error x times (x=numbert of plugins). Also when we had an error the result JSON was not stored. [#3522](https://github.com/sitespeedio/sitespeed.io/pull/3522).
+
+### Added
+* Updated to Firefox 95 and Edge 96 in the Docker container.
+* Include page summary URL in the result JSON [#3523](https://github.com/sitespeedio/sitespeed.io/pull/3523).
 
 ## 21.1.0 - 2021-12-06
 ### Added

package/Dockerfile

@@ -1,4 +1,4 @@
-FROM sitespeedio/webbrowsers:chrome-96.0-firefox-94.0-edge-95.0-dev
+FROM sitespeedio/webbrowsers:chrome-96.0-firefox-95.0-edge-96.0-dev
 
 ENV SITESPEED_IO_BROWSERTIME__XVFB true
 ENV SITESPEED_IO_BROWSERTIME__DOCKER true

package/Dockerfile-slim

@@ -6,7 +6,7 @@ ENV SITESPEED_IO_BROWSERTIME__BROWSER firefox
 ENV SITESPEED_IO_BROWSERTIME__VISUAL_METRICS false
 ENV SITESPEED_IO_BROWSERTIME__HEADLESS true
 
-ENV FIREFOX_VERSION 94.0
+ENV FIREFOX_VERSION 95.0
 
 ENV PATH="/usr/local/bin:${PATH}"
 

package/bin/sitespeed.js

@@ -14,9 +14,6 @@ async function run(options) {
   process.exitCode = 1;
   try {
     const result = await sitespeed.run(options);
-    if (result.errors.length > 0) {
-      throw new Error('Errors while running:\n' + result.errors.join('\n'));
-    }
 
     if (options.storeResult) {
       if (options.storeResult != 'true') {
@@ -26,6 +23,10 @@ async function run(options) {
       }
     }
 
+    if (result.errors.length > 0) {
+      throw new Error('Errors while running:\n' + result.errors.join('\n'));
+    }
+
     if ((options.open || options.o) && os.platform() === 'darwin') {
       execSync('open ' + result.localPath + '/index.html');
     }

package/lib/core/queueHandler.js

@@ -125,10 +125,6 @@ class QueueHandler {
         // FIXME handle rejections (i.e. failures while processing messages) properly
         queue.processingEnded(obj => {
           const message = obj.item;
-          // if we get a error message on the queue make sure we register that
-          if (message.type === 'error' || message.type.endsWith('.error')) {
-            this.errors.push(message.data);
-          }
           const err = obj.err;
           if (err) {
             let rejectionMessage =
@@ -197,6 +193,11 @@ class QueueHandler {
   postMessage(message) {
     validateMessageFormat(message);
 
+    // if we get a error message on the queue make sure we register that
+    if (message.type === 'error' || message.type.endsWith('.error')) {
+      this.errors.push('' + message.data);
+    }
+
     // Don't return promise in loop - we don't want to wait for completion,
     // just post the message.
     for (let item of this.queues) {

package/lib/sitespeed.js

@@ -163,6 +163,10 @@ module.exports = {
         resultUrl: resultUrls.hasBaseUrl()
           ? resultUrls.reportSummaryUrl() + '/index.html'
           : '',
+        pageSummaryUrl:
+          url && resultUrls.hasBaseUrl()
+            ? resultUrls.absoluteSummaryPageUrl(url) + 'index.html'
+            : '',
         localPath: storageManager.getBaseDir()
       };
     } catch (err) {

package/npm-shrinkwrap.json

@@ -1,19 +1,19 @@
 {
   "name": "sitespeed.io",
-  "version": "21.1.0",
+  "version": "21.2.0",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "sitespeed.io",
-      "version": "21.1.0",
+      "version": "21.2.0",
       "license": "MIT",
       "dependencies": {
         "@google-cloud/storage": "5.8.3",
         "@tgwf/co2": "0.8.0",
         "aws-sdk": "2.882.0",
         "axe-core": "4.3.5",
-        "browsertime": "14.12.1",
+        "browsertime": "14.12.2",
         "cli-color": "2.0.0",
         "coach-core": "7.0.0",
         "concurrent-queue": "7.0.2",
@@ -1465,9 +1465,9 @@
       }
     },
     "node_modules/browsertime": {
-      "version": "14.12.1",
-      "resolved": "https://registry.npmjs.org/browsertime/-/browsertime-14.12.1.tgz",
-      "integrity": "sha512-qWEGcJw9N2/x1oMYNkH/+m3KyYHNT7rQ2iNlkgC9NYb3khNdPhaV1FGRD5dQIiRgwnCDpiKWBgBSbtTfEMAkLA==",
+      "version": "14.12.2",
+      "resolved": "https://registry.npmjs.org/browsertime/-/browsertime-14.12.2.tgz",
+      "integrity": "sha512-y/ouU8ejg6OuxSw4GXVIHLL83vw65/45O5yWaYmj8rtoqaeghBzWsG+KiCtu5zJRH5B1SD+zM5x47hmxNYFvzQ==",
       "dependencies": {
         "@cypress/xvfb": "1.2.4",
         "@devicefarmer/adbkit": "2.11.3",
@@ -8631,9 +8631,9 @@
       }
     },
     "browsertime": {
-      "version": "14.12.1",
-      "resolved": "https://registry.npmjs.org/browsertime/-/browsertime-14.12.1.tgz",
-      "integrity": "sha512-qWEGcJw9N2/x1oMYNkH/+m3KyYHNT7rQ2iNlkgC9NYb3khNdPhaV1FGRD5dQIiRgwnCDpiKWBgBSbtTfEMAkLA==",
+      "version": "14.12.2",
+      "resolved": "https://registry.npmjs.org/browsertime/-/browsertime-14.12.2.tgz",
+      "integrity": "sha512-y/ouU8ejg6OuxSw4GXVIHLL83vw65/45O5yWaYmj8rtoqaeghBzWsG+KiCtu5zJRH5B1SD+zM5x47hmxNYFvzQ==",
       "requires": {
         "@cypress/xvfb": "1.2.4",
         "@devicefarmer/adbkit": "2.11.3",

package/package.json

@@ -4,7 +4,7 @@
     "sitespeed.io": "./bin/sitespeed.js",
     "sitespeed.io-wpr": "./bin/browsertimeWebPageReplay.js"
   },
-  "version": "21.1.0",
+  "version": "21.2.0",
   "description": "Analyze the web performance of your site",
   "keywords": [
     "performance",
@@ -74,7 +74,7 @@
     "@tgwf/co2": "0.8.0",
     "aws-sdk": "2.882.0",
     "axe-core": "4.3.5",
-    "browsertime": "14.12.1",
+    "browsertime": "14.12.2",
     "coach-core": "7.0.0",
     "cli-color": "2.0.0",
     "concurrent-queue": "7.0.2",