rails_error_dashboard 0.11.7 → 0.11.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f55384ffa88ee1326e9575a6432eb392df3995395b8b4e821b5ec2576d34ab53
-  data.tar.gz: a4166ab78f73a536986bb7b0c68c6ee95975db9769a611a69646900ca0d45d95
+  metadata.gz: f07dcfe9569e107c4f8b3bee4f15e106c4b537690d9bf002af887e832d9e0a9b
+  data.tar.gz: 5defe92b8156a8564a6a1286d2e5199b450b2f6ca894bcafd65586fe3723a7e0
 SHA512:
-  metadata.gz: 218881302d12e08887dc80dfcebd3d32b010d931dbec325c8f88317300f6c5006e88edab2d01e94d4ba8a12bc01daa0d19b94c9a787515b611ae980860ab4e0c
-  data.tar.gz: 7d7ddd33b49891aa68ad8fa206465b4aa7f2f3a6ae1f2ae5e354a3f19f8b8ee6a68776a2662a14bbaedfb4029ccc15c5c479c0e2cd10d26869b2aff75021df45
+  metadata.gz: 0fe0f1b0894c8c1c797e341b2e00d2120871dca2bcc2800527f69a99831cd700005f21245b6a3a0fef86f0c7466ec0e1f588e21cb503c8c3621f9d203c79ce4c
+  data.tar.gz: ba3e9269169d059eeab3cf3f469294ac520a9144558199c05e0dc0b5262b7bbbd0ec8a28896f15fd26ee5a605ed74f5707eae7e38a354160341b676cbe9d7d58

data/app/jobs/rails_error_dashboard/application_job.rb

@@ -6,8 +6,8 @@ module RailsErrorDashboard
     include Concerns::LocalizedJob
 
     # CRITICAL: Ensure job failures don't break the app or spam error logs
-    # Retry failed jobs with exponential backoff, but limit attempts
-    retry_on StandardError, wait: :exponentially_longer, attempts: 3
+    # Retry failed jobs with polynomial backoff, but limit attempts
+    retry_on StandardError, wait: :polynomially_longer, attempts: 3
 
     # Global exception handling for all dashboard jobs
     rescue_from StandardError do |exception|

data/db/migrate/20260909000001_replace_message_gin_with_searchable_text_index.rb

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+# The dashboard's PostgreSQL full-text search (Queries::ErrorsList#filter_by_search)
+# matches against message, backtrace and error_type together:
+#
+#   to_tsvector('english', COALESCE(message, '') || ' ' || COALESCE(backtrace, '') || ' ' || COALESCE(error_type, ''))
+#
+# PostgreSQL uses an expression index only when the query's expression is
+# identical to the index's, so of the two GIN indexes defined over time only
+# one can serve that search:
+#
+#   index_error_logs_on_message_gin       to_tsvector('english', message)              never used by any query
+#   index_error_logs_on_searchable_text   the expression above                          the one the search needs
+#
+# Which of them a host has depends on how it was installed. Hosts that ran the
+# incremental chain got both; hosts installed from the squashed first migration
+# got neither (the later index migrations skip themselves once the squash's
+# composite indexes exist), and 0.11.7 then added only the message-only one.
+#
+# This converges every install on index_error_logs_on_searchable_text alone.
+# No-op on other adapters.
+class ReplaceMessageGinWithSearchableTextIndex < ActiveRecord::Migration[7.0]
+  TABLE = :rails_error_dashboard_error_logs
+
+  def up
+    return unless postgresql?
+    return unless table_exists?(TABLE)
+
+    execute <<~SQL
+      CREATE INDEX IF NOT EXISTS index_error_logs_on_searchable_text
+      ON rails_error_dashboard_error_logs
+      USING gin(to_tsvector('english',
+        COALESCE(message, '') || ' ' || COALESCE(backtrace, '') || ' ' || COALESCE(error_type, '')
+      ))
+    SQL
+
+    execute "DROP INDEX IF EXISTS index_error_logs_on_message_gin"
+  end
+
+  def down
+    return unless postgresql?
+
+    # There is no single state to return to: before this migration a host had
+    # either both indexes or only the message-only one, depending on its
+    # install path. Dropping searchable_text here would take away, on
+    # incremental-chain hosts, an index they had before, and nothing needs
+    # message_gin back. Earlier gem versions run unchanged against the
+    # converged schema.
+    raise ActiveRecord::IrreversibleMigration,
+          "this migration converges two index layouts (see its header); the previous layout is not recoverable"
+  end
+
+  private
+
+  def postgresql?
+    connection.adapter_name.downcase == "postgresql"
+  end
+end

data/lib/generators/rails_error_dashboard/install/templates/initializer.rb

@@ -420,9 +420,10 @@ RailsErrorDashboard.configure do |config|
   # config.crash_capture_path = "/tmp/my_app_crashes"
 
 <% end -%>
