browserless 9.3.18 → 9.3.21

package/README.md

@@ -18,9 +18,9 @@ npm install browserless puppeteer --save
 
 ## License
 
-**browserless** © [Microlink](https://microlink.io), Released under the [MIT](https://github.com/microlinkhq/browserless/blob/master/LICENSE.md) License.<br>
+**browserless** © [Microlink](https://microlink.io), released under the [MIT](https://github.com/microlinkhq/browserless/blob/master/LICENSE.md) License.<br>
 Authored and maintained by [Microlink](https://microlink.io) with help from [contributors](https://github.com/microlinkhq/browserless/contributors).
 
 The [logo](https://thenounproject.com/term/browser/288309/) has been designed by [xinh studio](https://xinh.studio).
 
-> [microlink.io](https://microlink.io) · GitHub [@MicrolinkHQ](https://github.com/microlinkhq) · Twitter [@microlinkhq](https://twitter.com/microlinkhq)
+> [microlink.io](https://microlink.io) · GitHub [microlinkhq](https://github.com/microlinkhq) · Twitter [@microlinkhq](https://twitter.com/microlinkhq)

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.3.18",
+  "version": "9.3.21",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -31,11 +31,12 @@
     "text"
   ],
   "dependencies": {
-    "@browserless/errors": "^9.3.17",
-    "@browserless/goto": "^9.3.17",
-    "@browserless/pdf": "^9.3.17",
-    "@browserless/screenshot": "^9.3.17",
+    "@browserless/errors": "^9.3.21",
+    "@browserless/goto": "^9.3.21",
+    "@browserless/pdf": "^9.3.21",
+    "@browserless/screenshot": "^9.3.21",
     "debug-logfmt": "~1.0.4",
+    "kill-process-group": "~1.0.1",
     "mutexify": "~1.4.0",
     "p-reflect": "~2.1.0",
     "p-retry": "~4.6.1",
@@ -60,5 +61,5 @@
     "timeout": "2m",
     "verbose": true
   },
-  "gitHead": "f17ccdab013584350a3678908f013e596515029b"
+  "gitHead": "1cad664def78a0bc695a98bb5e1786fb41ee90f0"
 }

package/src/driver.js

@@ -1,11 +1,9 @@
 'use strict'
 
+const killProcessGroup = require('kill-process-group')
 const debug = require('debug-logfmt')('browserless')
 const requireOneOf = require('require-one-of')
 const pReflect = require('p-reflect')
-const { promisify } = require('util')
-
-const exec = promisify(require('child_process').exec)
 
 // flags explained: https://peter.sh/experiments/chromium-command-line-switches
 // default flags: https://github.com/puppeteer/puppeteer/blob/edb01972b9606d8b05b979a588eda0d622315981/src/node/Launcher.ts#L183
@@ -51,31 +49,29 @@ const spawn = ({
   ...launchOpts
 } = {}) => puppeteer[mode]({ ignoreHTTPSErrors: true, args, ...launchOpts })
 
-const getPid = childProcess => {
-  if (!childProcess) return null
-  if (childProcess.pid) return childProcess.pid
-  const browserProcess = childProcess.process ? childProcess.process() : undefined
-  if (!browserProcess) return null
-  return browserProcess.pid
+const getProcess = subprocess => {
+  if (!subprocess) return
+  if ('process' in subprocess) return subprocess.process()
+  if ('pid' in subprocess) return subprocess
 }
 
-const killProcesssGroupPID = (pid, signal) =>
-  process.platform === 'win32'
-    ? exec(`taskkill /pid ${this.proc.pid} /T /F`)
-    : Promise.resolve(process.kill(-pid, signal))
+const getPid = input => {
+  const subprocess = getProcess(input)
+  return subprocess ? subprocess.pid : undefined
+}
 
-const close = async (childProcess, { signal = 'SIGKILL', ...debugOpts } = {}) => {
-  const pid = getPid(childProcess)
-  if (!pid) return
+const close = async (input, { signal = 'SIGKILL', ...debugOpts } = {}) => {
+  const subprocess = getProcess(input)
+  if (!subprocess) 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(childProcess.close ? childProcess.close() : killProcesssGroupPID(pid, signal))
+  await pReflect('close' in subprocess ? subprocess.close() : killProcessGroup(subprocess, signal))
 
-  debug('close', { pid, signal, ...debugOpts })
-  return { pid }
+  debug('close', { pid: subprocess.pid, signal, ...debugOpts })
+  return { pid: subprocess.pid }
 }
 
-module.exports = { spawn, getPid, close, defaultArgs }
+module.exports = { spawn, getPid, getProcess, close, defaultArgs }

package/src/index.js

@@ -156,8 +156,8 @@ module.exports = ({ timeout: globalTimeout = 30000, ...launchOpts } = {}) => {
     const evaluate = (fn, gotoOpts) =>
       wrapError(
         page => async (url, opts) => {
-          const { response } = await goto(page, { url, ...gotoOpts, ...opts })
-          return fn(page, response)
+          const { response, error } = await goto(page, { url, ...gotoOpts, ...opts })
+          return fn(page, response, error)
         },
         gotoOpts
       )