bidi2pdf 0.1.7 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 766b41f0ee642cd7316d0f72d8dd707b0f45aae4a315a46c2b27fb6bb2d176a6
-  data.tar.gz: aeec0549f82ff7bdd68d1aa658ea6ad2033e5310fd5936f40b94007b4ae6c38f
+  metadata.gz: 3d7fa3c853f53e21a110cadb25fcd66bf97b890eb00663f90e73e8ee9d1ce07e
+  data.tar.gz: 5900568c47f526e9b00d95a15f8293abeaee88f6c0f323bc6e79cdcb3aba38ef
 SHA512:
-  metadata.gz: cc7f1da58549b642521808b9ea2acc4b04068bdb7c877cf52943d2ae69bb989f2ade02601b8bfd0e409440ad8644206bba1e6e16603eb56099b8963a2136e350
-  data.tar.gz: 6258250ac5de22034cbb7816d3ff33c62680747a3eaee171fdd784d309bf1cd7880ca60e45342da8ee9195814ef1c78ca9b45b2c435f9fcbc4aaa43a8d7f95e6
+  metadata.gz: 9ecffa81a6358c413dd24b1cef48c3b2282518f5934710558b2ea834a362b198d5a1e3f01a2b3eb8ed163dedffaae61fe2bb0c5f3edec5e3850ad375daf10304
+  data.tar.gz: 0b9d4b19f9d2d01a8babfe2a13c4256b9f93d34f2fcdc1ea2942fb780ce4ad02962050f1adffd9d2c5731e2e6c6d1dd83deaa3d31b348157197c982701a8995b

data/CHANGELOG.md

@@ -7,8 +7,65 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+[unreleased]: https://github.com/dieter-medium/bidi2pdf/compare/v0.1.8..HEAD
+
 <!-- generated by git-cliff end -->
 
+## [0.1.9] - 2025-05-04
+
+### 🎨 Refactored
+
+- Enhance reader thread management in Chromedriver by @dieter-medium
+- Improve event listener logging by @dieter-medium
+- Improve event listener management by @dieter-medium
+- Centralize chromedriver test helpers for reuse in other projects by @dieter-medium
+
+### 🐛 Fixed
+
+- Close event socket during session cleanup by @dieter-medium
+- Add test for generating PDFs in parallel by @dieter-medium
+
+### 💄 Style
+
+- Update Bootstrap stylesheet link by @dieter-medium
+
+### 🔧 Build
+
+- Update Ruby version in action.yml by @dieter-medium
+- Update Dockerfile.chromedriver and chromedriver.yml for enhancements by @dieter-medium
+- Update Dockerfile.chromedriver for improved environment setup and vnc support by @dieter-medium
+
+### 🚀 Added
+
+- Enhance testcontainers with shared network support by @dieter-medium
+
+## [0.1.8] - 2025-04-22
+
+### 🎨 Refactored
+
+- Modularize ChromedriverContainer implementation by @dieter-medium
+- Replace method calls for clarity and consistency by @dieter-medium
+- Namespace PDFTextSanitizer under Bidi2pdf::TestHelpers by @dieter-medium
+- Refactor command management with concurrent queues by @dieter-medium
+
+### 🐛 Fixed
+
+- Update CHANGELOG links to correct Markdown syntax by @dieter-medium
+
+### 📝 Docs
+
+- Add Rails integration section to README by @dieter-medium
+
+### 🚀 Added
+
+- Update Chromedriver container setup and default image by @dieter-medium
+- Add workflow for pushing Chromedriver Docker image by @dieter-medium
+- Return session status and add test coverage by @dieter-medium
+- Integrate concurrent-ruby for thread safety improvements by @dieter-medium
+- Add specific navigation error classes for better handling by @dieter-medium
+- Enhance navigation error handling in BrowserTab by @dieter-medium
+- Add test helpers and matchers for PDF validation by @dieter-medium
+
 ## [0.1.7] - 2025-04-17
 
 ### 🎨 Refactored
@@ -143,12 +200,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Initial release
 
-[unreleased]: https://github.com/dieter-medium/bidi2pdf/compare/v0.1.7..HEAD
-
-[unreleased]: https://github.com/dieter-medium/bidi2pdf/compare/v0.1.6..v0.1.7
-
-[0.1.6]: https://github.com/dieter-medium/bidi2pdf/compare/v0.1.5..v0.1.6
-
-[0.1.5]: https://github.com/dieter-medium/bidi2pdf/compare/v0.1.4..v0.1.5
 
