app_profiler 0.6.0 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eded37ca4bd7b77dcf4bd68a1814eb0900ca1ba7e2cfdb0c5d2490d68f61ba7d
-  data.tar.gz: e69bf61f50c4d7baf3a8d9b748fb5e295d0901fb7d0d0903d3689e230e5526df
+  metadata.gz: 9fa114b3d31e066d94da48641fa21f8fbbd718a5754fe5c95ae2f5625eed30b7
+  data.tar.gz: 5fc004366996324b172ea6c174930583dcd5d6c39be7ff8071fcc1324731336b
 SHA512:
-  metadata.gz: '098504043e35bf17909c781047ba83eacdff0dffd7115e9f3910d5d3e3664ef58ee726ced428c6f5fb4750b06eb0193755d096f93d41692074e2eeb049e7f23a'
-  data.tar.gz: 7ea8cbe0cc7d6adfd7b386dae7386f7b08179f112d52e4280cce305e296bb70efe6baab5da983510832b4c511faa3ce9a789a377bd94a9816994c23122cec9ff
+  metadata.gz: a1ba1717adb236ee9589ac25f1c685cfb7dfd1c05f8d0a53e502d883b15e8e5233bf80897e0312b716374bc2256aa98e38afedcbfbd017c818046e4a0551820a
+  data.tar.gz: 89a2bee08a75a8e91c9f6f950fb2c7b27817ee2aac437b8386e7d0575a1025aeb812f61af6982bfca68158d059320af8461ed51b2c269ad0577407eed8e26107

data/lib/app_profiler/backend/base_backend.rb

@@ -47,6 +47,10 @@ module AppProfiler
           @run_lock ||= Mutex.new
         end
 
+        def locked?
+          run_lock.locked?
+        end
+
         def name
           raise NotImplementedError
         end

data/lib/app_profiler/middleware/upload_action.rb

@@ -24,15 +24,15 @@ module AppProfiler
         def append_headers(response, upload:, autoredirect:)
           return unless upload
 
-          response[1][profile_header]      = AppProfiler.profile_url(upload)
+          location = AppProfiler.profile_url(upload)
+          response[1][profile_header] = location if location
           response[1][profile_data_header] = profile_data_url(upload)
 
           return unless autoredirect
 
           # Automatically redirect to profile if autoredirect is true.
-          location = AppProfiler.profile_url(upload)
           if response[0].to_i < 500 && location
-            response[1]["Location"] = location
+            response[1]["location"] = location
             response[0] = 303
           end
         end

data/lib/app_profiler/railtie.rb

@@ -26,7 +26,8 @@ module AppProfiler
       AppProfiler.server.transport = app.config.app_profiler.server_transport || default_appprofiler_transport
       AppProfiler.server.port = app.config.app_profiler.server_port || 0
       AppProfiler.server.duration = app.config.app_profiler.server_duration || 30
