Filter native method path calls. Verson bump.

callstacking · Nov 10, 2023 · ddf099d · ddf099d
1 parent 46a4b56
commit ddf099d
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 5 deletions.
diff --git a/lib/callstacking/rails/helpers/instrument_helper.rb b/lib/callstacking/rails/helpers/instrument_helper.rb
@@ -5,16 +5,36 @@ module InstrumentHelper
         extend ActiveSupport::Concern
         def callstacking_setup
           exception = nil
+          @last_callstacking_sample = TIme.utc.now
           Callstacking::Rails::Engine.start_tracing(self)
 
           yield
         rescue Exception => e
+          @last_callstacking_exception = Time.utc.now
           exception = e
           raise e
         ensure
           Callstacking::Rails::Engine.stop_tracing(self, exception)
         end
       end
+
+      def callstcking_sample_trace?
+        if @last_callstacking_exception.present? && @last_callstacking_exception < 1.minute.ago
+          @last_callstacking_exception = nil
+          return true
+        end
+
+        false
+      end
+
+      def callstacking_followup_exception_trace?
+        if @last_callstacking_sample.present? && @last_callstacking_sample < 1.hour.ago
+          @last_callstacking_exception = nil
+          return true
+        end
+
+        false
+      end
     end
   end
 end
diff --git a/lib/callstacking/rails/instrument.rb b/lib/callstacking/rails/instrument.rb
@@ -39,8 +39,8 @@ def instrument_method(klass, method_name, application_level: true)
 
             method_name = __method__
 
-            path = method(__method__).super_method.source_location.first
-            line_no = method(__method__).super_method.source_location.last
+            path = method(__method__).super_method.source_location&.first || ''
+            line_no = method(__method__).super_method.source_location&.last || ''
 
             p, l = caller.find { |c| c.to_s =~ /#{::Rails.root.to_s}/}&.split(':')
 
@@ -64,8 +64,8 @@ def instrument_method(klass, method_name, application_level: true)
 
             method_name = __method__
 
-            path = method(__method__).super_method.source_location.first
-            line_no = method(__method__).super_method.source_location.last
+            path = method(__method__).super_method.source_location&.first || ''
+            line_no = method(__method__).super_method.source_location&.last || ''
 
             p, l = caller.find { |c| c.to_s =~ /#{::Rails.root.to_s}/}&.split(':')
 

diff --git a/lib/callstacking/rails/version.rb b/lib/callstacking/rails/version.rb
@@ -1,5 +1,5 @@
 module Callstacking
   module Rails
-    VERSION = "0.1.36"
+    VERSION = "0.1.37"
   end
 end