-[0.1.4]: https://github.com/dieter-medium/bidi2pdf/compare/v0.1.3..v0.1.4 
+- [unreleased](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.9..HEAD)
+- [0.1.9](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.8..v0.1.9)
+- [0.1.8](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.7..v0.1.8)
+- [0.1.7](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.6..v0.1.7)
+- [0.1.6](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.5..v0.1.6)
+- [0.1.5](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.4..v0.1.5)
+- [0.1.4](https://github.com/dieter-medium/bidi2pdf/compare/v0.1.3..v0.1.4) 

data/README.md

@@ -257,6 +257,20 @@ docker compose -f docker/docker-compose.yml down
 
 ---
 
+## 🚂 Rails Integration
+
+Rails integration is available as an additional gem:
+
+```ruby
+# In your Gemfile
+gem 'bidi2pdf-rails'
+```
+
+For full documentation and usage examples,
+visit: [https://github.com/dieter-medium/bidi2pdf-rails](https://github.com/dieter-medium/bidi2pdf-rails)
+
+---
+
 ## 🛠 Development
 
 ```bash

data/docker/Dockerfile.chromedriver

@@ -1,13 +1,16 @@
-FROM ruby:3.3
+FROM debian:bookworm-slim
 
 ARG CHROMEDRIVER_PORT=3000
 
 ENV DEBIAN_FRONTEND=noninteractive
+ENV LANG=en_US.UTF-8
 
 # Install dependencies
-RUN apt-get update && apt-get upgrade -y && \
+RUN echo "deb http://deb.debian.org/debian bookworm contrib non-free" > /etc/apt/sources.list.d/contrib.list &&\
+    echo "ttf-mscorefonts-installer msttcorefonts/accepted-mscorefonts-eula select true" | debconf-set-selections &&\
+    apt-get update && apt-get upgrade -y && \
     apt-get install -y --no-install-recommends\
-    chromium \
+    chromium chromium-driver chromium-l10n chromium-sandbox\
     libglib2.0-0 \
     libnss3 \
     libxss1 \
@@ -18,23 +21,45 @@ RUN apt-get update && apt-get upgrade -y && \
     curl \
     unzip \
     xvfb \
+    x11vnc \
+    fluxbox \
+    xterm \
+    wmctrl \
+    net-tools xauth \
+    fonts-liberation fonts-dejavu-core  fonts-noto-core fonts-noto-cjk fonts-noto-color-emoji fonts-symbola fontconfig ttf-mscorefonts-installer\
+    libnss3 libatk1.0-0 \
+    libx11-6 libxss1 libgtk-3-0 libgbm1 \
+    locales sed \
+    && sed -i '/en_US.UTF-8/s/^# //g' /etc/locale.gen \
+    && locale-gen en_US.UTF-8 \
     && rm -rf /var/lib/apt/lists/*
 
 # Create a non-root user
-RUN groupadd -r appuser && useradd -r -g appuser -m -d /home/appuser appuser
+RUN groupadd -r appuser && useradd -r -g appuser -G audio,video -m -d /home/appuser appuser
 
 COPY ./docker/entrypoint.sh /usr/local/bin/entrypoint.sh
 RUN chmod +x /usr/local/bin/entrypoint.sh
 
+# ARM compatibility workaround:
+# On ARM architectures (such as Apple Silicon), downloading chromedriver via automated scripts may fail or cause ELF binary errors,
+# such as "rosetta error: failed to open elf at /lib64/ld-linux-x86-64.so.2".
+# To avoid these issues, we directly install 'chromium-driver' via the package manager and explicitly create a symlink in the expected location.
+
+RUN mkdir -p /home/appuser/.webdrivers && ln -s /usr/bin/chromedriver /home/appuser/.webdrivers/chromedriver
+
 # Set working directory
 WORKDIR /app
 
+RUN mkdir -p /tmp/.X11-unix && chmod 1777 /tmp/.X11-unix
+
 # Switch to non-root user
 USER appuser
 
-RUN gem install chromedriver-binary && ruby -e 'require "chromedriver/binary"; puts Chromedriver::Binary::ChromedriverDownloader.update'
+# RUN gem install chromedriver-binary && ruby -e 'require "chromedriver/binary"; puts Chromedriver::Binary::ChromedriverDownloader.update'
 
 ENV CHROMEDRIVER_PORT=${CHROMEDRIVER_PORT}
 EXPOSE ${CHROMEDRIVER_PORT}
+# VNC
+EXPOSE 5900
 
 CMD ["/usr/local/bin/entrypoint.sh"]

data/docker/entrypoint.sh

@@ -3,6 +3,47 @@
 USER_DATA_DIR=/home/appuser/.cache
 mkdir -p ${USER_DATA_DIR}
 
+if [ "$ENABLE_XVFB" = "true" ]; then
+  rm -rf /tmp/.X99-lock
+
+  export DISPLAY=:99
+  Xvfb :99 -screen 0 1920x1080x24 &
+
+  old_umask=$(umask)
+  umask 077
+
+  touch /home/appuser/.Xauthority
+  export XAUTHORITY=/home/appuser/.Xauthority
+
+  xauth generate :99 . trusted
+
+  umask $old_umask
+
+
+
+  until xdpyinfo -display ${DISPLAY} >/dev/null 2>&1; do
+      sleep 0.2
+  done
+
+  fluxbox &
+
+  until wmctrl -m > /dev/null 2>&1; do
+    sleep 0.2
+  done
+fi
+
+if [ "$ENABLE_VNC" = "true" ]; then
+  VNC_PASS=${VNC_PASS:-$(tr -dc A-Za-z0-9 </dev/urandom | head -c 12)}
+  echo "VNC password: $VNC_PASS"
+  old_umask=$(umask)
+  umask 077
+  mkdir -p /home/appuser/.vnc
+  x11vnc -storepasswd $VNC_PASS /home/appuser/.vnc/passwd
+  umask $old_umask
+  x11vnc -display WAIT:99 -xkb -noxrecord -noxfixes -noxdamage -forever -shared -noshm -usepw -rfbauth /home/appuser/.vnc/passwd &
+fi
+
+# DISPLAY=:99 /home/appuser/.webdrivers/chromedriver --port=33259 --whitelisted-ips=""  --allowed-origins="*" --disable-dev-shm-usage --disable-gpu  --verbose
 /home/appuser/.webdrivers/chromedriver --port=${CHROMEDRIVER_PORT} \
                                        --headless \
                                        --whitelisted-ips="" \

data/lib/bidi2pdf/bidi/browser_tab.rb

@@ -4,6 +4,7 @@ require "base64"
 
 require_relative "network_events"
 require_relative "logger_events"
+require_relative "navigation_failed_events"
 require_relative "auth_interceptor"
 require_relative "add_headers_interceptor"
 require_relative "js_logger_helper"
@@ -32,6 +33,11 @@ require_relative "js_logger_helper"
 # @param [String] user_context_id The ID of the user context.
 module Bidi2pdf
   module Bidi
+    # Represents a browser tab for managing interactions and communication
+    # using the Bidi2pdf library. This class provides methods for creating
+    # browser tabs, managing cookies, navigating to URLs, executing scripts,
+    # handling network events, and general tab lifecycle management.
+    #
     class BrowserTab
       include JsLoggerHelper
 
@@ -56,6 +62,9 @@ module Bidi2pdf
       # @return [LoggerEvents] The logger events handler.
       attr_reader :logger_events
 
+      # @return [NavigationFailedEvents] The navigation failed events handler.
+      attr_reader :navigation_failed_events
+
       # Initializes a new browser tab.
       #
       # @param [Object] client The WebSocket client for communication.
@@ -68,6 +77,7 @@ module Bidi2pdf
         @tabs = []
         @network_events = NetworkEvents.new browsing_context_id
         @logger_events = LoggerEvents.new browsing_context_id
+        @navigation_failed_events = NavigationFailedEvents.new browsing_context_id
         @open = true
       end
 
@@ -131,7 +141,7 @@ module Bidi2pdf
         headers:,
         url_patterns:
       )
-        AddHeadersInterceptor.new(
+        @header_interceptor = AddHeadersInterceptor.new(
           context: browsing_context_id,
           url_patterns: url_patterns,
           headers: headers
@@ -145,7 +155,7 @@ module Bidi2pdf
       # @param [Array<String>] url_patterns The URL patterns to match.
       # @return [AuthInterceptor] The interceptor instance.
       def basic_auth(username:, password:, url_patterns:)
-        AuthInterceptor.new(
+        @basic_auth_interceptor = AuthInterceptor.new(
           context: browsing_context_id,
           url_patterns: url_patterns,
           username: username, password: password
@@ -154,8 +164,21 @@ module Bidi2pdf
 
       # Navigates the browser tab to a specified URL.
       #
+      # This method registers necessary event listeners and sends a navigation
+      # command to the browser tab, instructing it to load the specified URL.
+      # It validates that the URL is properly formatted before attempting navigation.
+      #
       # @param [String] url The URL to navigate to.
+      # @raise [NavigationError] If the URL is invalid or improperly formatted.
+      # @example
+      #   browser_tab.navigate_to("https://example.com")
       def navigate_to(url)
+        begin
+          URI.parse(url)
+        rescue URI::InvalidURIError => e
+          raise NavigationError, "Invalid URL: #{url} - #{e.message}"
+        end
+
         Bidi2pdf.notification_service.instrument("navigate_to.bidi2pdf", url: url) do
           navigate_with_listeners url
         end
@@ -389,18 +412,32 @@ module Bidi2pdf
         client.send_cmd_and_wait(cmd) do |response|
           Bidi2pdf.logger.debug "Navigated to page url: #{url} response: #{response}"
         end
+      rescue Bidi2pdf::CmdError => e
+        msg = e.response["message"]
+        case msg
+        when /^net::ERR_INVALID_AUTH_CREDENTIALS/
+          raise NavigationAuthError.new(url, msg)
+        when /^net::ERR_NAME_NOT_RESOLVED/
+          raise NavigationDNSError.new(url, msg)
+        when /^net::/
+          raise NavigationError, "Connection error: #{url} #{msg}"
+        else
+          raise e
+        end
       end
 
       def register_event_listeners
         return if @event_handlers_registered
 
         @event_handlers_registered = true
+        @listener_refs ||= {}
 
-        client.on_event("network.beforeRequestSent", "network.responseStarted", "network.responseCompleted", "network.fetchError",
-                        &network_events.method(:handle_event))
+        @listener_refs[:network] = client.on_event("network.beforeRequestSent", "network.responseStarted", "network.responseCompleted", "network.fetchError",
+                                                   &network_events.method(:handle_event))
 
-        client.on_event("log.entryAdded",
-                        &logger_events.method(:handle_event))
+        @listener_refs[:logger] = client.on_event("log.entryAdded", &logger_events.method(:handle_event))
+
+        @listener_refs[:navigation_failed] = client.on_event("browsingContext.navigationFailed", &navigation_failed_events.method(:handle_event))
       end
 
       def handle_injection_exception(response, url, exception_class)
@@ -531,13 +568,27 @@ module Bidi2pdf
       end
 
       # Removes event listeners for the browser tab.
+      # rubocop:disable Metrics/AbcSize
       def remove_event_listeners
+        return if @listener_refs.nil? || @listener_refs.empty?
+
         Bidi2pdf.logger.debug2 "Network events: #{network_events.all_events.map(&:to_s)}"
 
-        client.remove_event_listener "network.responseStarted", "network.responseCompleted", "network.fetchError",
-                                     &network_events.method(:handle_event)
+        client.remove_event_listener("network.beforeRequestSent", "network.responseStarted", "network.responseCompleted", "network.fetchError",
+                                     @listener_refs[:network])
+
+        client.remove_event_listener("log.entryAdded", @listener_refs[:logger])
+
+        @header_interceptor&.unregister_with_client(client: client)
+        @basic_auth_interceptor&.unregister_with_client(client: client)
+
+        @header_interceptor = nil
+        @basic_auth_interceptor = nil
+        @listener_refs = {}
       end
 
+      # rubocop:enable Metrics/AbcSize
+
       # Closes all tabs associated with the browser tab.
       def close_tabs
         tabs.each do |tab|

data/lib/bidi2pdf/bidi/client.rb

@@ -125,23 +125,25 @@ module Bidi2pdf
       #
       # @param [Array<String>] names The names of the events to subscribe to.
       # @yield [event_data] A block to handle the event data.
-      def on_event(*names, &block)
-        names.each { |name| dispatcher.on_event(name, &block) }
+      def on_event(*names, &)
+        listener = dispatcher.on_event(*names, &)
         cmd = Bidi2pdf::Bidi::Commands::SessionSubscribe.new(events: names)
         send_cmd(cmd) if names.any?
+
+        listener
       end
 
       # Removes a message listener.
       #
       # @param [Proc] block The listener block to remove.
-      def remove_message_listener(block) = dispatcher.remove_message_listener(block)
+      def remove_message_listener(listener) = dispatcher.remove_message_listener(listener)
 
       # Removes event listeners for specific events.
       #
       # @param [Array<String>] names The names of the events to unsubscribe from.
       # @param [Proc] block The listener block to remove.
-      def remove_event_listener(*names, &block)
-        names.each { |event_name| dispatcher.remove_event_listener(event_name, block) }
+      def remove_event_listener(*names, listener)
+        names.each { |event_name| dispatcher.remove_event_listener(event_name, listener) }
       end
 
       # Closes the WebSocket connection.

data/lib/bidi2pdf/bidi/command_manager.rb

@@ -5,11 +5,10 @@ module Bidi2pdf
     class CommandManager
       class << self
         def initialize_counter
-          @id = 0
-          @id_mutex = Mutex.new
+          @id = Concurrent::AtomicFixnum.new(0)
         end
 
-        def next_id = @id_mutex.synchronize { @id += 1 }
+        def next_id = @id.increment
       end
 
       initialize_counter
@@ -17,19 +16,14 @@ module Bidi2pdf
       def initialize(socket)
         @socket = socket
 
-        @pending_responses = {}
-        @initiated_cmds = {}
+        @pending_responses = Concurrent::Hash.new
       end
 
-      def send_cmd(cmd, store_response: false)
+      def send_cmd(cmd, result_queue: nil)
         id = next_id
 
         Bidi2pdf.notification_service.instrument("send_cmd.bidi2pdf", id: id, cmd: cmd) do |instrumentation_payload|
-          if store_response
-            init_queue_for id
-          else
-            @initiated_cmds[id] = true
-          end
+          init_queue_for id, result_queue
 
           payload = cmd.as_payload(id)
 
@@ -42,17 +36,20 @@ module Bidi2pdf
       end
 
       def send_cmd_and_wait(cmd, timeout: Bidi2pdf.default_timeout, &block)
+        result_queue = Thread::Queue.new
+
         Bidi2pdf.notification_service.instrument("send_cmd_and_wait.bidi2pdf", cmd: cmd, timeout: timeout) do |instrumentation_payload|
-          id = send_cmd(cmd, store_response: true)
+          id = send_cmd(cmd, result_queue: result_queue)
 
           instrumentation_payload[:id] = id
 
-          response = pop_response id, timeout: timeout
+          response = result_queue.pop(timeout: timeout)
 
           instrumentation_payload[:response] = response
 
           raise CmdTimeoutError, "Timeout waiting for response to command ID #{id}" if response.nil?
-          raise CmdError, "Error response: #{response["error"]} #{cmd.inspect}" if response["error"]
+
+          raise Bidi2pdf::CmdError.new(cmd, response) if response["error"]
 
           block ? block.call(response) : response
         ensure
@@ -60,14 +57,6 @@ module Bidi2pdf
         end
       end
 
-      def pop_response(id, timeout:)
-        raise CmdResponseNotStoredError, "No response stored for command ID #{id} or already popped or this command was not send" unless @pending_responses.key?(id)
-
-        @pending_responses[id].pop(timeout: timeout)
-      ensure
-        @pending_responses.delete(id)
-      end
-
       def handle_response(data)
         Bidi2pdf.notification_service.instrument("handle_response.bidi2pdf", data: data) do |instrumentation_payload|
           instrumentation_payload[:error] = data["error"] if data["error"]
@@ -78,9 +67,6 @@ module Bidi2pdf
 
             if @pending_responses.key?(id)
               @pending_responses[id]&.push(data)
-              return true
-            elsif @initiated_cmds.key?(id)
-              @initiated_cmds.delete(id)
 
               return true
             end
@@ -89,12 +75,14 @@ module Bidi2pdf
           instrumentation_payload[:handled] = false
 
           false
+        ensure
+          @pending_responses.delete id
         end
       end
 
       private
 
-      def init_queue_for(id) = @pending_responses[id] = Thread::Queue.new
+      def init_queue_for(id, result_queue) = @pending_responses[id] = result_queue
 
       def next_id = self.class.next_id
     end

data/lib/bidi2pdf/bidi/connection_manager.rb

@@ -6,7 +6,7 @@ module Bidi2pdf
       def initialize(logger:)
         @logger = logger
         @connected = false
-        @connection_queue = Thread::Queue.new
+        @connection_latch = Concurrent::CountDownLatch.new(1)
       end
 
       def mark_connected
@@ -14,7 +14,7 @@ module Bidi2pdf
 
         @connected = true
         @logger.debug "WebSocket connection is open"
-        @connection_queue.push(true)
+        @connection_latch.count_down
       end
 
       def wait_until_open(timeout:)
@@ -22,13 +22,7 @@ module Bidi2pdf
 
         @logger.debug "Waiting for WebSocket connection to open"
 
-        begin
-          Timeout.timeout(timeout) do
-            @connection_queue.pop
-          end
-        rescue Timeout::Error
-          raise Bidi2pdf::WebsocketError, "WebSocket connection did not open in time #{timeout} sec."
-        end
+        raise Bidi2pdf::WebsocketError, "WebSocket connection did not open in time #{timeout} sec." unless @connection_latch.wait(timeout)
 
         true
       end

data/lib/bidi2pdf/bidi/event_manager.rb

@@ -3,20 +3,50 @@
 module Bidi2pdf
   module Bidi
     class EventManager
+      Listener = Struct.new(:block, :id, :source_location) do
+        def initialize(block, id = SecureRandom.uuid)
+          super
+          self.source_location = block.source_location
+        end
+
+        def call(*args)
+          block.call(*args)
+        end
+
+        def ==(other)
+          other.is_a?(Listener) && id == other.id
+        end
+
+        alias_method :eql?, :==
+
+        def hash
+          id.hash
+        end
+      end
+
       attr_reader :type
 
       def initialize(type)
-        @listeners = Hash.new { |h, k| h[k] = [] }
+        @listeners = Concurrent::Hash.new { |h, k| h[k] = [] }
         @type = type
       end
 
       def on(*event_names, &block)
-        event_names.each { |event_name| @listeners[event_name.to_sym] << block }
-
-        block
+        Listener.new(block).tap do |listener|
+          event_names.each do |event_name|
+            @listeners[event_name.to_sym] << listener
+            log_msg("Adding #{event_name} listener", listener)
+          end
+        end
       end
 
-      def off(event_name, block) = @listeners[event_name.to_sym].delete(block)
+      def off(event_name, listener)
+        raise ArgumentError, "Listener not registered" unless listener.is_a?(Listener)
+
+        log_msg("Removing #{event_name} listener", listener)
+
+        @listeners[event_name.to_sym].delete(listener)
+      end
 
       def dispatch(event_name, *args)
         listeners = @listeners[event_name.to_sym] || []

data/lib/bidi2pdf/bidi/interceptor.rb

@@ -27,14 +27,24 @@ module Bidi2pdf
         client.send_cmd_and_wait(cmd) do |response|
           @interceptor_id = response["result"]["intercept"]
 
-          Bidi2pdf.logger.debug "Interceptor added: #{@interceptor_id}"
+          Bidi2pdf.logger.debug2 "Interceptor added: #{@interceptor_id}"
 
-          client.on_event(*self.class.events, &method(:handle_event))
+          @handle_event_listener = client.on_event(*self.class.events, &method(:handle_event))
 
           self
         end
       end
 
+      def unregister_with_client(client:)
+        return unless @handle_event_listener
+
+        client.remove_event_listener(*self.class.events, @handle_event_listener)
+
+        Bidi2pdf.logger.debug2 "Interceptor removed: #{@interceptor_id}"
+
+        @handle_event_listener = nil
+      end
+
       # rubocop: disable Metrics/AbcSize
       def handle_event(response)
         event_response = response["params"]

data/lib/bidi2pdf/bidi/navigation_failed_events.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+require_relative "browser_console_logger"
+
+module Bidi2pdf
+  module Bidi
+    class NavigationFailedEvents
+      attr_reader :context_id, :browser_console_logger
+
+      def initialize(context_id)
+        @context_id = context_id
+      end
+
+      def handle_event(data)
+        event = data["params"]
+        method = data["method"]
+
+        if event["context"] == context_id
+          handle_response(method, event)
+        else
+          Bidi2pdf.logger.debug2 "Ignoring Log event: #{method}, context_id: #{context_id}, params: #{event}"
+        end
+      end
+
+      def handle_response(_method, event)
+        url = event["url"]
+        navigation = event["navigation"]
+        timestamp = event["timestamp"]
+
+        Bidi2pdf.notification_service.instrument("navigation_failed_received.bidi2pdf",
+                                                 {
+                                                   url: url,
+                                                   timestamp: timestamp,
+                                                   navigation: navigation
+                                                 })
+
+        Bidi2pdf.logger.error "Navigation failed for URL: #{url}, Navigation: #{navigation}"
+      end
+    end
+  end
+end