callstacking-rails 0.1.12 → 0.1.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 643536b00e33ea23bde289b22362e6aa38456fe24ee7dad49086a2b62c0eba3d
-  data.tar.gz: d49b09ee59ef7f64d4c5c8dbd68c003d0ef2bab4902870496c7affe7a653fa8f
+  metadata.gz: 5d44334293141ed52816f662a764b60a880c6a8942ad0803dc365b5eaa7bcec6
+  data.tar.gz: 979b68d010c25055b21ba482165a64b850f6b9eefb1cdb1445192cd6e1afbc70
 SHA512:
-  metadata.gz: ab33b68a03c006d14ff14f77f9096f762e19b60fbb2601a7eb4cda71a20deb4c2cd7b34410451bf44960303ce52ebb72489739aba7c7cc1ddec63ec10393b451
-  data.tar.gz: 7433ed646c676d6bbee5ee78e52752dfb310abf8b452b44f0c41299cfb3083462564689c418fe8e0aa7eb968f25351f9c713cd0d70048938ea40e5c6b331a125
+  metadata.gz: 997f19b2eda526ea52667967af3712605a3e7caf6daa1ceb40957e905d68dcad62cdfac50f02094838bd75b5108f1788d166600d997796d4de61cb4dcd0107fb
+  data.tar.gz: 45095a4346af6d99208e0e089f492d29d9d21196a13ef63570ad2d7403c6dd1ea3db2412db99aa8d3d0a0009144f0873cb9312ac092e6969afa49dcf4104d1ea

data/lib/callstacking/rails/engine.rb

@@ -16,7 +16,7 @@ module Callstacking
       include Settings
       isolate_namespace Callstacking::Rails
 
-      read_settings
+      read_settingsx
 
       spans = Spans.new
       trace = Trace.new(spans)

data/lib/callstacking/rails/instrument.rb

@@ -28,25 +28,52 @@ module Callstacking
                     tmp_module.instance_methods.include?(method_name) ||
                     tmp_module.singleton_methods.include?(method_name)
 
-        tmp_module.define_method(method_name) do |*args, **kwargs, &block|
-          method_name = __method__
+        new_method = nil
+        if RUBY_VERSION < "2.7.8"
+          new_method = tmp_module.define_method(method_name) do |*args, &block|
+            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(':')
+            p, l = caller.find { |c| c.to_s =~ /#{::Rails.root.to_s}/}&.split(':')
 
-          spans = tmp_module.instance_variable_get(:@spans)
-          klass = tmp_module.instance_variable_get(:@klass)
+            spans = tmp_module.instance_variable_get(:@spans)
+            klass = tmp_module.instance_variable_get(:@klass)
 
-          arguments = Callstacking::Rails::Instrument.arguments_for(method(__method__).super_method, args)
+            arguments = Callstacking::Rails::Instrument.arguments_for(method(__method__).super_method, args)
 
-          spans.call_entry(klass, method_name, arguments, p || path, l || line_no)
-          return_val = super(*args, **kwargs, &block)
-          spans.call_return(klass, method_name, p || path, l || line_no, return_val)
+            spans.call_entry(klass, method_name, arguments, p || path, l || line_no)
+            return_val = super(*args, &block)
+            spans.call_return(klass, method_name, p || path, l || line_no, return_val)
+
+            return_val
+          end
+          new_method.ruby2_keywords if new_method.respond_to?(:ruby2_keywords)
+        else
+          new_method = tmp_module.define_method(method_name) do |*args, **kwargs, &block|
+            method_name = __method__
+
+            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(':')
+
+            spans = tmp_module.instance_variable_get(:@spans)
+            klass = tmp_module.instance_variable_get(:@klass)
+
+            arguments = Callstacking::Rails::Instrument.arguments_for(method(__method__).super_method, args)
+
+            spans.call_entry(klass, method_name, arguments, p || path, l || line_no)
+            return_val = super(*args, **kwargs, &block)
+            spans.call_return(klass, method_name, p || path, l || line_no, return_val)
+
+            return_val
+          end
 
-          return_val
         end
+
+        new_method
       end
 
       def find_or_initialize_module

data/lib/callstacking/rails/settings.rb

@@ -32,6 +32,7 @@ module Callstacking
 
       def enabled?
         return false if settings.nil?
+        return false if ENV['CALLSTACKING_ENABLED'] == 'false'
         settings[:enabled]
       end
 

data/lib/callstacking/rails/setup.rb

@@ -2,6 +2,7 @@ require 'yaml'
 require "callstacking/rails/settings"
 require "callstacking/rails/client/authenticate"
 require "callstacking/rails/env"
+require 'io/console'
 
 module Callstacking
   module Rails
@@ -24,7 +25,7 @@ module Callstacking
         puts
         
         email      = prompt("Enter email:")
-        password   = prompt("Enter password:")
+        password   = prompt("Enter password:", echo: false)
 
         url = if Callstacking::Rails::Env.production? && ENV['CHECKPOINT_RAILS_LOCAL_TEST'].nil?
                 PRODUCTION_URL
@@ -50,9 +51,14 @@ module Callstacking
         write_settings(complete_settings.merge(props))
       end
 
-      def prompt(label)
+      def prompt(label, echo: true)
         puts label
-        value = STDIN.gets.chomp
+
+        value = if echo
+                  STDIN.gets.chomp
+                else
+                  STDIN.noecho(&:gets).chomp
+                end
         puts
 
         return nil if value == ''

data/lib/callstacking/rails/trace.rb

@@ -30,7 +30,7 @@ module Callstacking
         ActiveSupport::Notifications.subscribe("start_processing.action_controller") do |name, start, finish, id, payload|
           trace_id, max_trace_entries = start_request(payload[:request]&.request_id, payload[:method], payload[:controller],
                                                       payload[:action], payload[:format], ::Rails.root,
-                                                      payload[:request]&.original_url,
+                                                      payload[:request]&.original_url || payload[:path],
                                                       payload[:headers], payload[:params])
         end
 
@@ -45,8 +45,8 @@ module Callstacking
         ActiveSupport::Notifications.subscribe("process_action.action_controller") do |name, start, finish, id, payload|
           complete_request(payload[:method], payload[:controller],
                            payload[:action], payload[:format],
-                           payload[:request]&.original_url,
-                           trace_id, max_trace_entries)
+                           payload[:request]&.original_url || payload[:path],
+                           trace_id, @traces, max_trace_entries)
         end
       end
 
@@ -123,7 +123,7 @@ module Callstacking
         end
       end
       def start_request(request_id, method, controller, action, format, path, original_url, headers, params)
-        return if do_not_track_request?(original_url)
+        return if do_not_track_request?(original_url, format)
         
         request_id = request_id || SecureRandom.uuid
         Callstacking::Rails::Trace.current_request_id = request_id
@@ -140,23 +140,25 @@ module Callstacking
         return trace_id, max_trace_entries
       end
 
-      def complete_request(method, controller, action, format, original_url, trace_id, max_trace_entries)
-        if do_not_track_request?(original_url)
+      def complete_request(method, controller, action, format, original_url, trace_id, traces, max_trace_entries)
+        if do_not_track_request?(original_url, format)
           traces.clear
           return
         end
 
         create_message(completed_request_message(method, controller, action, format),
-                       spans.increment_order_num, @traces)
+                       spans.increment_order_num, traces)
 
-        send_traces!(trace_id, @traces[0..max_trace_entries])
+        send_traces!(trace_id, traces[0..max_trace_entries])
       end
 
       def track_request?(url)
         !(track_request?(url))
       end
 
-      def do_not_track_request?(url)
+      def do_not_track_request?(url, format)
+        return false if format =~ /(html|json)/i
+        
         url =~ /(\/stylesheets\/|\/javascripts\/|\/css\/|\/js\/|\.js|\.css)/i
       end
 

data/lib/callstacking/rails/version.rb

@@ -1,5 +1,5 @@
 module Callstacking
   module Rails
-    VERSION = "0.1.12"
+    VERSION = "0.1.14"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: callstacking-rails
 version: !ruby/object:Gem::Version
-  version: 0.1.12
+  version: 0.1.14
 platform: ruby
 authors:
 - Jim Jones
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-01-31 00:00:00.000000000 Z
+date: 2023-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -28,16 +28,16 @@ dependencies:
   name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.10.0
+        version: 1.10.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.10.0
+        version: 1.10.3
 - !ruby/object:Gem::Dependency
   name: faraday-follow_redirects
   requirement: !ruby/object:Gem::Requirement