browserless 14.1.2 → 14.1.4

package/index.d.ts

@@ -40,7 +40,10 @@ export interface Context {
   text: (url: string, opts?: GotoOptions) => Promise<string>
   getDevice: (deviceName: string) => Viewport | undefined
   report: (opts?: { benchmark?: boolean }) => Promise<HardwareInfo>
-  destroyContext: (opts?: { force?: boolean }) => Promise<void>
+  destroyContext: (opts?: {
+    /** @deprecated A destroyed context never retries, so `force` has no effect. */
+    force?: boolean
+  }) => Promise<void>
   withPage: <T>(fn: (page: Page, goto: unknown) => Promise<T>, opts?: { timeout?: number, keepPage?: boolean }) => Promise<T>
 }
 

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": "14.1.2",
+  "version": "14.1.4",
   "types": "index.d.ts",
   "main": "src/index.js",
   "author": {
@@ -36,9 +36,9 @@
   ],
   "dependencies": {
     "@browserless/errors": "14.0.22",
-    "@browserless/goto": "14.1.2",
-    "@browserless/pdf": "14.1.2",
-    "@browserless/screenshot": "14.1.2",
+    "@browserless/goto": "14.1.4",
+    "@browserless/pdf": "14.1.4",
+    "@browserless/screenshot": "14.1.4",
     "debug-logfmt": "~1.4.15",
     "kill-process-group": "~1.0.16",
     "p-reflect": "~2.1.0",
@@ -70,5 +70,5 @@
     "timeout": "2m",
     "workerThreads": false
   },
-  "gitHead": "700bbabadea0a16facf30d76ecf60ff9602e06f8"
+  "gitHead": "136b2d9f50772f096aa81af66780c1b625ec2fc1"
 }

package/src/index.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const { ensureError, browserTimeout } = require('@browserless/errors')
+const { ensureError, browserTimeout, contextDisconnected } = require('@browserless/errors')
 const createScreenshot = require('@browserless/screenshot')
 const debug = require('debug-logfmt')('browserless')
 const createGoto = require('@browserless/goto')
@@ -111,18 +111,23 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
   }
 
   const createContext = async ({ retry = 2, timeout: contextTimeout, ...contextOpts } = {}) => {
-    const forwardedContextOpts = isClosed
-      ? Promise.resolve(contextOpts)
-      : forwardProxy(contextOpts, { mode: launchOpts.mode })
-    forwarders.add(forwardedContextOpts)
-    const newBrowserContext = () => forwardedContextOpts.then(createBrowserContext)
-    const usesForwarder = () => forwardedContextOpts.then(opts => opts !== contextOpts)
+    let forwardedContextOpts
+    const forwardContextOpts = () => {
+      forwardedContextOpts = isClosed
+        ? Promise.resolve(contextOpts)
+        : forwardProxy(contextOpts, { mode: launchOpts.mode })
+      forwarders.add(forwardedContextOpts)
+      return forwardedContextOpts
+    }
+    const newBrowserContext = () =>
+      (forwardedContextOpts ?? forwardContextOpts()).then(createBrowserContext)
     let _contextPromise = newBrowserContext()
-    let isDestroyedForced = false
     let isDestroyed = false
     const pageMetadata = new WeakMap()
+    const isAbandoned = () => isClosed || isDestroyed
 
-    const getBrowserContext = () => _contextPromise
+    const getBrowserContext = () =>
+      isDestroyed ? Promise.reject(contextDisconnected()) : _contextPromise
 
     const createPage = async name => {
       const duration = debug.duration('createPage')
@@ -172,9 +177,11 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
           }
 
           try {
+            if (isAbandoned()) throw contextDisconnected()
             page = await createPage(name)
             closePageTimeout = startCloseTimeout()
             const value = await fn(page, goto)(...args)
+            if (isAbandoned()) throw contextDisconnected()
             // A timeout already rejected the caller, who never received this
             // page. Retaining it would clear the in-call watchdog and arm a
             // new one with nobody holding the page.
@@ -200,11 +207,11 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
             onFailedAttempt: async error => {
               debug('onFailedAttempt', { name: error.name, code: error.code, isRejected })
               if (error.name === 'AbortError') throw error
-              if (isRejected || isDestroyedForced) throw new AbortError()
+              if (isRejected) throw new AbortError()
               const isRetryable =
                 error.code === 'EBRWSRCONTEXTCONNRESET' || error.code === 'EPROTOCOL'
               if (!isRetryable) throw error
-              if (isDestroyed && (await usesForwarder())) throw new AbortError()
+              if (isAbandoned()) throw new AbortError()
               const { message, attemptNumber, retriesLeft } = error
               // Every attempt builds its own page, so both codes are retryable
               // inside the existing context: `EBRWSRCONTEXTCONNRESET` covers a
@@ -247,20 +254,24 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
       )
     }
 
-    const destroyContext = async ({ force = false } = {}) => {
+    const destroyContext = async () => {
       isDestroyed = true
-      if (force) isDestroyedForced = true
+      const closingContextPromise = _contextPromise
+      const closingContextOpts = forwardedContextOpts
+      _contextPromise = undefined
+      forwardedContextOpts = undefined
+      if (!closingContextPromise) return
 
       try {
         const [browserProcess, browserContext] = await Promise.all([
           getBrowser(),
-          getBrowserContext()
+          closingContextPromise
         ])
         await pReflect(browserContext.close())
-        debug('destroyContext', { pid: driver.pid(browserProcess), force, id: browserContext.id })
+        debug('destroyContext', { pid: driver.pid(browserProcess), id: browserContext.id })
       } finally {
-        forwarders.delete(forwardedContextOpts)
-        await closeForwarder(forwardedContextOpts)
+        forwarders.delete(closingContextOpts)
+        await closeForwarder(closingContextOpts)
       }
     }