newrelic_rpm 8.2.0 → 8.3.0

data/lib/new_relic/agent/instrumentation/httprb/prepend.rb

@@ -8,7 +8,7 @@ module NewRelic::Agent::Instrumentation
       include NewRelic::Agent::Instrumentation::HTTPrb
 
       def perform(request, options)
-        with_tracing(request) { super }        
+        with_tracing(request) { super }
       end
     end
   end

data/lib/new_relic/agent/instrumentation/ignore_actions.rb

@@ -15,9 +15,9 @@ module NewRelic
 
             should_filter = case ignore_actions
             when Hash
-              only_actions   = Array(ignore_actions[:only])
+              only_actions = Array(ignore_actions[:only])
               except_actions = Array(ignore_actions[:except])
-              action_name    = action_name.to_sym
+              action_name = action_name.to_sym
 
               only_actions.include?(action_name) || (!except_actions.empty? && !except_actions.include?(action_name))
             else
@@ -38,4 +38,3 @@ module NewRelic
     end
   end
 end
-

data/lib/new_relic/agent/instrumentation/logger/instrumentation.rb

@@ -31,7 +31,6 @@ module NewRelic
           @size_metrics[severity] ||= "Logging/size/#{severity}".freeze
         end
 
-
         def format_message_with_tracing(severity, datetime, progname, msg)
           formatted_message = yield
           return formatted_message if skip_instrumenting?

data/lib/new_relic/agent/instrumentation/logger.rb

@@ -12,7 +12,7 @@ DependencyDetection.defer do
   depends_on { defined?(::Logger) }
 
   executes do
-    ::NewRelic::Agent.logger.info  "Installing Logger instrumentation"
+    ::NewRelic::Agent.logger.info "Installing Logger instrumentation"
   end
 
   executes do

data/lib/new_relic/agent/instrumentation/memcache/dalli.rb

@@ -9,6 +9,7 @@ module NewRelic
       module Memcache
         module Dalli
           extend Helper
+
           module_function
 
           def instrument!
@@ -69,6 +70,7 @@ module NewRelic
         module DalliCAS
           extend Dalli
           extend Helper
+
           module_function
 
           def should_instrument?
@@ -84,4 +86,3 @@ module NewRelic
     end
   end
 end
-

data/lib/new_relic/agent/instrumentation/memcache/helper.rb

@@ -58,4 +58,3 @@ module NewRelic::Agent::Instrumentation
     end
   end
 end
-

data/lib/new_relic/agent/instrumentation/memcache/prepend.rb

@@ -7,6 +7,7 @@ module NewRelic::Agent::Instrumentation
   module Memcache
     module Prepend
       extend Helper
+
       module_function
 
       def client_prepender client_class

data/lib/new_relic/agent/instrumentation/middleware_proxy.rb

@@ -15,7 +15,7 @@ module NewRelic
       class MiddlewareProxy
         include MiddlewareTracing
 
-        ANONYMOUS_CLASS   = "AnonymousClass".freeze
+        ANONYMOUS_CLASS = "AnonymousClass".freeze
         OBJECT_CLASS_NAME = "Object".freeze
 
         # This class is used to wrap classes that are passed to
@@ -50,7 +50,7 @@ module NewRelic
           )
         end
 
-        def self.wrap(target, is_app=false)
+        def self.wrap(target, is_app = false)
           if needs_wrapping?(target)
             self.new(target, is_app)
           else
@@ -60,13 +60,13 @@ module NewRelic
 
         attr_reader :target, :category, :transaction_options
 
-        def initialize(target, is_app=false)
-          @target            = target
-          @is_app            = is_app
-          @category          = determine_category
+        def initialize(target, is_app = false)
+          @target = target
+          @is_app = is_app
+          @category = determine_category
           @target_class_name = determine_class_name
-          @transaction_name  = "#{determine_prefix}#{@target_class_name}/call"
-          @transaction_options  = {
+          @transaction_name = "#{determine_prefix}#{@target_class_name}/call"
+          @transaction_options = {
             :transaction_name => @transaction_name
           }
         end
@@ -93,7 +93,7 @@ module NewRelic
 
           name = clazz.name
           name = clazz.superclass.name if name.nil? || name == ""
-          name = ANONYMOUS_CLASS       if name.nil? || name == OBJECT_CLASS_NAME
+          name = ANONYMOUS_CLASS if name.nil? || name == OBJECT_CLASS_NAME
           name
         end
 
@@ -104,7 +104,6 @@ module NewRelic
             @target.class
           end
         end
-
       end
     end
   end

data/lib/new_relic/agent/instrumentation/middleware_tracing.rb

@@ -27,10 +27,9 @@ module NewRelic
   module Agent
     module Instrumentation
       module MiddlewareTracing
-
         TXN_STARTED_KEY = 'newrelic.transaction_started'
-        CONTENT_TYPE    = 'Content-Type'
-        CONTENT_LENGTH  = 'Content-Length'
+        CONTENT_TYPE = 'Content-Type'
+        CONTENT_LENGTH = 'Content-Length'
 
         def _nr_has_middleware_tracing
           true
@@ -71,7 +70,7 @@ module NewRelic
           if result.is_a?(Array) && state.current_transaction
             _, headers, _ = result
             length = headers[CONTENT_LENGTH]
-            length = length.reduce(0){|sum, h| sum + h.to_i} if length.is_a?(Array)
+            length = length.reduce(0) { |sum, h| sum + h.to_i } if length.is_a?(Array)
             state.current_transaction.response_content_length = length
           end
         end
@@ -98,7 +97,7 @@ module NewRelic
 
             events.notify(:before_call, env) if first_middleware
 
-            result = (target == self) ? traced_call(env) : target.call(env)
+            result = target == self ? traced_call(env) : target.call(env)
 
             if first_middleware
               capture_response_attributes(state, result)

data/lib/new_relic/agent/instrumentation/mongodb_command_subscriber.rb

@@ -19,11 +19,11 @@ module NewRelic
           end
         end
 
-        ERROR_KEYS = %w{ writeErrors writeConcernError writeConcernErrors }.freeze
+        ERROR_KEYS = %w[ writeErrors writeConcernError writeConcernErrors ].freeze
 
         def error_key_present?(event)
           if reply = event.reply
-            ERROR_KEYS.detect{ |key| reply[key] }
+            ERROR_KEYS.detect { |key| reply[key] }
           end
         rescue
           false
@@ -35,7 +35,7 @@ module NewRelic
             segment = segments.delete(event.operation_id)
             return unless segment
 
-            # operations that succeed but have errors return CommandSucceeded 
+            # operations that succeed but have errors return CommandSucceeded
             # with an error_key that is populated with error specfics
             if error_key = error_key_present?(event)
               # taking the last error as there can potentially be many
@@ -76,7 +76,7 @@ module NewRelic
         def operation(command_name)
           # from 2.0 to 2.5, :findandmodify was the command_name
           if command_name == :findandmodify
-            :findAndModify 
+            :findAndModify
           else
             command_name
           end

data/lib/new_relic/agent/instrumentation/net_http/chain.rb

@@ -4,7 +4,6 @@
 
 require_relative 'instrumentation'
 
-
 module NewRelic::Agent::Instrumentation
   module NetHTTP
     module Chain
@@ -14,12 +13,12 @@ module NewRelic::Agent::Instrumentation
 
           def request_with_newrelic_trace(request, *args, &block)
             request_with_tracing(request) { request_without_newrelic_trace(request, *args, &block) }
-          end  
-    
+          end
+
           alias request_without_newrelic_trace request
           alias request request_with_newrelic_trace
         end
       end
     end
   end
-end
+end

data/lib/new_relic/agent/instrumentation/net_http/instrumentation.rb

@@ -2,8 +2,8 @@
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
 
-module NewRelic 
-  module Agent 
+module NewRelic
+  module Agent
     module Instrumentation
       module NetHTTP
         def request_with_tracing(request)
@@ -14,11 +14,11 @@ module NewRelic
             uri: wrapped_request.uri,
             procedure: wrapped_request.method
           )
-  
+
           begin
             response = nil
             segment.add_request_headers wrapped_request
-  
+
             # RUBY-1244 Disable further tracing in request to avoid double
             # counting if connection wasn't started (which calls request again).
             NewRelic::Agent.disable_all_tracing do
@@ -26,7 +26,7 @@ module NewRelic
                 yield
               end
             end
-  
+
             wrapped_response = NewRelic::Agent::HTTPClients::NetHTTPResponse.new response
             segment.process_response_headers wrapped_response
             response

data/lib/new_relic/agent/instrumentation/net_http/prepend.rb

@@ -4,13 +4,13 @@
 
 require_relative 'instrumentation'
 
-module NewRelic 
-  module Agent 
+module NewRelic
+  module Agent
     module Instrumentation
       module NetHTTP
         module Prepend
           include NewRelic::Agent::Instrumentation::NetHTTP
-          
+
           def request(request, *args, &block)
             request_with_tracing(request) { super }
           end
@@ -18,4 +18,4 @@ module NewRelic
       end
     end
   end
-end
+end

data/lib/new_relic/agent/instrumentation/net_http.rb

@@ -37,7 +37,7 @@ DependencyDetection.defer do
   executes do
     if use_prepend?
       prepend_instrument ::Net::HTTP, ::NewRelic::Agent::Instrumentation::NetHTTP::Prepend
-    else 
+    else
       chain_instrument ::NewRelic::Agent::Instrumentation::NetHTTP::Chain
     end
   end

data/lib/new_relic/agent/instrumentation/notifications_subscriber.rb

@@ -12,7 +12,7 @@ module NewRelic
         end
 
         def self.subscribed?
-          find_all_subscribers.find{|s| s.instance_variable_get(:@delegate).class == self }
+          find_all_subscribers.find { |s| s.instance_variable_get(:@delegate).class == self }
         end
 
         def self.find_all_subscribers
@@ -64,7 +64,7 @@ module NewRelic
         end
 
         def segment_stack
-          Thread.current[@queue_key] ||= Hash.new {|h,id| h[id] = [] }
+          Thread.current[@queue_key] ||= Hash.new { |h, id| h[id] = [] }
         end
 
         def state
@@ -94,7 +94,6 @@ module NewRelic
           end
         end
       end
-
     end
   end
 end

data/lib/new_relic/agent/instrumentation/padrino/instrumentation.rb

@@ -15,7 +15,8 @@ module NewRelic::Agent::Instrumentation
         txn_name = ::NewRelic::Agent::Instrumentation::Sinatra::TransactionNamer.transaction_name_for_route(env, request)
         unless txn_name.nil?
           ::NewRelic::Agent::Transaction.set_default_transaction_name(
-            "#{self.class.name}/#{txn_name}", :sinatra)
+            "#{self.class.name}/#{txn_name}", :sinatra
+          )
         end
       rescue => e
         ::NewRelic::Agent.logger.debug("Failed during invoke_route to set transaction name", e)
@@ -24,4 +25,4 @@ module NewRelic::Agent::Instrumentation
       yield
     end
   end
-end
+end

data/lib/new_relic/agent/instrumentation/padrino/prepend.rb

@@ -17,4 +17,4 @@ module NewRelic::Agent::Instrumentation
       end
     end
   end
-end
+end

data/lib/new_relic/agent/instrumentation/padrino.rb

@@ -30,8 +30,3 @@ DependencyDetection.defer do
     end
   end
 end
-
-
-
-
-

data/lib/new_relic/agent/instrumentation/queue_time.rb

@@ -9,10 +9,10 @@ module NewRelic
       # Record queue time metrics based on any of three headers
       # which can be set on the request.
       module QueueTime
-        REQUEST_START_HEADER    = 'HTTP_X_REQUEST_START'.freeze
-        QUEUE_START_HEADER      = 'HTTP_X_QUEUE_START'.freeze
+        REQUEST_START_HEADER = 'HTTP_X_REQUEST_START'.freeze
+        QUEUE_START_HEADER = 'HTTP_X_QUEUE_START'.freeze
         MIDDLEWARE_START_HEADER = 'HTTP_X_MIDDLEWARE_START'.freeze
-        ALL_QUEUE_METRIC        = 'WebFrontend/QueueTime'.freeze
+        ALL_QUEUE_METRIC = 'WebFrontend/QueueTime'.freeze
         # any timestamps before this are thrown out and the parser
         # will try again with a larger unit (2000/1/1 UTC)
         EARLIEST_ACCEPTABLE_TIME = Time.at(946684800).to_f
@@ -27,7 +27,7 @@ module NewRelic
 
         module_function
 
-        def parse_frontend_timestamp(headers, now=Process.clock_gettime(Process::CLOCK_REALTIME))
+        def parse_frontend_timestamp(headers, now = Process.clock_gettime(Process::CLOCK_REALTIME))
           earliest = nil
 
           CANDIDATE_HEADERS.each do |header|
@@ -52,7 +52,7 @@ module NewRelic
           when /^\s*([\d+\.]+)\s*$/ then $1
           # following regexp intentionally unanchored to handle
           # (ie ignore) leading server names
-          when /t=([\d+\.]+)/       then $1
+          when /t=([\d+\.]+)/ then $1
           end
         end
 

data/lib/new_relic/agent/instrumentation/rack/chain.rb

@@ -10,24 +10,24 @@ module NewRelic::Agent::Instrumentation
 
         builder_class.class_eval do
           include ::NewRelic::Agent::Instrumentation::RackBuilder
-  
+
           def to_app_with_newrelic_deferred_dependency_detection
             with_deferred_dependency_detection { to_app_without_newrelic }
           end
 
           alias_method :to_app_without_newrelic, :to_app
           alias_method :to_app, :to_app_with_newrelic_deferred_dependency_detection
-  
+
           if ::NewRelic::Agent::Instrumentation::RackHelpers.middleware_instrumentation_enabled?
             ::NewRelic::Agent.logger.info "Installing #{builder_class} middleware instrumentation"
 
             def run_with_newrelic(app, *args)
               run_with_tracing(app) { |wrapped_app| run_without_newrelic(wrapped_app, *args) }
             end
-   
+
             alias_method :run_without_newrelic, :run
             alias_method :run, :run_with_newrelic
-  
+
             def use_with_newrelic(middleware_class, *args, &block)
               use_with_tracing(middleware_class) { |wrapped_class| use_without_newrelic(wrapped_class, *args, &block) }
             end
@@ -45,7 +45,7 @@ module NewRelic::Agent::Instrumentation
         def self.instrument! url_map_class
           url_map_class.class_eval do
             alias_method :initialize_without_newrelic, :initialize
-    
+
             def initialize(map = {})
               traced_map = ::NewRelic::Agent::Instrumentation::RackURLMap.generate_traced_map(map)
               initialize_without_newrelic(traced_map)

data/lib/new_relic/agent/instrumentation/rack/helpers.rb

@@ -29,4 +29,3 @@ module NewRelic::Agent::Instrumentation
     end
   end
 end
-

data/lib/new_relic/agent/instrumentation/rack/instrumentation.rb

@@ -6,7 +6,6 @@ module NewRelic
   module Agent
     module Instrumentation
       module RackBuilder
-
         def self.track_deferred_detection builder_class
           class << builder_class
             attr_accessor :_nr_deferred_detection_ran
@@ -16,7 +15,7 @@ module NewRelic
 
         def deferred_dependency_check
           return if self.class._nr_deferred_detection_ran
-  
+
           NewRelic::Agent.logger.info "Doing deferred dependency-detection before Rack startup"
           DependencyDetection.detect!
           self.class._nr_deferred_detection_ran = true
@@ -28,7 +27,7 @@ module NewRelic
           if middleware_instrumentation_enabled?
             if ::NewRelic::Agent::Instrumentation::MiddlewareProxy.needs_wrapping?(app)
               ::NewRelic::Agent.logger.info("We weren't able to instrument all of your Rack middlewares.",
-                                            "To correct this, ensure you 'require \"newrelic_rpm\"' before setting up your middleware stack.")
+                "To correct this, ensure you 'require \"newrelic_rpm\"' before setting up your middleware stack.")
             end
           end
         end
@@ -41,7 +40,7 @@ module NewRelic
         # this point.
         def with_deferred_dependency_detection
           deferred_dependency_check
-          yield.tap{ |result| check_for_late_instrumentation(result) }
+          yield.tap { |result| check_for_late_instrumentation(result) }
         end
 
         def middleware_instrumentation_enabled?
@@ -70,4 +69,4 @@ module NewRelic
       end
     end
   end
-end
+end

data/lib/new_relic/agent/instrumentation/rack/prepend.rb

@@ -4,7 +4,6 @@
 
 module NewRelic::Agent::Instrumentation
   module Rack
-    
     module URLMap
       module Prepend
         def initialize(map = {})

data/lib/new_relic/agent/instrumentation/rack.rb

@@ -57,7 +57,7 @@ DependencyDetection.defer do
       prepend_instrument ::Rack::URLMap, NewRelic::Agent::Instrumentation::Rack::URLMap::Prepend
     else
       chain_instrument_target ::Rack::URLMap, NewRelic::Agent::Instrumentation::Rack::URLMap::Chain
-      ::NewRelic::Agent::Instrumentation::RackHelpers.instrument_url_map 
+      ::NewRelic::Agent::Instrumentation::RackHelpers.instrument_url_map
     end
   end
 end

data/lib/new_relic/agent/instrumentation/rails3/action_controller.rb

@@ -20,17 +20,16 @@ module NewRelic
             end
           end
 
-          def process_action(*args) #THREAD_LOCAL_ACCESS
+          def process_action(*args) # THREAD_LOCAL_ACCESS
             munged_params = NewRelic::Agent::ParameterFiltering.filter_rails_request_parameters(request.filtered_parameters)
             perform_action_with_newrelic_trace(:category => :controller,
-                                               :name => self.action_name,
-                                               :path => newrelic_metric_path,
-                                               :params => munged_params,
-                                               :class_name => self.class.name)  do
+              :name => self.action_name,
+              :path => newrelic_metric_path,
+              :params => munged_params,
+              :class_name => self.class.name) do
               super
             end
           end
-
         end
 
         module ActionView
@@ -47,6 +46,7 @@ module NewRelic
                 identifier
               end
             end
+
             def render_type(file_path)
               file = File.basename(file_path)
               if file.starts_with?('_')

data/lib/new_relic/agent/instrumentation/rails_notifications/action_cable.rb

@@ -9,8 +9,8 @@ DependencyDetection.defer do
 
   depends_on do
     defined?(::Rails::VERSION::MAJOR) &&
-     ::Rails::VERSION::MAJOR.to_i >= 5 &&
-       defined?(::ActionCable)
+      ::Rails::VERSION::MAJOR.to_i >= 5 &&
+      defined?(::ActionCable)
   end
 
   depends_on do

data/lib/new_relic/agent/instrumentation/rails_notifications/action_controller.rb

@@ -4,7 +4,6 @@
 require 'new_relic/agent/instrumentation/action_controller_subscriber'
 require 'new_relic/agent/prepend_supportability'
 
-
 DependencyDetection.defer do
   @name = :action_controller_notifications
 

data/lib/new_relic/agent/instrumentation/rake/chain.rb

@@ -18,8 +18,3 @@ module NewRelic::Agent::Instrumentation
     end
   end
 end
-
-
-
-
-

data/lib/new_relic/agent/instrumentation/rake/instrumentation.rb

@@ -99,14 +99,14 @@ module NewRelic
 
         def record_attributes(args, task)
           command_line = task.application.top_level_tasks.join(" ")
-          NewRelic::Agent::Transaction.merge_untrusted_agent_attributes({ :command => command_line },
-                                                                        :'job.rake',
-                                                                        NewRelic::Agent::AttributeFilter::DST_NONE)
+          NewRelic::Agent::Transaction.merge_untrusted_agent_attributes({:command => command_line},
+            :'job.rake',
+            NewRelic::Agent::AttributeFilter::DST_NONE)
           named_args = name_the_args(args, task.arg_names)
           unless named_args.empty?
             NewRelic::Agent::Transaction.merge_untrusted_agent_attributes(named_args,
-                                                                          :'job.rake.args',
-                                                                          NewRelic::Agent::AttributeFilter::DST_NONE)
+              :'job.rake.args',
+              NewRelic::Agent::AttributeFilter::DST_NONE)
           end
         rescue => e
           NewRelic::Agent.logger.error("Error during Rake task attribute recording.", e)
@@ -141,4 +141,4 @@ module NewRelic
       end
     end
   end
-end
+end

data/lib/new_relic/agent/instrumentation/rake.rb

@@ -17,7 +17,7 @@ DependencyDetection.defer do
   depends_on { ::NewRelic::Agent::Instrumentation::Rake.safe_from_third_party_gem? }
 
   executes do
-    ::NewRelic::Agent.logger.info  "Installing Rake instrumentation"
+    ::NewRelic::Agent.logger.info "Installing Rake instrumentation"
     ::NewRelic::Agent.logger.debug "Instrumenting Rake tasks: #{::NewRelic::Agent.config[:'rake.tasks']}"
   end
 

data/lib/new_relic/agent/instrumentation/redis/chain.rb

@@ -5,25 +5,25 @@
 
 module NewRelic::Agent::Instrumentation
   module Redis
-    module Chain 
+    module Chain
       def self.instrument!
         ::Redis::Client.class_eval do
           include NewRelic::Agent::Instrumentation::Redis
 
           alias_method :call_without_new_relic, :call
-    
+
           def call(*args, &block)
             call_with_tracing(args[0]) { call_without_new_relic(*args, &block) }
           end
-      
+
           alias_method :call_pipeline_without_new_relic, :call_pipeline
-    
+
           def call_pipeline(*args, &block)
             call_pipeline_with_tracing(args[0]) { call_pipeline_without_new_relic(*args, &block) }
           end
 
           alias_method :connect_without_new_relic, :connect
-    
+
           def connect(*args, &block)
             connect_with_tracing { connect_without_new_relic(*args, &block) }
           end
@@ -31,4 +31,4 @@ module NewRelic::Agent::Instrumentation
       end
     end
   end
-end
+end

data/lib/new_relic/agent/instrumentation/redis/instrumentation.rb

@@ -32,7 +32,7 @@ module NewRelic::Agent::Instrumentation
 
     private
 
-    def with_tracing operation, statement=nil
+    def with_tracing operation, statement = nil
       segment = NewRelic::Agent::Tracer.start_datastore_segment(
         product: PRODUCT_NAME,
         operation: operation,
@@ -62,4 +62,4 @@ module NewRelic::Agent::Instrumentation
       UNKNOWN
     end
   end
-end
+end

data/lib/new_relic/agent/instrumentation/redis/prepend.rb

@@ -21,4 +21,4 @@ module NewRelic::Agent::Instrumentation
       end
     end
   end
-end
+end

data/lib/new_relic/agent/instrumentation/redis.rb

@@ -13,7 +13,7 @@ DependencyDetection.defer do
   # Why not :redis? newrelic-redis used that name, so avoid conflicting
   named :redis_instrumentation
   configure_with :redis
-  
+
   depends_on do
     defined?(::Redis) && defined?(::Redis::VERSION)
   end