browserless 10.10.1 → 10.10.2

package/package.json

@@ -2,7 +2,7 @@
   "name": "browserless",
   "description": "The headless Chrome/Chromium driver on top of Puppeteer. Take screenshots, generate PDFs, extract text and HTML with a production-ready API.",
   "homepage": "https://browserless.js.org",
-  "version": "10.10.1",
+  "version": "10.10.2",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -35,9 +35,9 @@
   ],
   "dependencies": {
     "@browserless/errors": "^10.10.1",
-    "@browserless/goto": "^10.10.1",
-    "@browserless/pdf": "^10.10.1",
-    "@browserless/screenshot": "^10.10.1",
+    "@browserless/goto": "^10.10.2",
+    "@browserless/pdf": "^10.10.2",
+    "@browserless/screenshot": "^10.10.2",
     "debug-logfmt": "~1.4.7",
     "kill-process-group": "~1.0.13",
     "p-reflect": "~2.1.0",
@@ -67,5 +67,5 @@
     "timeout": "2m",
     "workerThreads": false
   },
-  "gitHead": "32f6e72bcb489a83ac9659520a3961aeb97c47b7"
+  "gitHead": "ec6a614923a1a692bd717ecc8e6f1b09417801d9"
 }

package/src/index.js

@@ -74,22 +74,22 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
   const createContext = async ({ retry = 2, timeout: contextTimeout, ...contextOpts } = {}) => {
     let _contextPromise = createBrowserContext(contextOpts)
     let isDestroyedForced = false
+    const pageMetadata = new WeakMap()
 
     const getBrowserContext = () => _contextPromise
 
     const createPage = async name => {
       const duration = debug.duration('createPage')
-      const [browserProcess, browserContext] = await Promise.all([
-        getBrowser(),
-        getBrowserContext()
-      ])
+      const browserContext = await getBrowserContext()
       const page = await browserContext.newPage()
-      duration({
-        name,
+      const browser = typeof page.browser === 'function' ? page.browser() : undefined
+      const metadata = {
         id: page._client().id(),
         contextId: browserContext.id,
-        browserPid: driver.pid(browserProcess)
-      })
+        browserPid: browser ? driver.pid(browser) : undefined
+      }
+      pageMetadata.set(page, metadata)
+      duration({ name, ...metadata })
       return page
     }
 
@@ -97,17 +97,9 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
       if (page && !page.isClosed()) {
         const duration = debug.duration('closePage')
         if (page.disableAdblock) page.disableAdblock()
-        const [browserProcess, browserContext] = await Promise.all([
-          getBrowser(),
-          getBrowserContext(),
-          pReflect(page.close())
-        ])
-        duration({
-          name,
-          id: page._client().id(),
-          contextId: browserContext.id,
-          browserPid: driver.pid(browserProcess)
-        })
+        await pReflect(page.close())
+        duration({ name, ...(pageMetadata.get(page) || {}) })
+        pageMetadata.delete(page)
       }
     }
 
@@ -119,21 +111,25 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
 
         async function run () {
           let page
+          let closePageTimeout
 
           try {
             page = await createPage(name)
-            setTimeout(() => {
+            closePageTimeout = setTimeout(() => {
               closePage(page, name).catch(error => {
                 const { message, code, name } = ensureError(error)
                 debug('closePage:timeout:error', { message, code, name })
               })
-            }, timeout).unref()
+            }, timeout)
+            if (typeof closePageTimeout.unref === 'function') closePageTimeout.unref()
             const value = await fn(page, goto)(...args)
             await closePage(page, `${name}:success`)
             return value
           } catch (error) {
             await closePage(page, `${name}:error`)
             if (!isRejected) throw ensureError(error)
+          } finally {
+            if (closePageTimeout) clearTimeout(closePageTimeout)
           }
         }
 
@@ -144,9 +140,9 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
               debug('onFailedAttempt', { name: error.name, code: error.code, isRejected })
               if (error.name === 'AbortError') throw error
               if (isRejected || isDestroyedForced) throw new AbortError()
-              if (error.code === 'EBRWSRCONTEXTCONNRESET') {
-                _contextPromise = createBrowserContext(contextOpts)
-              }
+              const isRetryable = error.code === 'EBRWSRCONTEXTCONNRESET'
+              if (!isRetryable) throw error
+              _contextPromise = createBrowserContext(contextOpts)
               const { message, attemptNumber, retriesLeft } = error
               debug('retry', { attemptNumber, retriesLeft, message })
             }