-      AppProfiler.server.cors = app.config.app_profiler.server_cors || true
+      server_cors = app.config.app_profiler.server_cors
+      AppProfiler.server.cors = server_cors.nil? ? true : server_cors
       AppProfiler.server.cors_host = app.config.app_profiler.server_cors_host || "*"
       AppProfiler.autoredirect = app.config.app_profiler.autoredirect || false
       AppProfiler.speedscope_host = app.config.app_profiler.speedscope_host || ENV.fetch(

data/lib/app_profiler/request_parameters.rb

@@ -31,9 +31,7 @@ module AppProfiler
         return false
       end
 
-      return false if backend != AppProfiler::Backend::StackprofBackend.name && !AppProfiler.vernier_supported?
-
-      if AppProfiler.vernier_supported? && backend == AppProfiler::VernierProfile::BACKEND_NAME &&
+      if backend == AppProfiler::VernierProfile::BACKEND_NAME &&
           !AppProfiler::Backend::VernierBackend::AVAILABLE_MODES.include?(mode.to_sym)
         AppProfiler.logger.info("[AppProfiler] unsupported profiling mode=#{mode} for backend #{backend}")
         return false

data/lib/app_profiler/vernier_profile.rb

@@ -7,7 +7,7 @@ module AppProfiler
 
     class << self
       def backend_name
-        # cannot reference Backend::VernierBackend because of different ruby versions we have to support
+        # Keep metadata access independent of the optional Vernier gem.
         BACKEND_NAME.to_s
       end
     end

data/lib/app_profiler/version.rb

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

data/lib/app_profiler/viewer/base_middleware.rb

@@ -86,7 +86,7 @@ module AppProfiler
       def render(html)
         [
           200,
-          { "Content-Type" => "text/html" },
+          { "content-type" => "text/html" },
           [
             +<<~HTML,
               <!doctype html>

data/lib/app_profiler/viewer/firefox_remote_viewer/middleware.rb

@@ -11,7 +11,7 @@ module AppProfiler
 
         def initialize(app)
           super
-          @firefox_profiler = Rack::File.new(
+          @firefox_profiler = Rack::Files.new(
             File.join(AppProfiler.root, "node_modules/firefox-profiler/dist"),
           )
         end
@@ -40,9 +40,9 @@ module AppProfiler
             source = "#{proto}://#{host}/app_profiler/firefox/#{path}"
             target = "/from-url/#{CGI.escape(source)}"
 
-            [302, { "Location" => target }, [""]]
+            [302, { "location" => target }, [""]]
           else
-            env[Rack::PATH_INFO] = path.delete_prefix("/app_profiler")
+            env[Rack::PATH_INFO] = "/#{path.delete_prefix("/")}"
             firefox_profiler.call(env)
           end
         end
@@ -50,7 +50,7 @@ module AppProfiler
         def from(env, path)
           setup_yarn unless yarn_setup
           index = File.read(File.join(AppProfiler.root, "node_modules/firefox-profiler/dist/index.html"))
-          [200, { "Content-Type" => "text/html" }, [index]]
+          [200, { "content-type" => "text/html" }, [index]]
         end
 
         def show(_env, name)
@@ -58,7 +58,7 @@ module AppProfiler
             id(file) == name
           end || raise(ArgumentError)
 
-          [200, { "Content-Type" => "application/json" }, [profile.read]]
+          [200, { "content-type" => "application/json" }, [profile.read]]
         end
       end
     end

data/lib/app_profiler/viewer/firefox_remote_viewer.rb

@@ -22,7 +22,7 @@ module AppProfiler
         id = Middleware.id(@profile.file)
 
         if response && response[0].to_i < 500
-          response[1]["Location"] = "/app_profiler/#{NAME}/viewer/#{id}"
+          response[1]["location"] = "/app_profiler/#{NAME}/viewer/#{id}"
           response[0] = 303
         else
           AppProfiler.logger.info("[Profiler] Profile available at /app_profiler/#{id}\n")

data/lib/app_profiler/viewer/speedscope_remote_viewer/middleware.rb

@@ -11,7 +11,7 @@ module AppProfiler
 
         def initialize(app)
           super
-          @speedscope = Rack::File.new(
+          @speedscope = Rack::Files.new(
             File.join(AppProfiler.root, "node_modules/speedscope/dist/release"),
           )
         end
@@ -32,7 +32,7 @@ module AppProfiler
         def viewer(env, path)
           setup_yarn unless yarn_setup
 
-          env[Rack::PATH_INFO] = path.delete_prefix("/app_profiler/speedscope")
+          env[Rack::PATH_INFO] = "/#{path.delete_prefix("/")}"
           speedscope.call(env)
         end
 
@@ -41,7 +41,7 @@ module AppProfiler
             id(file) == name
           end || raise(ArgumentError)
 
-          [200, { "Content-Type" => "application/json" }, [profile.read]]
+          [200, { "content-type" => "application/json" }, [profile.read]]
         end
       end
     end

data/lib/app_profiler/viewer/speedscope_remote_viewer.rb

@@ -29,7 +29,7 @@ module AppProfiler
         id = Middleware.id(@profile.file)
 
         if response && response[0].to_i < 500
-          response[1]["Location"] = "/app_profiler/#{NAME}/viewer/#{id}"
+          response[1]["location"] = "/app_profiler/#{NAME}/viewer/#{id}"
           response[0] = 303
         else
           AppProfiler.logger.info("[Profiler] Profile available at /app_profiler/#{id}\n")

data/lib/app_profiler/yarn/with_firefox_profiler.rb

@@ -42,7 +42,7 @@ module AppProfiler
           package_json = JSON.parse(package_contents)
           package_json["name"] ||= "firefox-profiler"
           package_json["version"] ||= "0.0.1"
-          File.write("firefox-profiler/package.json", package_json.to_json)
+          File.write("firefox-profiler/package.json", JSON.generate(package_json))
         end
         yarn("--cwd", "#{dir}/firefox-profiler")
 

data/lib/app_profiler.rb

@@ -58,8 +58,8 @@ module AppProfiler
 
   mattr_accessor :speedscope_host, default: "https://speedscope.app"
   mattr_accessor :autoredirect, default: false
-  mattr_reader   :profile_header, default: "X-Profile"
-  mattr_accessor :profile_async_header, default: "X-Profile-Async"
+  mattr_reader   :profile_header, default: "x-profile"
+  mattr_reader :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
@@ -92,14 +92,9 @@ module AppProfiler
       end
       profiler.run(*args, **kwargs, &block)
     rescue BackendError => e
-      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
+      ActiveSupport.error_reporter.report(e, context: { app_profiler: { backend: backend } })
       yield
+      nil # no profile result
     ensure
       self.backend = original_backend if backend
       ProfileId::Current.reset
@@ -126,7 +121,7 @@ module AppProfiler
     def backend=(new_backend)
       return if (new_profiler_backend = backend_for(new_backend)) == profiler_backend
 
-      if running?
+      if profiler_backend.locked?
         raise BackendError,
           "cannot change backend to #{new_backend} while #{backend} backend is running"
       end
@@ -165,13 +160,7 @@ module AppProfiler
 
       @profile_sampler_enabled.is_a?(Proc) ? @profile_sampler_enabled.call : @profile_sampler_enabled
     rescue => e
-      if ActiveSupport.respond_to?(:error_reporter)
-        ActiveSupport.error_reporter.report(e)
-      else
-        logger.error(
-          "[AppProfiler.profile_sampler_enabled] exception: #{e}, message: #{e.message}",
-        )
-      end
+      ActiveSupport.error_reporter.report(e)
       false
     end
 
@@ -185,8 +174,7 @@ module AppProfiler
     end
 
     def backend_for(backend_name)
-      if vernier_supported? &&
-          backend_name&.to_sym == AppProfiler::VernierProfile::BACKEND_NAME
+      if backend_name&.to_sym == AppProfiler::VernierProfile::BACKEND_NAME
         AppProfiler::Backend::VernierBackend
       elsif backend_name&.to_sym == AppProfiler::Backend::StackprofBackend.name
         AppProfiler::Backend::StackprofBackend
@@ -199,22 +187,22 @@ module AppProfiler
       profiler_backend.name
     end
 
-    def vernier_supported?
-      RUBY_VERSION >= "3.2.1" && defined?(AppProfiler::VernierProfile::BACKEND_NAME)
-    end
-
     def profile_header=(profile_header)
-      @@profile_header = profile_header # rubocop:disable Style/ClassVars
+      @@profile_header = profile_header.downcase # rubocop:disable Style/ClassVars
       @@request_profile_header = nil    # rubocop:disable Style/ClassVars
       @@profile_data_header = nil       # rubocop:disable Style/ClassVars
     end
 
+    def profile_async_header=(profile_async_header)
+      @@profile_async_header = profile_async_header.downcase # rubocop:disable Style/ClassVars
+    end
+
     def request_profile_header
       @@request_profile_header ||= profile_header.upcase.tr("-", "_").prepend("HTTP_") # rubocop:disable Style/ClassVars
     end
 
     def profile_data_header
-      @@profile_data_header ||= profile_header.dup << "-Data" # rubocop:disable Style/ClassVars
+      @@profile_data_header ||= profile_header.dup << "-data" # rubocop:disable Style/ClassVars
     end
 
     def profile_url_formatter=(block)
@@ -268,7 +256,7 @@ module AppProfiler
     end
 
     def clear
-      profiler.stop if running?
+      profiler.stop if profiler_backend.locked?
       @profiler = nil
       @profiler_backend = nil
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: app_profiler
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.1
 platform: ruby
 authors:
 - Gannon McGibbon
@@ -20,14 +20,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '7.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: base64
   requirement: !ruby/object:Gem::Requirement
@@ -56,20 +56,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: mutex_m
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
@@ -116,16 +102,16 @@ dependencies:
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '6'
 - !ruby/object:Gem::Dependency
   name: minitest-stub-const
   requirement: !ruby/object:Gem::Requirement
@@ -236,14 +222,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 3.2.1
+      version: '3.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 4.0.23
 specification_version: 4
 summary: Collect performance profiles for your Rails application.
 test_files: []