grover 1.1.6 → 1.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a1de71127ef3e7173bce45deeec104ec9c876dea335ad73dfc80016da64c64f7
-  data.tar.gz: 50dce7d9aa7ef01e12144f303868667e4d6587d0dc9a7be337c34d9679e483c0
+  metadata.gz: e791010e46d99d4347edc987fc2a7b90b08d038a1449cf927b57b828e58f1d59
+  data.tar.gz: 62837cb867ec1728c000c91b33352f01a1c4beb5b0823647dff74946d09c0cbd
 SHA512:
-  metadata.gz: '084b8e564818202eead00cb7faef9c181e5bd394e41113fb4b87233fa9fe44ed434318240034a375f20799af909a647fc8dd816f9d74aa79ad101efaa329387f'
-  data.tar.gz: 5f03611f7f945dac5b75424bc879f90e639f4a97430759028f3cf36727c4c490813dc15fc68271767b1996716278484eae0d7383a42741fbd24f2d572124d4e2
+  metadata.gz: 1471dda6f9817f921ae1f7ef88cbdf0ffab143448b1057ef1e3f71f5d91a9fce95b77fbece05f10c0823b51fbc181a4138ae5a6c6db437074c2d95143e90fa4d
+  data.tar.gz: fad1c63de120fb414aa23ffcf0b2adec6e419550ec2add1c5008ce3e1d855343934bbae4de1e9a14f8c74e974699ad7a74aad2a22bc2d6ef7ba1cd8850a73b36

data/lib/grover/js/processor.cjs

@@ -1,13 +1,25 @@
 // Setup imports
 try {
   const Module = require('module');
-  // resolve puppeteer from the CWD instead of where this script is located
-  var puppeteer = require(require.resolve('puppeteer', { paths: Module._nodeModulePaths(process.cwd()) }));
+
+  try {
+    // resolve puppeteer from the CWD instead of where this script is located
+    var puppeteer = require(require.resolve('puppeteer', { paths: Module._nodeModulePaths(process.cwd()) }));
+  } catch (puppeteerError) {
+    try {
+      // try resolve `puppeteer-core` library instead
+      var puppeteer = require(require.resolve('puppeteer-core', { paths: Module._nodeModulePaths(process.cwd()) }));
+    } catch (coreError) {
+      // raise the original puppeteer load issue so we don't send people don't the wrong rabbit hole by default.
+      throw puppeteerError;
+    }
+  }
 } catch (e) {
   process.stdout.write(JSON.stringify(['err', e.toString()]));
   process.stdout.write("\n");
   process.exit(1);
 }
+
 process.stdout.write("[\"ok\"]\n");
 
 const fs = require('fs');
@@ -15,7 +27,7 @@ const os = require('os');
 const path = require('path');
 
 const _processPage = (async (convertAction, urlOrHtml, options) => {
-  let browser, errors = [], tmpDir;
+  let browser, page, errors = [], tmpDir, wsConnection = false;
 
   try {
     // Configure puppeteer debugging options
@@ -25,8 +37,18 @@ const _processPage = (async (convertAction, urlOrHtml, options) => {
       const connectParams = {
         browserWSEndpoint: browserWsEndpoint,
       };
-
-      browser = await puppeteer.connect(connectParams);
+      wsConnection = true;
+
+      try {
+        browser = await puppeteer.connect(connectParams);
+      } catch {
+        function WsConnectFailedError() {
+          this.name = "WsConnectFailedError";
+          this.message = `Failed to connect to browser WS endpoint: ${browserWsEndpoint}`;
+        }
+        WsConnectFailedError.prototype = Error.prototype;
+        throw new WsConnectFailedError();
+      }
     } else {
       tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'grover-'));
 
@@ -34,29 +56,29 @@ const _processPage = (async (convertAction, urlOrHtml, options) => {
         args: process.env.GROVER_NO_SANDBOX === 'true' ? ['--no-sandbox', '--disable-setuid-sandbox'] : [],
         userDataDir: tmpDir
       };
-  
+
       if (typeof debug === 'object' && !!debug) {
         if (debug.headless !== undefined) { launchParams.headless = debug.headless; }
         if (debug.devtools !== undefined) { launchParams.devtools = debug.devtools; }
       }
-  
+
       // Configure additional launch arguments
       const args = options.launchArgs; delete options.launchArgs;
       if (Array.isArray(args)) {
         launchParams.args = launchParams.args.concat(args);
       }
-  
+
       // Set executable path if given
       const executablePath = options.executablePath; delete options.executablePath;
       if (executablePath) {
         launchParams.executablePath = executablePath;
       }
-  
+
       // Launch the browser and create a page
       browser = await puppeteer.launch(launchParams);
     }
 
-    const page = await browser.newPage();
+    page = await browser.newPage();
 
     // Basic auth
     const username = options.username; delete options.username
@@ -261,7 +283,12 @@ const _processPage = (async (convertAction, urlOrHtml, options) => {
     }
   } finally {
     if (browser) {
-      await browser.close();
+      if (wsConnection) {
+        if (page) await page.close();
+        await browser.disconnect();
+      } else {
+        await browser.close();
+      }
     }
 
     try {

data/lib/grover/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class Grover
-  VERSION = '1.1.6'
+  VERSION = '1.1.7'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grover
 version: !ruby/object:Gem::Version
-  version: 1.1.6
+  version: 1.1.7
 platform: ruby
 authors:
 - Andrew Bromwich
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-15 00:00:00.000000000 Z
+date: 2024-03-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: combine_pdf