app_profiler 0.3.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e78035c478029d4ad4de636efab44240957c2bd05ea761dc9fe28b753b4fa1d3
-  data.tar.gz: c4d6659a31eaa7440c6c8a2550c9cf41fbadb567426cae7c6bd0cd4442e8039a
+  metadata.gz: 2e510f0a0d20ec300e0ae5593adbdb03195202ea68e6a8ecad5fb13abfef9473
+  data.tar.gz: 66dd66baaa5e863a0b0311431da9ab17f794c3e8902e73b6d29f998c0867db74
 SHA512:
-  metadata.gz: 45610912af0b6952b2c104d27bfc3bf2b37b92f91d2c873bdf9e2bfc43498529decf9568af63b9ee47d84794dd921dbf6312de11aac272be296f99416d5f2b31
-  data.tar.gz: 29ee1a661cc8c81274fbdb7f6123ce90d7dbeaa351ca3b16a8a6bfe933ec3f68385bfe0f2d0afb2ae6c49ba59fa318d733bd23d6cf7724a3d03f1246e36b65ca
+  metadata.gz: 67c22d47cea0a3fe91928495b4d14f476c727bd98c976bb7ae310b61b317c0a54dc91d350c870cc8cc155a044f33f49dfc5a1c739bd68609261af537c43b8913
+  data.tar.gz: 0e57a16a1bf6195fb8b723ae213e8fb2226a7858448420ef4e495363402033f1e9612971411981f505b9901dde425fc910d82023b788e1d60311d44ff3a0d258

data/lib/app_profiler/backend/base_backend.rb

@@ -3,8 +3,27 @@
 module AppProfiler
   module Backend
     class BaseBackend
-      def run(params = {}, &block)
-        raise NotImplementedError
+      def run(params = {})
+        started = start(params)
+        start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+
+        yield
+
+        return unless started
+
+        duration = Process.clock_gettime(Process::CLOCK_MONOTONIC) - start_time
+
+        stop
+        results_data = results
+
+        if results_data
+          results_data.metadata[:duration] = duration
+        end
+
+        results_data
+      ensure
+        # Only stop the profiler if profiling was started in this context.
+        stop if started
       end
 
       def start(params = {})

data/lib/app_profiler/backend/stackprof_backend.rb

@@ -22,20 +22,6 @@ module AppProfiler
         end
       end
 
-      def run(params = {})
-        started = start(params)
-
-        yield
-
-        return unless started
-
-        stop
-        results
-      ensure
-        # Only stop the profiler if profiling was started in this context.
-        stop if started
-      end
-
       def start(params = {})
         # Do not start the profiler if StackProf was started somewhere else.
         return false if running?

data/lib/app_profiler/backend/vernier_backend.rb

@@ -21,20 +21,6 @@ module AppProfiler
         end
       end
 
-      def run(params = {})
-        started = start(params)
-
-        yield
-
-        return unless started
-
-        stop
-        results
-      ensure
-        # Only stop the profiler if profiling was started in this context.
-        stop if started
-      end
-
       def start(params = {})
         # Do not start the profiler if we already have a collector started somewhere else.
         return false if running?

data/lib/app_profiler/base_profile.rb

@@ -47,6 +47,10 @@ module AppProfiler
       metadata[PROFILE_ID_METADATA_KEY]
     end
 
+    def duration
+      metadata[:duration]
+    end
+
     def upload
       AppProfiler.storage.upload(self).tap do |upload|
         if upload && defined?(upload.url)

data/lib/app_profiler/railtie.rb

@@ -34,6 +34,7 @@ module AppProfiler
       )
       AppProfiler.profile_header = app.config.app_profiler.profile_header || "X-Profile"
       AppProfiler.profile_async_header = app.config.app_profiler.profile_async_header || "X-Profile-Async"
