rails-profiler 0.34.2 → 0.34.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b6365d7a34d882d20ca0257247629d6099917508ad78cecc96726dc6788c7a75
-  data.tar.gz: e7dffe134ddcac64202aae83bb73963837068ae258b1cdc1d959e78504b31861
+  metadata.gz: d4912dd474a748e657c35111e3224d83500e75960ef2592e5cd652155dc62617
+  data.tar.gz: d0d4cb0f34f25d98f5be08875bdc59ec4cc84894f7cbe0b512fd1ecd153dda4b
 SHA512:
-  metadata.gz: 8912bad06bc0824942c4e696d9fd408dff3a76238af92faaa7c262815e477e3850fe90c14d9acd6ceced45271180c990c8d6d597cf591d998eae48edf3dc9e11
-  data.tar.gz: aeb7cec816ce9de90ab3dcc72629d36c6f9b1ff69bfa618416e72fa5cbb6301cae5a5d12dd09d6d0a4c247c76095c397777dad9f42a18bbf5aaafa7d375b7248
+  metadata.gz: 3ca18ad3dd582e68c2bb935a5ace8628dc466a4999fa655e54117879548d543c5a291ec98765eb25456b0ff8fa774ee0f98f939544a346aa9b3368d37c421f72
+  data.tar.gz: e144011d2fd93d37aa3166748bf2d3671d8d246325a475cf13d756b188dd99999ef6f95b232fd36eca444c91e24e2715c1902695697ea1930407bc932b2e952d

data/CHANGELOG.md

@@ -19,6 +19,44 @@ every commit of every tag interval is accounted for one way or the other.
 
 ## [Unreleased]
 
+## [0.34.4] - 2026-10-16
+
+<!-- stamped -->
+
+### Fixed
+
+- **Sidekiq:** A job run through the ActiveJob adapter is no longer profiled twice. It reaches the
+  Sidekiq middleware wrapped (`job["wrapped"]`) and is already profiled by the ActiveJob
+  `around_perform` callback, so the middleware now leaves it alone when ActiveJob is instrumented
+  and the same work is recorded once.
+- **Logs:** The log buffer behind `CaptureLogger` is now guarded by a `Mutex`, so lines recorded
+  from several threads at once (a streamed body iterated by another thread, work handed the buffer)
+  keep the byte count and the stored lines consistent.
+- **Flame graph and views:** Nested durations are counted once. The total of the flame graph and
+  the total of the views and partials now cover each moment a single time, so a template that
+  renders partials, or a controller action that runs queries, is no longer added to what it
+  contains.
+
+## [0.34.3] - 2026-10-15
+
+<!-- stamped -->
+
+### Changed
+
+- **Test suite:** `bundle exec rspec` now measures the line coverage of the gem's Ruby code with
+  SimpleCov, and the CI fails below 92.0%, the level measured on Ruby 3.3 once the MCP
+  tools, the function profiler, the storage base class and the cluster client got specs of their
+  own. A run of a few spec files is held to no floor outside the CI.
+- **Test suite:** The suite loads and passes under any locale, `LANG=C` included: `spec_helper.rb`
+  reads files as UTF-8 whatever the locale, and a `.rspec` file loads it for every spec. The specs
+  of the masking of sensitive data now live together under `spec/redaction/`.
+
+### Fixed
+
+- **Middleware:** The default `skip_paths` are now anchored to the whole path, so a page such as `/users/favicon.ico.html` is no longer skipped by mistake, and the Sprockets assets (`/assets`) and Action Cable (`/cable`) requests are no longer profiled. A `skip_paths` list you set yourself is used as given.
+- **Mailers:** Visiting the mailer previews (`/rails/mailers`) no longer records phantom delivered emails in the profile.
+- **Security:** An `authorize_with` block that raises is now treated as a denial: the error is logged server side and the request is served unprofiled, instead of breaking every request of the application.
+
 ## [0.34.2] - 2026-10-15
 
 <!-- stamped -->

data/lib/profiler/collectors/base_collector.rb

@@ -124,6 +124,24 @@ module Profiler
 
       protected
 
+      # The time covered by +intervals+ ([started, finished] pairs of Time or Float seconds), in
+      # milliseconds, each moment counted once: a template that renders partials, or a controller
+      # action that runs queries, is not added to what it contains.
+      def covered_ms(intervals)
+        covered = 0.0
+        current_end = nil
+        intervals.sort_by(&:first).each do |started, finished|
+          if current_end.nil? || started >= current_end
+            covered += finished - started
+            current_end = finished
+          elsif finished > current_end
+            covered += finished - current_end
+            current_end = finished
+          end
+        end
+        (covered * 1000).round(2)
+      end
+
       def store_data(data)
         @data = data
         @profile.add_collector_data(name, data)

data/lib/profiler/collectors/flamegraph_collector.rb

@@ -139,7 +139,7 @@ module Profiler
 
         store_data({
           total_events: @events.size,
-          total_duration: @events.empty? ? 0 : @events.map(&:duration).sum.round(2),
+          total_duration: @events.empty? ? 0 : covered_ms(@events.map { |e| [e.started_at, e.finished_at] }),
           root_events: root_events.map(&:to_h)
         })
       end

data/lib/profiler/collectors/log_collector.rb

@@ -20,13 +20,22 @@ module Profiler
           @bytes = 0
           @dropped = 0
           @truncated = false
+          @mutex = Mutex.new
         end
 
         def truncated?
           @truncated
         end
 
