browserless 10.10.0 → 10.10.1

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.0",
+  "version": "10.10.1",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -34,10 +34,10 @@
     "javascript"
   ],
   "dependencies": {
-    "@browserless/errors": "^10.9.18",
-    "@browserless/goto": "^10.10.0",
-    "@browserless/pdf": "^10.10.0",
-    "@browserless/screenshot": "^10.10.0",
+    "@browserless/errors": "^10.10.1",
+    "@browserless/goto": "^10.10.1",
+    "@browserless/pdf": "^10.10.1",
+    "@browserless/screenshot": "^10.10.1",
     "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": "9b80e677418f2defb804d39043680fe65e3e277b"
+  "gitHead": "32f6e72bcb489a83ac9659520a3961aeb97c47b7"
 }

package/src/driver.js

@@ -71,16 +71,27 @@ const getPid = subprocess => {
 
 const close = async (subprocess, { signal = 'SIGKILL', ...debugOpts } = {}) => {
   const pid = getPid(subprocess)
-  if (pid === undefined) return
+  const hasDisconnect = subprocess && typeof subprocess.disconnect === 'function'
+  const hasClose = subprocess && typeof subprocess.close === 'function'
+
+  if (pid === undefined && !hasDisconnect && !hasClose) return
 
   // It's necessary to call `browser.close` for removing temporal files associated
   // and remove listeners attached to the main process; check
   // - https://github.com/puppeteer/puppeteer/blob/778ac92469d66c542c3c12fe0aa23703dd6315c2/src/node/BrowserRunner.ts#L146
   // - https://github.com/puppeteer/puppeteer/blob/69d85e874416d62de6e821bef30e5cebcfd42f15/src/node/BrowserRunner.ts#L189
-  await pReflect('close' in subprocess ? subprocess.close() : killProcessGroup(subprocess, signal))
+  await pReflect(
+    pid === undefined
+      ? hasDisconnect
+        ? subprocess.disconnect()
+        : subprocess.close()
+      : hasClose
+        ? subprocess.close()
+        : killProcessGroup(subprocess, signal)
+  )
 
   debug('close', { pid, signal, ...debugOpts })
-  return { pid }
+  return pid === undefined ? {} : { pid }
 }
 
 module.exports = { spawn, pid: getPid, close, defaultArgs }

package/src/index.js

@@ -14,9 +14,8 @@ const { AbortError } = pRetry
 
 const driver = require('./driver')
 
-const lock = withLock()
-
 module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
+  const lock = withLock()
   const goto = createGoto({ timeout: globalTimeout, ...launchOpts })
   const { defaultViewport } = goto
 
@@ -123,7 +122,12 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
 
           try {
             page = await createPage(name)
-            setTimeout(() => closePage(page, name), timeout).unref()
+            setTimeout(() => {
+              closePage(page, name).catch(error => {
+                const { message, code, name } = ensureError(error)
+                debug('closePage:timeout:error', { message, code, name })
+              })
+            }, timeout).unref()
             const value = await fn(page, goto)(...args)
             await closePage(page, `${name}:success`)
             return value