+      AppProfiler.profile_param = app.config.app_profiler.profile_param || "profile"
       AppProfiler.profile_root = app.config.app_profiler.profile_root || Rails.root.join(
         "tmp", "app_profiler"
       )

data/lib/app_profiler/request_parameters.rb

@@ -66,7 +66,7 @@ module AppProfiler
     private
 
     def mode
-      query_param("profile") || profile_header_param("mode")
+      query_param(AppProfiler.profile_param) || profile_header_param("mode")
     end
 
     def ignore_gc

data/lib/app_profiler/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AppProfiler
-  VERSION = "0.3.0"
+  VERSION = "0.5.0"
 end

data/lib/app_profiler.rb

@@ -3,6 +3,7 @@
 require "active_support"
 require "active_support/core_ext/class"
 require "active_support/core_ext/module"
+require "active_support/core_ext/time/zones"
 require "logger"
 require "app_profiler/version"
 
@@ -40,15 +41,16 @@ module AppProfiler
   end
 
   autoload(:Middleware, "app_profiler/middleware")
-  autoload(:Parameters, "app_profiler/parameters")
-  autoload(:RequestParameters, "app_profiler/request_parameters")
   autoload(:BaseProfile, "app_profiler/base_profile")
   autoload :StackprofProfile, "app_profiler/stackprof_profile"
   autoload :VernierProfile, "app_profiler/vernier_profile"
   autoload(:Backend, "app_profiler/backend")
   autoload(:Server, "app_profiler/server")
-  autoload(:Sampler, "app_profiler/sampler")
-  autoload(:ProfileId, "app_profiler/profile_id")
+
+  require "app_profiler/parameters"
+  require "app_profiler/request_parameters"
+  require "app_profiler/profile_id"
+  require "app_profiler/sampler"
 
   mattr_accessor :logger, default: Logger.new($stdout)
   mattr_accessor :root
@@ -58,6 +60,7 @@ module AppProfiler
   mattr_accessor :autoredirect, default: false
   mattr_reader   :profile_header, default: "X-Profile"
   mattr_accessor :profile_async_header, default: "X-Profile-Async"
+  mattr_accessor :profile_param, default: "profile"
   mattr_accessor :context, default: nil
   mattr_reader   :profile_url_formatter, default: DefaultProfileFormatter
   mattr_accessor :storage, default: Storage::FileStorage
@@ -89,9 +92,13 @@ module AppProfiler
       end
       profiler.run(*args, **kwargs, &block)
     rescue BackendError => e
-      logger.error(
-        "[AppProfiler.run] exception #{e} configuring backend #{backend}: #{e.message}",
-      )
+      if ActiveSupport.respond_to?(:error_reporter)
+        ActiveSupport.error_reporter.report(e, context: { app_profiler: { backend: backend } })
+      else
+        logger.error(
+          "[AppProfiler.run] exception #{e} configuring backend #{backend}: #{e.message}",
+        )
+      end
       yield
     ensure
       self.backend = original_backend if backend
@@ -158,9 +165,13 @@ module AppProfiler
 
       @profile_sampler_enabled.is_a?(Proc) ? @profile_sampler_enabled.call : @profile_sampler_enabled
     rescue => e
-      logger.error(
-        "[AppProfiler.profile_sampler_enabled] exception: #{e}, message: #{e.message}",
-      )
+      if ActiveSupport.respond_to?(:error_reporter)
+        ActiveSupport.error_reporter.report(e)
+      else
+        logger.error(
+          "[AppProfiler.profile_sampler_enabled] exception: #{e}, message: #{e.message}",
+        )
+      end
       false
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: app_profiler
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Gannon McGibbon
@@ -12,7 +12,7 @@ authors:
 - Scott Francis
 bindir: bin
 cert_chain: []
-date: 2025-02-24 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -201,7 +201,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.3
+rubygems_version: 4.0.6
 specification_version: 4
 summary: Collect performance profiles for your Rails application.
 test_files: []