-  # Repository settings (auto-detected from git remote, optional override)
-  # config.repository_url = ENV["REPOSITORY_URL"]  # e.g., "https://github.com/user/repo"
-  # config.repository_branch = ENV.fetch("REPOSITORY_BRANCH", "main")  # Default branch
+  # "View Source" links need config.git_repository_url (set under Enhanced
+  # metrics below); nothing is auto-detected from the git remote.
+  # Which ref the links point at:
+  # config.git_branch_strategy = :commit_sha  # :commit_sha (default), :current_branch, or :main
 
   # ============================================================================
   # INTERNAL LOGGING (Silent by Default)

data/lib/rails_error_dashboard/services/github_link_generator.rb

@@ -15,6 +15,11 @@ module RailsErrorDashboard
     #   generator.generate_link
     #   # => "https://github.com/user/repo/blob/abc123def456/app/models/user.rb#L42"
     class GithubLinkGenerator
+      # The generated link is an href handed to every dashboard user, so only
+      # web URLs are ever linked. javascript:, data: and scp-style git@ forms
+      # produce no link even though the value comes from deployer configuration.
+      ALLOWED_SCHEMES = %w[http https].freeze
+
       attr_reader :repository_url, :file_path, :line_number, :commit_sha, :branch, :error
 
       # Initialize a new link generator
@@ -42,6 +47,11 @@ module RailsErrorDashboard
         # Normalize repository URL
         normalized_repo = normalize_repository_url
 
+        unless repository_host
+          @error = "Unsupported repository URL scheme: only http and https URLs are linked"
+          return nil
+        end
+
         # Determine reference (commit SHA or branch)
         reference = determine_reference
 
@@ -77,16 +87,33 @@ module RailsErrorDashboard
         url
       end
 
-      # Detect repository type from URL
+      # Host of the repository URL, downcased, or nil when the value is not an
+      # http(s) URL with a host.
+      #
+      # @return [String, nil]
+      def repository_host
+        uri = URI.parse(normalize_repository_url)
+        return nil unless ALLOWED_SCHEMES.include?(uri.scheme.to_s.downcase) && uri.host.present?
+
+        uri.host.downcase
+      rescue URI::InvalidURIError
+        nil
+      end
+
+      # Detect repository type from the URL's host. Only the host is consulted,
+      # so "github.com" appearing in a path, query string or userinfo does not
+      # count. Self-hosted GitLab, Bitbucket, Gitea and Forgejo instances are
+      # recognised by their conventional subdomain ("gitlab.example.com").
       #
       # @return [Symbol] :github, :gitlab, :bitbucket, :codeberg, or :unknown
       def detect_repository_type
-        normalized = normalize_repository_url.downcase
+        host = repository_host
+        return :unknown unless host
 
-        return :github if normalized.include?("github.com")
-        return :gitlab if normalized.include?("gitlab.com") || normalized.include?("gitlab.")
-        return :bitbucket if normalized.include?("bitbucket.org") || normalized.include?("bitbucket.")
-        return :codeberg if normalized.include?("codeberg.org") || normalized.include?("gitea.") || normalized.include?("forgejo.")
+        return :github if host == "github.com" || host.end_with?(".github.com")
+        return :gitlab if host == "gitlab.com" || host.include?("gitlab.")
+        return :bitbucket if host == "bitbucket.org" || host.include?("bitbucket.")
+        return :codeberg if host == "codeberg.org" || host.include?("gitea.") || host.include?("forgejo.")
 
         :unknown
       end

data/lib/rails_error_dashboard/version.rb

@@ -1,3 +1,3 @@
 module RailsErrorDashboard
-  VERSION = "0.11.7"
+  VERSION = "0.11.9"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_error_dashboard
 version: !ruby/object:Gem::Version
-  version: 0.11.7
+  version: 0.11.9
 platform: ruby
 authors:
 - Anjan Jagirdar
@@ -410,6 +410,7 @@ files:
 - db/migrate/20260826000001_add_environment_to_error_logs.rb
 - db/migrate/20260908000001_add_release_to_error_occurrences.rb
 - db/migrate/20260908220001_add_missing_postgresql_indexes_to_error_logs.rb
+- db/migrate/20260909000001_replace_message_gin_with_searchable_text_index.rb
 - lib/generators/rails_error_dashboard/install/install_generator.rb
 - lib/generators/rails_error_dashboard/install/templates/README
 - lib/generators/rails_error_dashboard/install/templates/initializer.rb
@@ -578,7 +579,7 @@ metadata:
   funding_uri: https://github.com/sponsors/AnjanJ
 post_install_message: |
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-    RED (Rails Error Dashboard) v0.11.7
+    RED (Rails Error Dashboard) v0.11.9
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
   First install: