newrelic_rpm 4.1.0.333 → 4.2.0.334

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b3d8215fa6708cb77746b4bb7042b19ce62c863b
-  data.tar.gz: 817f76b9f47cc4d993b919ba6b25d27874635ab3
+  metadata.gz: eed94154d98e333b8c2c091ff3630815d4165045
+  data.tar.gz: 4206e75fd386e8f3ea148aba808854a2ec5cb91f
 SHA512:
-  metadata.gz: 207d7d3c3bf702d48da4caa3e45f9414914fcad1aa5ba1dee9e1f3e8c72dff1b893e5d117ef54add04eb6e6ee05aeef7012ba242322910902a970c2e2c13b1e9
-  data.tar.gz: 359c0dcdeda0ba4380df1715e116753dc54e4249cf38c72f85c9b57dde3102e5fd5432e1a71c9a209561c42927c7fa2fc7d776e6e575be7f6f59745c4689866b
+  metadata.gz: 31f37e060cab1014f3c2163b890eb2e68d8296a06146360187121250e93291eb9fb2399ef0b18e14604ee0b0fc2563e55494844a1bf0705df4199a3b71eb73ef
+  data.tar.gz: fb5e6ddacced4c1d72e703582da15fa099bddddb32ca7cc0c535aadbd11faa48b76454094d30ac03e13cae234670906f63c57365d360b082864fc7133df24f65

data/.travis.yml

@@ -31,10 +31,10 @@ notifications:
 
 rvm:
   # Run slowest builds first to try and optimize overall cycle time.
-  - jruby-9.1.2.0
-  - 2.4.0
-  - 2.3.3
-  - 2.2.6
+  - jruby-9.1.8.0
+  - 2.4.1
+  - 2.3.4
+  - 2.2.7
   - 2.1.10
   - 2.0.0-p648
 
@@ -61,6 +61,7 @@ env:
     - TYPE=UNIT ENVIRONMENT=rails41
     - TYPE=UNIT ENVIRONMENT=rails42
     - TYPE=UNIT ENVIRONMENT=rails50
+    - TYPE=UNIT ENVIRONMENT=rails51
     - TYPE=UNIT ENVIRONMENT=norails
     - TYPE=FUNCTIONAL GROUP=agent
     - TYPE=FUNCTIONAL GROUP=api
@@ -79,39 +80,39 @@ matrix:
   exclude:
     # Unsupported Rails/Ruby combinations
     # 2.4
-    - rvm: 2.4.0
+    - rvm: 2.4.1
       env: TYPE=UNIT ENVIRONMENT=rails21
-    - rvm: 2.4.0
+    - rvm: 2.4.1
       env: TYPE=UNIT ENVIRONMENT=rails22
-    - rvm: 2.4.0
+    - rvm: 2.4.1
       env: TYPE=UNIT ENVIRONMENT=rails23
-    - rvm: 2.4.0
+    - rvm: 2.4.1
       env: TYPE=UNIT ENVIRONMENT=rails30
-    - rvm: 2.4.0
+    - rvm: 2.4.1
       env: TYPE=UNIT ENVIRONMENT=rails31
-    - rvm: 2.4.0
+    - rvm: 2.4.1
       env: TYPE=UNIT ENVIRONMENT=rails32
-    - rvm: 2.4.0
+    - rvm: 2.4.1
       env: TYPE=UNIT ENVIRONMENT=rails40
-    - rvm: 2.4.0
+    - rvm: 2.4.1
       env: TYPE=UNIT ENVIRONMENT=rails41
-    - rvm: 2.4.0
+    - rvm: 2.4.1
       env: TYPE=UNIT ENVIRONMENT=rails42
 
     # 2.3
-    - rvm: 2.3.3
+    - rvm: 2.3.4
       env: TYPE=UNIT ENVIRONMENT=rails21
-    - rvm: 2.3.3
+    - rvm: 2.3.4
       env: TYPE=UNIT ENVIRONMENT=rails22
