builder_apm 0.5.10 → 0.5.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 76836856ebaf118f3b50eb04f1e03a9dcdcb1d3779a197c178bce179e895bbc4
-  data.tar.gz: 1590eb4a9204c231cdeb32683745d97c84656ab655641eb84eea4ae30de8d96d
+  metadata.gz: 2f9d7c7af0913b72c61b524ecc012de2bf7184cbeb0feeb42dbf61e6b7fb71d6
+  data.tar.gz: c4337273e42bec6234f1cb8c2987ab82a5dddb28ee150e179b3162d5d53ebdf5
 SHA512:
-  metadata.gz: 06fce8a116b3a6edee614ed982b5e83e648c6bfdf4a2166336a182898f3acb04e290639bc982702cea0a34cefa8aefe8bc86ca29156f9714d18bc7c814d88313
-  data.tar.gz: 7d9e190154d9b178f3eb7ea921fe0918e047ea99cbb6a4bcd7a424b5e88a8db594afeab1f74377f1cd25fe116c91c5b8c3b62a98ba38f758a9488725d8f07c0b
+  metadata.gz: 70c6b5cbe90f474260100e7d7c563a555f73cfe2f1f8072462f6e6934be7990ebf6c486b02d2b820b879c4fb65828abe86b0cc66b9dcc95348417a84ea051dbe
+  data.tar.gz: 3b8b18b6c41bacc1b3cfd0a0a6ca2f691f097d2ce495bc99583b0582a61c28d654443235982360e3912cf256448de817c8de6a679f1009b1b13a91cddd19912b

data/lib/builder_apm/engine.rb

@@ -5,6 +5,7 @@ module BuilderApm
     initializer 'builder_apm.start' do |app|
       BuilderApm::Controllers::Instrumenter.new.start
       BuilderApm::Models::Instrumenter.start if BuilderApm.configuration.enable_active_record_profiler
+      BuilderApm::Methods::Instrumenter.new.start if BuilderApm.configuration.enable_methods_profiler
     end
 
     config.after_initialize do

data/lib/builder_apm/methods/instrumenter.rb

@@ -10,7 +10,7 @@ module BuilderApm
       def start
         @trace = setup_trace
         @trace.enable
-        self
+        Thread.current[:trace_point] = @trace
       end
 
       def stop
@@ -21,7 +21,7 @@ module BuilderApm
         me = self
         TracePoint.new(:call, :return, :end, :raise) do |tp|
           starttime = Time.now.to_f * 1000
-          me.process_trace_point(tp) if me.valid_trace_point?(tp)
+          me.process_trace_point(tp) if Thread.current[:request_id] && me.valid_trace_point?(tp)
           duration = (Time.now.to_f * 1000) - starttime
           
           Thread.current[:method_tracing] = (Thread.current[:method_tracing] ||= 0) + duration
@@ -30,7 +30,6 @@ module BuilderApm
 
       def valid_trace_point?(tp)
         return false unless tp.path.start_with?(@root_path)
-        return false unless Thread.current[:request_id]
 
         start_controller = Thread.current[:stack]&.first
         start_controller && "#{tp.defined_class}##{tp.method_id}" != start_controller[:method]
@@ -40,7 +39,7 @@ module BuilderApm
       end
 
       def process_trace_point(tp)
-        if tp.event == :call || tp.event == :b_call || tp.event == :c_call
+        if tp.event == :call
           method_id = "#{tp.defined_class}##{tp.method_id}"
           (@call_times[method_id]||= []) << Process.clock_gettime(Process::CLOCK_MONOTONIC)
           caller_info = caller_locations(4,1).first

data/lib/builder_apm/middleware/timing.rb

@@ -15,12 +15,12 @@ module BuilderApm
         Thread.current[:db_runtime] = 0
         Thread.current[:method_tracing] = 0
         Thread.current[:db_tracing] = 0
-        @trace = BuilderApm::Methods::Instrumenter.new.start if BuilderApm.configuration.enable_methods_profiler
+        Thread.current[:trace_point].disable if Thread.current[:trace_point]
+        Thread.current[:trace_point] = nil
     
         @status, @headers, @response = @app.call(env)
 
       ensure
-        @trace.stop if BuilderApm.configuration.enable_methods_profiler
         end_time = Time.now.to_f * 1000
         handle_timing(start_time, end_time, request_id)
 

data/lib/builder_apm/version.rb

@@ -1,3 +1,3 @@
 module BuilderApm
-  VERSION = "0.5.10"
+  VERSION = "0.5.12"
 end

data/lib/generators/builder_apm/templates/builder_apm_config.rb

@@ -2,8 +2,6 @@ BuilderApm.configure do |config|
   config.redis_url = 'redis://localhost:6379/0'   # optional - defaults to 'redis://localhost:6379/0'
   config.api_key = ENV["OPENAI_API_KEY"]          #required only if you want to use Ai Doctor
   config.api = "Bravo"                            # optional - default to 'Bravo' - "Bravo" - internal Ai / "OpenAi" - use openai ChatGPT
-  config.gems_to_track = []                       # optional - default to empty array []
-  config.enable_controller_profiler = true        # optional - defaults to true
   config.enable_active_record_profiler = true     # optional - defaults to true
   config.enable_methods_profiler = true           # optional - defaults to true
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: builder_apm
 version: !ruby/object:Gem::Version
-  version: 0.5.10
+  version: 0.5.12
 platform: ruby
 authors:
 - Paul Ketelle
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-09-26 00:00:00.000000000 Z
+date: 2023-10-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -130,7 +130,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.1.0
 signing_key:
 specification_version: 4
 summary: Write a short summary, because RubyGems requires one.