browsertime 16.13.2 → 16.13.3

package/CHANGELOG.md

@@ -1,6 +1,11 @@
 # Browsertime changelog (we do [semantic versioning](https://semver.org))
 
 
+## 16.13.3 - 2022-08-17
+
+### Fixed
+* Make sure set cookies work when navigating in script for Chrome/Edge [#1830](https://github.com/sitespeedio/browsertime/pull/1830).
+
 ## 16.13.2 - 2022-08-14
 ### Fixed
 * Disable adding lastCPULongTask to the HAR data since it breaks waterfall view on slow mobile phones with late CPU long tasks [#1828](https://github.com/sitespeedio/browsertime/pull/1828).

package/lib/chrome/webdriver/chromium.js

@@ -444,6 +444,10 @@ class Chromium {
       return { har: harBuilder.mergeHars(this.hars) };
     } else return {};
   }
+
+  async setCookies(url, cookies) {
+    return this.cdpClient.setCookies(url, cookies);
+  }
 }
 
 module.exports = Chromium;

package/lib/core/engine/command/measure.js

@@ -141,6 +141,15 @@ class Measure {
     log.info('Navigating to url %s iteration %s', url, this.index);
     if (this.numberOfVisitedPages === 0) {
       await this.extensionServer.setup(url, this.browser, this.options);
+
+      // There's a bug that we introduced when moving cookie handling to CDP
+      // and this is the hack to fix that.
+      if (
+        (this.options.cookie && this.options.browser === 'chrome') ||
+        this.options.browser === 'edge'
+      ) {
+        await this.engineDelegate.setCookies(url, this.options.cookie);
+      }
     }
     if (this.numberOfVisitedPages === 0) {
       await this.engineDelegate.beforeStartIteration(this.browser, this.index);

package/package.json

@@ -1,6 +1,6 @@
 {
   "description": "Browsertime",
-  "version": "16.13.2",
+  "version": "16.13.3",
   "bin": "./bin/browsertime.js",
   "dependencies": {
     "@cypress/xvfb": "1.2.4",