-    - rvm: 2.3.3
+    - rvm: 2.3.4
       env: TYPE=UNIT ENVIRONMENT=rails23
 
     # 2.2
-    - rvm: 2.2.6
+    - rvm: 2.2.7
       env: TYPE=UNIT ENVIRONMENT=rails21
-    - rvm: 2.2.6
+    - rvm: 2.2.7
       env: TYPE=UNIT ENVIRONMENT=rails22
-    - rvm: 2.2.6
+    - rvm: 2.2.7
       env: TYPE=UNIT ENVIRONMENT=rails23
 
     # 2.1
@@ -123,6 +124,8 @@ matrix:
       env: TYPE=UNIT ENVIRONMENT=rails23
     - rvm: 2.1.10
       env: TYPE=UNIT ENVIRONMENT=rails50
+    - rvm: 2.1.10
+      env: TYPE=UNIT ENVIRONMENT=rails51
 
     # 2.0
     - rvm: 2.0.0-p648
@@ -133,17 +136,19 @@ matrix:
       env: TYPE=UNIT ENVIRONMENT=rails23
     - rvm: 2.0.0-p648
       env: TYPE=UNIT ENVIRONMENT=rails50
+    - rvm: 2.0.0-p648
+      env: TYPE=UNIT ENVIRONMENT=rails51
 
     # jruby 9.0
-    - rvm: jruby-9.1.2.0
+    - rvm: jruby-9.1.8.0
       env: TYPE=UNIT ENVIRONMENT=rails21
-    - rvm: jruby-9.1.2.0
+    - rvm: jruby-9.1.8.0
       env: TYPE=UNIT ENVIRONMENT=rails22
-    - rvm: jruby-9.1.2.0
+    - rvm: jruby-9.1.8.0
       env: TYPE=UNIT ENVIRONMENT=rails23
-    - rvm: jruby-9.1.2.0
+    - rvm: jruby-9.1.8.0
       env: TYPE=UNIT ENVIRONMENT=rails30
-    - rvm: jruby-9.1.2.0
+    - rvm: jruby-9.1.8.0
       env: TYPE=UNIT ENVIRONMENT=rails31
-    - rvm: jruby-9.1.2.0
+    - rvm: jruby-9.1.8.0
       env: TYPE=UNIT ENVIRONMENT=rails32

data/CHANGELOG.md

@@ -1,5 +1,24 @@
 # New Relic Ruby Agent Release Notes #
 
+## v4.2.0 ##
+
+  * Sinatra 2.0 and Padrino 0.14.x Support
+
+  The agent has been verified against the latest versions of Sinatra and Padrino.
+
+  * Rails 5.1 Support
+
+  The Ruby agent has been validated against the latest release of Ruby on Rails!
+
+  * APP_ENV considered when determining environment
+
+  The agent will now consider the APP_ENV environment when starting up.
+
+  * Test files excluded from gem
+
+  The gemspec has been updated to exclude test files from being packaged into the
+  gem. Thanks dimko for the contribution!
+
 ## v4.1.0 ##
 
   * Developer Mode removed

data/lib/new_relic/agent/database.rb

@@ -24,6 +24,7 @@ module NewRelic
   module Agent
     module Database
       MAX_QUERY_LENGTH = 16384
+      ELLIPSIS = "...".freeze
 
       extend self
 
@@ -42,7 +43,7 @@ module NewRelic
 
       def truncate_query(query)
         if query.length > (MAX_QUERY_LENGTH - 4)
-          query[0..MAX_QUERY_LENGTH - 4] + '...'
+          query[0..MAX_QUERY_LENGTH - 4] << ELLIPSIS
         else
           query
         end
@@ -221,6 +222,13 @@ module NewRelic
           end
         end
 
+        NEWLINE = "\n".freeze
+
+        def append_sql new_sql
+          return if new_sql.empty?
+          @sql = Database.truncate_query(@sql << NEWLINE << new_sql)
+        end
+
         private
 
         POSTGRES_PREFIX = 'postgres'.freeze
@@ -244,8 +252,6 @@ module NewRelic
           end
         end
 
-        ELLIPSIS = "...".freeze
-
         def explainable?
           return false unless @explainer && is_select?(@sql)
 

data/lib/new_relic/agent/datastores.rb

@@ -147,9 +147,10 @@ module NewRelic
       # @api public
       #
       def self.notice_sql(query, scoped_metric, elapsed)
-        agent = NewRelic::Agent.instance
-        agent.transaction_sampler.notice_sql(query, nil, elapsed)
-        agent.sql_sampler.notice_sql(query, scoped_metric, nil, elapsed)
+        state = TransactionState.tl_get
+        if (txn = state.current_transaction) && (segment = txn.current_segment) && segment.respond_to?(:notice_sql)
+          segment.notice_sql(query)
+        end
         nil
       end
 
@@ -179,10 +180,10 @@ module NewRelic
       def self.notice_statement(statement, elapsed)
         # Settings may change eventually, but for now we follow the same
         # capture rules as SQL for non-SQL statements.
-        return unless NewRelic::Agent::Database.should_record_sql?
-
-        agent = NewRelic::Agent.instance
-        agent.transaction_sampler.notice_nosql_statement(statement, elapsed)
+        state = TransactionState.tl_get
+        if (txn = state.current_transaction) && (segment = txn.current_segment) && segment.respond_to?(:notice_nosql_statement)
+          segment.notice_nosql_statement(statement)
+        end
         nil
       end
 

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

@@ -45,13 +45,12 @@ module NewRelic
                             :request          => event.request,
                             :filtered_params  => filter(event.payload[:params]),
                             :apdex_start_time => event.queue_start,
-                            :transaction_name => event.metric_name)
+                            :transaction_name => event.metric_name,
+                            :ignore_apdex     => event.apdex_ignored?,
+                            :ignore_enduser   => event.enduser_ignored?)
         end
 
         def stop_transaction(event)
-          txn = state.current_transaction
-          txn.ignore_apdex!   if event.apdex_ignored?
-          txn.ignore_enduser! if event.enduser_ignored?
           Transaction.stop(state)
         end
 

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

@@ -14,14 +14,36 @@ module NewRelic
         CACHED_QUERY_NAME = 'CACHE'.freeze
 
         def initialize
+          define_cachedp_method
           # We cache this in an instance variable to avoid re-calling method
           # on each query.
           @explainer = method(:get_explain_plan)
           super
         end
 
+        # The cached? method is dynamically defined based on ActiveRecord version.
+        # This file can and often is required before ActiveRecord is loaded. For
+        # that reason we define the cache? method in initialize. The behavior
+        # difference is that AR 5.1 includes a key in the payload to check,
+        # where older versions set the :name to CACHE.
+
+        def define_cachedp_method
+          # we don't expect this to be called more than once, but we're being
+          # defensive.
+          return if defined?(cached?)
+          if ::ActiveRecord::VERSION::STRING >= "5.1.0"
+            def cached?(payload)
+              payload.fetch(:cached, false)
+            end
+          else
+            def cached?(payload)
+              payload[:name] == CACHED_QUERY_NAME
+            end
+          end
+        end
+
         def start(name, id, payload) #THREAD_LOCAL_ACCESS
-          return if payload[:name] == CACHED_QUERY_NAME
+          return if cached?(payload)
           return unless NewRelic::Agent.tl_is_execution_traced?
           config = active_record_config(payload)
           event = ActiveRecordEvent.new(name, Time.now, nil, id, payload, @explainer, config)
@@ -31,7 +53,7 @@ module NewRelic
         end
 
         def finish(name, id, payload) #THREAD_LOCAL_ACCESS
-          return if payload[:name] == CACHED_QUERY_NAME
+          return if cached?(payload)
           return unless state.is_execution_traced?
           event = pop_event(id)
           event.finish
@@ -109,8 +131,8 @@ module NewRelic
           # See comment for start_segment as we continue to work around limitations of the
           # current tracer in this method.
           def finish
-            if state.current_transaction
-              state.traced_method_stack.push_segment state, @segment
+            if txn = state.current_transaction
+              txn.add_segment @segment
             end
             @segment.finish
           end

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

@@ -2,17 +2,19 @@
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/rpm/blob/master/LICENSE for complete details.
 
+require 'new_relic/agent/database'
+
 module NewRelic
   module Instrumentation
     module ActsAsSolrInstrumentation
       module ParserMethodsInstrumentation
         def parse_query_with_newrelic(*args)
           self.class.trace_execution_scoped(["SolrClient/ActsAsSolr/query"]) do
-            t0 = Time.now
             begin
               parse_query_without_newrelic(*args)
             ensure
-              NewRelic::Agent.instance.transaction_sampler.notice_nosql(args.first.inspect, (Time.now - t0).to_f) rescue nil
+              return unless txn = ::NewRelic::Agent::TransactionState.tl_get.current_transaction
+              txn.current_segment.params[:statement] = ::NewRelic::Agent::Database.truncate_query(args.first.inspect) rescue nil
             end
           end
         end

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

@@ -356,7 +356,7 @@ module NewRelic
           txn_options   = create_transaction_options(trace_options, category, state)
 
           begin
-            txn = Transaction.start(state, category, txn_options)
+            Transaction.start(state, category, txn_options)
 
             begin
               yield
@@ -366,10 +366,6 @@ module NewRelic
             end
 
           ensure
-            if txn
-              txn.ignore_apdex!   if ignore_apdex?
-              txn.ignore_enduser! if ignore_enduser?
-            end
             Transaction.stop(state)
           end
         end
@@ -430,6 +426,8 @@ module NewRelic
           txn_options[:filtered_params] = trace_options[:params]
           txn_options[:transaction_name] = TransactionNamer.name_for(nil, self, category, trace_options)
           txn_options[:apdex_start_time] = detect_queue_start_time(state)
+          txn_options[:ignore_apdex] = ignore_apdex?
+          txn_options[:ignore_enduser] = ignore_enduser?
           txn_options
         end
 

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

@@ -184,8 +184,11 @@ module NewRelic
           state = NewRelic::Agent::TransactionState.tl_get
           return unless state.is_execution_traced?
 
-          duration = msg.duration / 1000000.0
-          NewRelic::Agent.instance.transaction_sampler.notice_sql(msg.query, nil, duration, state)
+          txn = state.current_transaction
+
+          if txn && txn.current_segment.respond_to?(:notice_sql)
+            txn.current_segment.notice_sql msg.query
+          end
         ensure
           super
         end

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

@@ -56,10 +56,10 @@ DependencyDetection.defer do
 
       # It's key that this method eats all exceptions, as it rests between the
       # Mongo operation the user called and us returning them the data. Be safe!
-      def new_relic_notice_statement(t0, payload, name)
+      def new_relic_notice_statement(segment, payload, name)
         statement = NewRelic::Agent::Datastores::Mongo::StatementFormatter.format(payload, name)
         if statement
-          NewRelic::Agent.instance.transaction_sampler.notice_nosql_statement(statement, (Time.now - t0).to_f)
+          segment.notice_nosql_statement statement
         end
       rescue => e
         NewRelic::Agent.logger.debug("Exception during Mongo statement gathering", e)
@@ -85,7 +85,7 @@ DependencyDetection.defer do
             instrument_without_new_relic_trace(name, payload, &block)
           end
 
-          new_relic_notice_statement(segment.start_time, payload, name) if segment
+          new_relic_notice_statement(segment, payload, name) if segment
           result
         ensure
           segment.finish if segment
@@ -107,7 +107,7 @@ DependencyDetection.defer do
             save_without_new_relic_trace(doc, opts, &block)
           end
 
-          new_relic_notice_statement(segment.start_time, doc, :save) if segment
+          new_relic_notice_statement(segment, doc, :save) if segment
           result
         ensure
           segment.finish if segment
@@ -138,7 +138,7 @@ DependencyDetection.defer do
                    spec.dup
                  end
 
-          new_relic_notice_statement(segment.start_time, spec, :ensureIndex) if segment
+          new_relic_notice_statement(segment, spec, :ensureIndex) if segment
           result
         ensure
           segment.finish if segment

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

@@ -102,7 +102,9 @@ module NewRelic
             def invoke_prerequisites_concurrently(*_)
               NewRelic::Agent::MethodTracer.trace_execution_scoped("Rake/execute/multitask") do
                 prereqs = self.prerequisite_tasks.map(&:name).join(", ")
-                NewRelic::Agent::Datastores.notice_statement("Couldn't trace concurrent prereq tasks: #{prereqs}", 0)
+                if txn = ::NewRelic::Agent::TransactionState.tl_get.current_transaction
+                  txn.current_segment.params[:statement] = NewRelic::Agent::Database.truncate_query("Couldn't trace concurrent prereq tasks: #{prereqs}")
+                end
                 super
               end
             end

data/lib/new_relic/agent/method_tracer_helpers.rb

@@ -9,63 +9,6 @@ module NewRelic
 
       extend self
 
-      # helper for logging errors to the newrelic_agent.log
-      # properly. Logs the error at error level
-      def log_errors(code_area)
-        yield
-      rescue => e
-        ::NewRelic::Agent.logger.error("Caught exception in #{code_area}.", e)
-      end
-
-      def trace_execution_scoped_header(state, t0)
-        log_errors(:trace_execution_scoped_header) do
-          stack = state.traced_method_stack
-          stack.push_frame(state, :method_tracer, t0)
-        end
-      end
-
-      def record_metrics(state, first_name, other_names, duration, exclusive, options)
-        record_scoped_metric = options.has_key?(:scoped_metric) ? options[:scoped_metric] : true
-        stat_engine = NewRelic::Agent.instance.stats_engine
-        if record_scoped_metric
-          stat_engine.record_scoped_and_unscoped_metrics(state, first_name, other_names, duration, exclusive)
-        else
-          metrics = [first_name].concat(other_names)
-          stat_engine.record_unscoped_metrics(state, metrics, duration, exclusive)
-        end
-      end
-
-      def trace_execution_scoped_footer(state, t0, first_name, metric_names, expected_frame, options, t1=Time.now.to_f)
-        log_errors(:trace_method_execution_footer) do
-          if expected_frame
-            stack = state.traced_method_stack
-            create_metrics = options.has_key?(:metric) ? options[:metric] : true
-            frame = stack.pop_frame(state, expected_frame, first_name, t1, create_metrics)
-            if create_metrics
-              duration = t1 - t0
-              exclusive = duration - frame.children_time
-
-              if duration < MAX_ALLOWED_METRIC_DURATION
-                if duration < 0
-                  ::NewRelic::Agent.logger.log_once(:warn, "metric_duration_negative:#{first_name}",
-                    "Metric #{first_name} has negative duration: #{duration} s")
-                end
-
-                if exclusive < 0
-                  ::NewRelic::Agent.logger.log_once(:warn, "metric_exclusive_negative:#{first_name}",
-                    "Metric #{first_name} has negative exclusive time: duration = #{duration} s, child_time = #{frame.children_time}")
-                end
-
-                record_metrics(state, first_name, metric_names, duration, exclusive, options)
-              else
-                ::NewRelic::Agent.logger.log_once(:warn, "too_huge_metric:#{first_name}",
-                  "Ignoring metric #{first_name} with unacceptably large duration: #{duration} s")
-              end
-            end
-          end
-        end
-      end
-
       def trace_execution_scoped(metric_names, options={}) #THREAD_LOCAL_ACCESS
         state = NewRelic::Agent::TransactionState.tl_get
         return yield unless state.is_execution_traced?