browserless 9.12.2-0 → 9.12.3

package/package.json

@@ -2,7 +2,7 @@
   "name": "browserless",
   "description": "The headless Chrome/Chromium performance driver for Node.js",
   "homepage": "https://browserless.js.org",
-  "version": "9.12.2-0",
+  "version": "9.12.3",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -44,9 +44,9 @@
     "superlock": "~1.0.1"
   },
   "devDependencies": {
-    "ps-list": "7",
     "@browserless/test": "^9.11.0",
-    "ava": "latest"
+    "ava": "latest",
+    "ps-list": "7"
   },
   "engines": {
     "node": ">= 12"
@@ -54,13 +54,14 @@
   "files": [
     "src"
   ],
+  "scripts": {
+    "test": "ava"
+  },
   "license": "MIT",
   "ava": {
     "serial": true,
     "timeout": "30s",
     "workerThreads": false
   },
-  "scripts": {
-    "test": "ava"
-  }
+  "gitHead": "94c06dacf906c3573fb6a6980bd5644140212663"
 }

package/src/index.js

@@ -24,9 +24,7 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
 
   const close = opts => {
     isClosed = true
-    return browserProcessPromise
-      .then(browserProcess => driver.close(browserProcess, opts))
-      .catch(() => {})
+    return browserProcessPromise.then(browserProcess => driver.close(browserProcess, opts))
   }
 
   const respawn = () =>
@@ -109,46 +107,48 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
       }
     }
 
-    const withPage = (fn, { timeout: evaluateTimeout } = {}) => async (...args) => {
-      let isRejected = false
-
-      async function run () {
-        let page
-
-        try {
-          page = await createPage(args)
-          setTimeout(() => closePage(page), timeout).unref()
-          const value = await fn(page, goto)(...args)
-          await closePage(page)
-          return value
-        } catch (error) {
-          await closePage(page)
-          if (!isRejected) throw ensureError(error)
-        }
-      }
-
-      const task = () =>
-        pRetry(run, {
-          retries: retry,
-          onFailedAttempt: async error => {
-            debug('onFailedAttempt', { name: error.name, code: error.code, isRejected })
-            if (error.name === 'AbortError') throw error
-            if (isRejected) throw new AbortError()
-            if (error.code === 'EBRWSRCONTEXTCONNRESET') {
-              _contextPromise = createBrowserContext(contextOpts)
+    const withPage =
+      (fn, { timeout: evaluateTimeout } = {}) =>
+        async (...args) => {
+          let isRejected = false
+
+          async function run () {
+            let page
+
+            try {
+              page = await createPage(args)
+              setTimeout(() => closePage(page), timeout).unref()
+              const value = await fn(page, goto)(...args)
+              await closePage(page)
+              return value
+            } catch (error) {
+              await closePage(page)
+              if (!isRejected) throw ensureError(error)
             }
-            const { message, attemptNumber, retriesLeft } = error
-            debug('retry', { attemptNumber, retriesLeft, message })
           }
-        })
-
-      const timeout = evaluateTimeout || contextTimeout || globalTimeout
 
-      return pTimeout(task(), timeout, () => {
-        isRejected = true
-        throw browserTimeout({ timeout })
-      })
-    }
+          const task = () =>
+            pRetry(run, {
+              retries: retry,
+              onFailedAttempt: async error => {
+                debug('onFailedAttempt', { name: error.name, code: error.code, isRejected })
+                if (error.name === 'AbortError') throw error
+                if (isRejected) throw new AbortError()
+                if (error.code === 'EBRWSRCONTEXTCONNRESET') {
+                  _contextPromise = createBrowserContext(contextOpts)
+                }
+                const { message, attemptNumber, retriesLeft } = error
+                debug('retry', { attemptNumber, retriesLeft, message })
+              }
+            })
+
+          const timeout = evaluateTimeout || contextTimeout || globalTimeout
+
+          return pTimeout(task(), timeout, () => {
+            isRejected = true
+            throw browserTimeout({ timeout })
+          })
+        }
 
     const evaluate = (fn, gotoOpts) =>
       withPage(