+        # A streamed body iterated by another thread, or work handed the buffer, can record while
+        # the request's own thread does: the byte count and the lines are updated together.
         def record(entry)
+          @mutex.synchronize { record_unlocked(entry) }
+        end
+
+        private
+
+        def record_unlocked(entry)
           message = entry[:message]
           if @limit
             room = @limit - @bytes

data/lib/profiler/collectors/mailer_collector.rb

@@ -316,8 +316,9 @@ module Profiler
       end
 
       def rails_preview_request?
-        path = Thread.current[:profiler_request_path]
-        path&.start_with?("/rails/mailers")
+        # The profile of the request being served carries its path; the thread-local is kept
+        # for a caller that sets it by hand.
+        [profile&.path, Thread.current[:profiler_request_path]].any? { |path| path.to_s.match?(%r{\A/rails/mailers(/|\z)}) }
       end
 
       def to_path(class_name)

data/lib/profiler/collectors/view_collector.rb

@@ -9,6 +9,7 @@ module Profiler
         super
         @views = []
         @partials = []
+        @intervals = []
         @subscriptions = []
       end
 
@@ -36,6 +37,7 @@ module Profiler
 
         @subscriptions << subscribe_notification("render_template.action_view") do |name, started, finished, unique_id, payload|
           duration = ((finished - started) * 1000).round(2)
+          @intervals << [started, finished]
           @views << {
             identifier: payload[:identifier],
             layout: payload[:layout],
@@ -45,6 +47,7 @@ module Profiler
 
         @subscriptions << subscribe_notification("render_partial.action_view") do |name, started, finished, unique_id, payload|
           duration = ((finished - started) * 1000).round(2)
+          @intervals << [started, finished]
           @partials << {
             identifier: payload[:identifier],
             duration: duration
@@ -60,7 +63,7 @@ module Profiler
           partials: @partials,
           total_views: @views.size,
           total_partials: @partials.size,
-          total_duration: (@views + @partials).sum { |v| v[:duration] }.round(2)
+          total_duration: covered_ms(@intervals)
         }
 
         store_data(data)
@@ -76,7 +79,7 @@ module Profiler
       end
 
       def toolbar_summary
-        total_duration = (@views + @partials).sum { |v| v[:duration] }.round(2)
+        total_duration = covered_ms(@intervals)
 
         {
           text: "#{@views.size} views, #{@partials.size} partials (#{total_duration}ms)",

data/lib/profiler/configuration.rb

@@ -65,7 +65,8 @@ module Profiler
       @storage = :memory
       @storage_options = {}
       @collectors = []
-      @skip_paths = [%r{^/_profiler}, /\.well-known/, /favicon\.ico/, /manifest\.json/]
+      @skip_paths = [%r{\A/_profiler}, %r{\A/\.well-known/}, %r{\A/favicon\.ico\z}, %r{\A(?:/[^/]+)*/manifest\.json\z},
+                     %r{\A/assets/}, %r{\A/cable(/|\z)}]
       @slow_query_threshold = 100 # milliseconds
       @max_queries_warning = 50
       # Where each query comes from: :first_and_slow captures the caller the first time a
@@ -284,7 +285,7 @@ module Profiler
         LocalRequest.warn_once(reason)
         false
       when :allow_authorized
-        @authorize_block ? @authorize_block.call(request) : false
+        run_authorize_block(request)
       else
         false
       end
@@ -373,5 +374,16 @@ module Profiler
       raise ArgumentError, "config.mcp_allowed_hosts: #{pattern.inspect} cannot be matched against the whole " \
                            "host name (#{e.message})"
     end
+
+    # An exception in the application's own authorization code is a denial, never an error
+    # for the request: it would otherwise break every request. The detail stays in the log.
+    def run_authorize_block(request)
+      return false unless @authorize_block
+
+      @authorize_block.call(request) ? true : false
+    rescue StandardError => e
+      Profiler.log_error("authorize_with block raised, treating the request as unauthorized", e)
+      false
+    end
   end
 end

data/lib/profiler/instrumentation/sidekiq_middleware.rb

@@ -12,6 +12,11 @@ module Profiler
     class SidekiqMiddleware
       def call(worker, job, queue, &block)
         Profiler.env_override_store.apply!
+        # A job run through the ActiveJob adapter reaches Sidekiq wrapped (job["wrapped"] names the
+        # real class) and is profiled by ActiveJobInstrumentation's around_perform: profiling the
+        # wrapper too would record the same work twice.
+        return block.call if job["wrapped"] && active_job_profiled?
+
         Profiler::JobProfiler.profile(
           job_class: job["class"],
           job_id: job["jid"],
@@ -22,6 +27,13 @@ module Profiler
           &block
         )
       end
+
+      private
+
+      def active_job_profiled?
+        defined?(ActiveJob::Base) && defined?(Profiler::Instrumentation::ActiveJobInstrumentation) &&
+          ActiveJob::Base.include?(Profiler::Instrumentation::ActiveJobInstrumentation)
+      end
     end
   end
 end

data/lib/profiler/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Profiler
-  VERSION = "0.34.2"
+  VERSION = "0.34.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails-profiler
 version: !ruby/object:Gem::Version
-  version: 0.34.2
+  version: 0.34.4
 platform: ruby
 authors:
 - Sébastien Duplessy
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-10-15 00:00:00.000000000 Z
+date: 2026-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails