grover 1.1.10 → 1.1.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 89de769dbb4ef96309c640d664ac90d9771968c02f596b6e6370d2f77b10cd1e
-  data.tar.gz: 5df2d5e82e1155757d859c95e01b0ed24dd49fdae6d4d8cb5c23276ef774603d
+  metadata.gz: 07f9183a814d7d598e29ee39cc4d7af3df619ba262aa1bef5be2de0cd9e7da3c
+  data.tar.gz: 368d618c68428ff2f02832962463532a305ca48a2d6b8956dd978a80dc375052
 SHA512:
-  metadata.gz: 9a6b7be08fb0d719cd464033d1d9ebf63343f8cf7126ea4dfead19eb2b022d9bf124898b58b6842690714f7d7d05fc01e847340a9ed47f9966b6c8eddfb768fe
-  data.tar.gz: 90acf98c742ad66686108735828b0a378776c07375562df8c4db6f483a9f1669c7a0409381764caddb9a788c6ddbe9cfc7802edfa05961bedeb4c7896da4558d
+  metadata.gz: 0deb217329f6141c3a7ed3da596945aa79213ca2216dbc914bb6b1e17104283bff1f8f45057f4fa099689fd8295560f9e0e2d023fa0e93551599a447649dace2
+  data.tar.gz: 6e9b91209b59e8bc91ef38aa42906468c2b79614950f6d9e345d002468248519a633b69cbaeb4e6ad6e520e5f8acafa2356e084c1b167ac7717c344a287bcd72

data/lib/grover/errors.rb

@@ -11,6 +11,18 @@ class Grover
   module JavaScript # rubocop:disable Style/Documentation
     Error = Class.new(::Grover::Error)
     UnknownError = Class.new(Error)
+
+    ErrorWithDetails = Class.new(Error) do
+      def initialize(name, error_details)
+        super(name)
+        @error_details = Grover::Utils.deep_transform_keys_in_object error_details, &:to_sym
+      end
+
+      attr_reader :error_details
+    end
+    RequestFailedError = Class.new(ErrorWithDetails)
+    PageRenderError = Class.new(ErrorWithDetails)
+
     def self.const_missing(name)
       const_set name, Class.new(Error)
     end

data/lib/grover/js/processor.cjs

@@ -26,8 +26,32 @@ const fs = require('fs');
 const os = require('os');
 const path = require('path');
 
+function GroverError(name, errors) {
+  this.name = name;
+  this.message = errors.map(e => e.message).join("\n");
+  this.errors = errors;
+}
+GroverError.prototype = Error.prototype;
+
 const _processPage = (async (convertAction, uriOrHtml, options) => {
-  let browser, page, errors = [], tmpDir, wsConnection = false;
+  let browser, page, tmpDir, wsConnection = false;
+  const requestErrors = [], pageErrors = [];
+
+  const captureRequestError = (request) => {
+    const requestError = { url: request.url() };
+
+    if (request.failure()) {
+      requestError.reason = request.failure().errorText;
+      requestError.message = requestError.reason + " at " + requestError.url;
+    } else if (request.response() && request.response().status()) {
+      requestError.status = request.response().status();
+      requestError.message = requestError.status + " " + requestError.url;
+    } else {
+      requestError.message = "UnknownError " + requestError.url;
+    }
+
+    requestErrors.push(requestError);
+  };
 
   try {
     // Configure puppeteer debugging options
@@ -163,12 +187,24 @@ const _processPage = (async (convertAction, uriOrHtml, options) => {
     const raiseOnRequestFailure = options.raiseOnRequestFailure; delete options.raiseOnRequestFailure;
     if (raiseOnRequestFailure) {
       page.on('requestfinished', (request) => {
-        if (request.response() && !(request.response().ok() || request.response().status() === 304) && !request.redirectChain().length > 0) {
-          errors.push(request);
+        if (request.response() &&
+            !(request.response().ok() || request.response().status() === 304) &&
+            !request.redirectChain().length > 0) {
+          captureRequestError(request);
         }
       });
       page.on('requestfailed', (request) => {
-        errors.push(request);
+        captureRequestError(request);
+      });
+    }
+
+    const raiseOnJSError = options.raiseOnJSError; delete options.raiseOnJSError;
+    if (raiseOnJSError) {
+      page.on('pageerror', (error) => {
+        pageErrors.push({
+          message: error.toString().replace(new RegExp('^' + error.name + ': '), ''),
+          type: error.name || 'Error'
+        });
       });
     }
 
@@ -257,21 +293,12 @@ const _processPage = (async (convertAction, uriOrHtml, options) => {
       await page.hover(hoverSelector);
     }
 
-    if (errors.length > 0) {
-      function RequestFailedError(errors) {
-        this.name = "RequestFailedError";
-        this.message = errors.map(e => {
-          if (e.failure()) {
-            return e.failure().errorText + " at " + e.url();
-          } else if (e.response() && e.response().status()) {
-            return e.response().status() + " " + e.url();
-          } else {
-            return "UnknownError " + e.url()
-          }
-        }).join("\n");
-      }
-      RequestFailedError.prototype = Error.prototype;
-      throw new RequestFailedError(errors);
+    if (requestErrors.length > 0) {
+      throw new GroverError("RequestFailedError", requestErrors);
+    }
+
+    if (pageErrors.length > 0) {
+      throw new GroverError("PageRenderError", pageErrors);
     }
 
     // Setup conversion timeout
@@ -301,7 +328,9 @@ const _processPage = (async (convertAction, uriOrHtml, options) => {
 });
 
 function _handleError(error) {
-  if (error instanceof Error) {
+  if (error instanceof GroverError) {
+    process.stdout.write(JSON.stringify(['err', error.message, error.name, error.errors]));
+  } else if (error instanceof Error) {
     process.stdout.write(
       JSON.stringify(['err', error.toString().replace(new RegExp('^' + error.name + ': '), ''), error.name])
     );

data/lib/grover/options_fixer.rb

@@ -34,7 +34,8 @@ class Grover
     def fix_boolean_options!
       fix_options!(
         'display_header_footer', 'full_page', 'landscape', 'omit_background', 'prefer_css_page_size',
-        'print_background', 'viewport.has_touch', 'viewport.is_landscape', 'viewport.is_mobile', 'bypass_csp'
+        'print_background', 'viewport.has_touch', 'viewport.is_landscape', 'viewport.is_mobile', 'bypass_csp',
+        'raise_on_request_failure', 'raise_on_js_error'
       ) { |value| !FALSE_VALUES.include?(value) }
     end
 

data/lib/grover/processor.rb

@@ -83,14 +83,14 @@ class Grover
       input = stdout.gets
       raise Errno::EPIPE, "Can't read from worker" if input.nil?
 
-      status, message, error_class = JSON.parse(input)
+      status, message, error_class, errors = JSON.parse(input)
 
       if status == 'ok'
         message
       elsif error_class.nil?
         raise Grover::JavaScript::UnknownError, message
       else
-        raise Grover::JavaScript.const_get(error_class, false), message
+        raise Grover::JavaScript.const_get(error_class, false).new(*[message, errors].compact)
       end
     rescue JSON::ParserError
       raise Grover::Error, 'Malformed worker response'

data/lib/grover/utils.rb

@@ -8,7 +8,8 @@ class Grover
     ACRONYMS = {
       'css' => 'CSS',
       'csp' => 'CSP',
-      'http' => 'HTTP'
+      'http' => 'HTTP',
+      'js' => 'JS'
     }.freeze
     private_constant :ACRONYMS
 

data/lib/grover/version.rb

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

metadata

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