sitespeed.io 20.3.1 → 20.4.0

package/.github/workflows/docker.yml

@@ -19,6 +19,8 @@ jobs:
       run: docker run --rm sitespeedio/sitespeed.io https://www.sitespeed.io -n 1 -b chrome
     - name: Run test on default container for Firefox 
       run: docker run --rm sitespeedio/sitespeed.io https://www.sitespeed.io -n 1 -b firefox
+    - name: Run test on default container for Edge 
+      run: docker run --rm sitespeedio/sitespeed.io https://www.sitespeed.io -n 1 -b edge
     - name: Run test on slim container  
       run: docker run --rm sitespeedio/sitespeed.io:slim https://www.sitespeed.io -n 1
     - name: Test WebPageReplay in the default container    

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # CHANGELOG - sitespeed.io  (we use [semantic versioning](https://semver.org))
 
+
+## 20.4.0 - 2021-10-29
+
+### Added
+* Updated to [Browsertime 14.8.0](https://github.com/sitespeedio/browsertime/blob/main/CHANGELOG.md) with Edgeriver 95.
+* Updated to Edge 95 in the Docker container.
+* Update to use NodeJS 16 in the Docker container [#3495](https://github.com/sitespeedio/sitespeed.io/pull/3495).
+* On Mac OS: Use `--open` or `-o` to open the result in your default browser. Thank you [Johanna Lindh](https://github.com/zhouhana) for the idea! PR [#3493](https://github.com/sitespeedio/sitespeed.io/pull/3493).
+* The +1 container now had Lighthouse 8.6.0.
+
+### Fixed
+* Removed npm from the Docker container when everything has been installed to fix npm security issues + decrease the size of the container.
 ## 20.3.1 - 2021-10-21
 ### Fixed
 * Fix to handle integer value Android device serial, thank you [Saurav Kumar](https://github.com/svkrclg) for PR [#3490](https://github.com/sitespeedio/sitespeed.io/pull/3490).

package/Dockerfile

@@ -1,4 +1,4 @@
-FROM sitespeedio/webbrowsers:chrome-95.0-firefox-93.0-edge-93.0-dev
+FROM sitespeedio/webbrowsers:chrome-95.0-firefox-93.0-edge-95.0-dev-2
 
 ENV SITESPEED_IO_BROWSERTIME__XVFB true
 ENV SITESPEED_IO_BROWSERTIME__DOCKER true
@@ -25,7 +25,7 @@ RUN mkdir -p /usr/src/app
 WORKDIR /usr/src/app
 
 COPY package.* /usr/src/app/
-RUN npm install --production
+RUN npm install --production && npm cache clean --force && npm uninstall npm -g
 COPY . /usr/src/app
 
 COPY docker/scripts/start.sh /start.sh
@@ -35,6 +35,7 @@ RUN mkdir -m 0750 /root/.android
 ADD docker/adb/insecure_shared_adbkey /root/.android/adbkey
 ADD docker/adb/insecure_shared_adbkey.pub /root/.android/adbkey.pub
 
+
 # Allow all users to run commands needed by sitespeedio/throttle via sudo
 # See https://github.com/sitespeedio/throttle/blob/main/lib/tc.js
 RUN echo 'ALL ALL=NOPASSWD: /usr/sbin/tc, /usr/sbin/route, /usr/sbin/ip' > /etc/sudoers.d/tc

package/Dockerfile-slim

@@ -30,7 +30,7 @@ RUN buildDeps='wget bzip2' && apt-get update && apt -y install $buildDeps && \
 RUN mkdir -p /usr/src/app
 WORKDIR /usr/src/app
 COPY . /usr/src/app
-RUN CHROMEDRIVER_SKIP_DOWNLOAD=true EGDEDRIVER_SKIP_DOWNLOAD=true npm install --production
+RUN CHROMEDRIVER_SKIP_DOWNLOAD=true EGDEDRIVER_SKIP_DOWNLOAD=true npm install --production && npm cache clean --force && npm uninstall npm -g
 WORKDIR /usr/src/app
 COPY docker/scripts/start-slim.sh /start.sh
 

package/bin/sitespeed.js

@@ -7,6 +7,8 @@
 const fs = require('fs');
 const cli = require('../lib/cli/cli');
 const sitespeed = require('../lib/sitespeed');
+const { execSync } = require('child_process');
+const os = require('os');
 
 async function run(options) {
   process.exitCode = 1;
@@ -20,6 +22,10 @@ async function run(options) {
       fs.writeFileSync('result.json', JSON.stringify(result));
     }
 
+    if ((options.open || options.o) && os.platform() === 'darwin') {
+      execSync('open ' + result.localPath + '/index.html');
+    }
+
     if (
       parsed.options.budget &&
       Object.keys(result.budgetResult.failing).length > 0

package/lib/cli/cli.js

@@ -1455,6 +1455,10 @@ module.exports.parseCommandLine = function parseCommandLine() {
     .option('name', {
       describe: 'Give your test a name.'
     })
+    .option('open', {
+      alias: 'o',
+      describe: 'Open your test result in your default browser (Mac OS only).'
+    })
     .option('slug', {
       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.'

package/lib/sitespeed.js

@@ -156,12 +156,14 @@ module.exports = {
       if (options.summary && options.summary.out) {
         console.log(options.summary.out); // eslint-disable-line no-console
       }
+
       return {
         errors,
         budgetResult,
         resultUrl: resultUrls.hasBaseUrl()
           ? resultUrls.reportSummaryUrl() + '/index.html'
-          : ''
+          : '',
+        localPath: storageManager.getBaseDir()
       };
     } catch (err) {
       log.error(err);