newrelic_rpm 3.5.2.17 → 3.5.3.24

data/test/multiverse/suites/agent_only/thread_profiling_test.rb

@@ -0,0 +1,80 @@
+# https://newrelic.atlassian.net/wiki/display/eng/Agent+Thread+Profiling
+# https://newrelic.atlassian.net/browse/RUBY-917
+
+if RUBY_VERSION >= '1.9'
+class ThreadProfilingTest < Test::Unit::TestCase
+  def setup
+    NewRelic::Agent.manual_start(:'thread_profiler.enabled' => true)
+
+    @agent = NewRelic::Agent.instance
+    @thread_profiler = @agent.thread_profiler
+
+    $collector ||= NewRelic::FakeCollector.new
+    $collector.reset
+    $collector.mock['connect'] = [200, {'return_value' => {"agent_run_id" => 666 }}]
+    $collector.mock['get_agent_commands'] = [200, {'return_value' => START_COMMAND}]
+    $collector.mock['agent_command_results'] = [200, {'return_value' => []}]
+    $collector.run
+  end
+
+  def teardown
+    $collector.reset
+  end
+
+  START_COMMAND = [[666,{
+      "name" => "start_profiler",
+      "arguments" => {
+        "profile_id" => -1,
+        "sample_period" => 0.01,
+        "duration" => 0.5,
+        "only_runnable_threads" => false,
+        "only_request_threads" => false,
+        "profile_agent_code" => false
+      }
+    }]]
+
+  STOP_COMMAND = [[666,{
+      "name" => "stop_profiler",
+      "arguments" => {
+        "profile_id" => -1,
+        "report_data" => true
+      }
+    }]]
+
+  # These are potentially fragile for being timing based
+  # START_COMMAND with 0.01 sampling and 0.5 duration expects to get
+  # roughly 50 polling cycles in. We check signficiantly less than that.
+
+  # STOP_COMMAND when immediately issued after a START_COMMAND is expected
+  # go only let a few cycles through, so we check less than 10
+
+  def test_thread_profiling
+    @agent.send(:check_for_agent_commands)
+    sleep(1)
+    NewRelic::Agent.shutdown
+
+    profile_data = $collector.calls_for('profile_data')[0]
+    assert_equal(666, profile_data[0])
+
+    poll_count = profile_data[1][0][3]
+    assert poll_count > 25, "Expected poll_count > 25, but was #{poll_count}"
+  end
+
+  def test_thread_profiling_can_stop
+    @agent.send(:check_for_agent_commands)
+
+    $collector.mock['get_agent_commands'] = [200, {'return_value' => STOP_COMMAND}]
+    @agent.send(:check_for_agent_commands)
+
+    sleep(0.1)
+    NewRelic::Agent.shutdown
+
+    profile_data = $collector.calls_for('profile_data')[0]
+    assert_equal(666, profile_data[0])
+
+    poll_count = profile_data[1][0][3]
+    assert poll_count < 10, "Expected poll_count < 10, but was #{poll_count}"
+  end
+end
+end
+

data/test/multiverse/suites/datamapper/Envfile

@@ -0,0 +1,8 @@
+suite_condition("Datamapper not supported in 1.8.6") do
+  RUBY_VERSION != '1.8.6'
+end
+
+gemfile <<-RB
+  gem 'datamapper'
+RB
+

data/test/multiverse/suites/datamapper/config/newrelic.yml

@@ -0,0 +1,22 @@
+---
+development:
+  error_collector:
+    capture_source: true
+    enabled: true
+  apdex_t: 0.5
+  ssl: false
+  monitor_mode: true
+  license_key: bootstrap_newrelic_admin_license_key_000
+  developer_mode: false
+  app_name: test
+  host: 127.0.0.1
+  api_host: 127.0.0.1
+  port: 30303
+  transaction_tracer:
+    record_sql: obfuscated
+    enabled: true
+    stack_trace_threshold: 0.5
+    transaction_threshold: 1.0
+  capture_params: false
+  log_level: debug
+  log_file_path: agent.log

data/test/multiverse/suites/datamapper/encoding_test.rb

@@ -0,0 +1,36 @@
+# https://support.newrelic.com/tickets/2101
+# https://github.com/newrelic/rpm/pull/42
+# https://github.com/newrelic/rpm/pull/45
+require 'test/unit'
+require 'new_relic/agent/instrumentation/data_mapper'
+
+class DatabaseAdapter
+  # we patch in here
+  def log(*args)
+  end
+  include ::NewRelic::Agent::Instrumentation::DataMapperInstrumentation
+end
+
+class EncodingTest < Test::Unit::TestCase
+  # datamapper wants a msg object
+  MSG = Object.new
+  def MSG.query
+    # Contains invalid UTF8 Byte
+    q = "select ICS95095010000000000083320000BS01030000004100+\xFF00000000000000000"
+    if RUBY_VERSION >= '1.9'
+      # Force the query to an invalid encoding
+      q.force_encoding 'UTF-8'
+    end
+    q
+  end
+  def MSG.duration; 1.0; end
+
+  def test_should_not_bomb_out_if_a_query_is_in_an_invalid_encoding
+    if RUBY_VERSION >= '1.9'
+      assert_equal false, MSG.query.valid_encoding?
+    end
+    db = DatabaseAdapter.new
+    db.send(:log, MSG)
+  end
+end
+

data/test/multiverse/suites/monitor_mode_false/Envfile

@@ -0,0 +1,2 @@
+self.newrelic_gemfile_options=(':require => false')
+

data/test/multiverse/suites/monitor_mode_false/config/newrelic.yml

@@ -0,0 +1,25 @@
+development:
+  browser_monitoring:
+    auto_instrument: true
+  error_collector:
+    capture_source: true
+    enabled: true
+  apdex_t: 0.5
+  ssl: false
+  monitor_mode: false
+  license_key: bootstrap_newrelic_admin_license_key_000
+  developer_mode: false
+  app_name: test
+  host: staging-collector.newrelic.com
+  api_host: 127.0.0.1
+  port: 80
+  transaction_tracer:
+    record_sql: obfuscated
+    enabled: true
+    stack_trace_threshold: 0.5
+    transaction_threshold: 1.0
+  capture_params: false
+  log_level: debug
+  disable_serialization: false
+  log_file_name: mon_mode_false_newrelic.log
+  agent_enabled: false

data/test/multiverse/suites/monitor_mode_false/no_dns_resolv.rb

@@ -0,0 +1,29 @@
+require 'resolv'
+
+class NoDnsResolv < Test::Unit::TestCase
+  def test_should_no_resolve_hostname_when_agent_is_disabled
+    Resolv.class_eval do
+
+      if RUBY_VERSION == '1.9.2'
+        class_variable_set(:@@getaddress_called,false)
+      else
+        @@getaddress_called = false
+      end
+
+      def self.getaddress(host)
+        @@getaddress_called = true
+        puts "address for #{host} requested"
+        "127.0.0.1"
+      end
+
+      def self.getaddress_called?
+        @@getaddress_called
+      end
+    end
+
+    require 'newrelic_rpm'
+
+    assert(!Resolv.getaddress_called?,
+          'called Resolv.getaddress when we should not have')
+  end
+end

data/test/multiverse/suites/no_load/Envfile

@@ -0,0 +1,2 @@
+self.newrelic_gemfile_options=(':require => false')
+

data/test/multiverse/suites/no_load/config/newrelic.yml

@@ -0,0 +1,23 @@
+---
+development:
+  error_collector:
+    capture_source: true
+    enabled: true
+  apdex_t: 0.5
+  ssl: false
+  monitor_mode: true
+  license_key: bootstrap_newrelic_admin_license_key_000
+  developer_mode: false
+  app_name: test
+  host: 127.0.0.1
+  api_host: 127.0.0.1
+  port: 30303
+  transaction_tracer:
+    record_sql: obfuscated
+    enabled: true
+    stack_trace_threshold: 0.5
+    transaction_threshold: 1.0
+  capture_params: false
+  log_level: debug
+  #log_file_path: STDOUT
+  disable_serialization: false

data/test/multiverse/suites/no_load/start_up_test.rb

@@ -0,0 +1,14 @@
+# RUBY-839 make sure there is no STDOUT chatter
+require 'open3'
+
+class StartUpTest < Test::Unit::TestCase
+  def test_should_not_print_to_stdout_when_logging_available
+    ruby = 'require "newrelic_rpm"; NewRelic::Agent.manual_start; NewRelic::Agent.shutdown'
+    cmd = "bundle exec ruby -e '#{ruby}'"
+
+    sin, sout, serr = Open3.popen3(cmd)
+
+    jruby_noise = "JRuby limited openssl loaded. http://jruby.org/openssl\ngem install jruby-openssl for full support.\n"
+    assert_equal '', (sout.read + serr.read).sub(jruby_noise, '')
+  end
+end

data/test/multiverse/suites/rails_3_error_tracing/Envfile

@@ -0,0 +1,15 @@
+suite_condition("Rails 3 does not support 1.8.6") do
+  RUBY_VERSION != '1.8.6'
+end
+
+gemfile <<-RB
+  gem 'rails', '~>3.2.0'
+RB
+
+gemfile <<-RB
+  gem 'rails', '~>3.1.0'
+RB
+
+gemfile <<-RB
+  gem 'rails', '~>3.0.0'
+RB

data/test/multiverse/suites/rails_3_error_tracing/config/newrelic.yml

@@ -0,0 +1,165 @@
+#
+# This file configures the NewRelic RPM Agent, NewRelic RPM monitors Rails 
+# applications with deep visibility and low overhead.  For more information, 
+# visit www.newrelic.com.
+#
+# 
+#
+# here are the settings that are common to all environments
+common: &default_settings
+
+  # ============================== LICENSE KEY ===============================
+  # You must specify the licence key associated with your New Relic account.
+  # This key binds your Agent's data to your account in the New Relic RPM service.
+
+  #license_key: 'bootstrap_newrelic_admin_license_key_000'
+  #host: localhost
+  #port: 3000
+
+  license_key: 'bd0e1d52adade840f7ca727d29a86249e89a6f1c'
+  host: localhost
+
+  # Agent Enabled
+  # Use this setting to force the agent to run or not run.
+  # Default is 'auto' which means the agent will install and run only if a
+  # valid dispatcher such as Mongrel is running.  This prevents it from running
+  # with Rake or the console.  Set to false to completely turn the agent off
+  # regardless of the other settings.  Valid values are true, false and auto.
+  # agent_enabled: auto
+  
+  # Application Name
+  # Set this to be the name of your application as you'd like it show up in RPM.
+  # RPM will then auto-map instances of your application into a RPM "application"
+  # on your home dashboard page. This setting does not prevent you from manually
+  # defining applications.
+  app_name: Rails 3 view test ap
+
+  # When 'enabled' is turned on, the agent collects performance data by inserting lightweight
+  # tracers on key methods inside the rails framework and asynchronously aggregating
+  # and reporting this performance data to the NewRelic RPM service at newrelic.com.
+  enabled: false
+
+  # The newrelic agent generates its own log file to keep its logging information
+  # separate from that of your application.  Specify its log level here.
+  log_level: debug
+  
+  # The newrelic agent communicates with the RPM service via http by default.
+  # If you want to communicate via https to increase security, then turn on
+  # SSL by setting this value to true.  Note, this will result in increased
+  # CPU overhead to perform the encryption involved in SSL communication, but this
+  # work is done asynchronously to the threads that process your application code, so
+  # it should not impact response times.
+  ssl: false
+  
+  # Set your application's Apdex threshold value with the 'apdex_t' setting, in seconds. The
+  # apdex_t value determines the buckets used to compute your overall Apdex score. Requests
+  # that take less than apdex_t seconds to process will be classified as Satisfying transactions;
+  # more than apdex_t seconds as Tolerating transactions; and more than four times the apdex_t 
+  # value as Frustrating transactions. For more
+  # about the Apdex standard, see http://support.newrelic.com/faqs/general/apdex
+  apdex_t: 1.0
+    
+
+  # Proxy settings for connecting to the RPM server.
+  #
+  # If a proxy is used, the host setting is required.  Other settings are optional.  Default
+  # port is 8080.
+  #
+  # proxy_host: hostname
+  # proxy_port: 8080
+  # proxy_user:
+  # proxy_pass:
+
+  
+  # Tells transaction tracer and error collector (when enabled) whether or not to capture HTTP params. 
+  # When true, the RoR filter_parameter_logging mechanism is used so that sensitive parameters are not recorded
+  capture_params: true
+
+
+  # Transaction tracer captures deep information about slow
+  # transactions and sends this to the RPM service once a minute. Included in the
+  # transaction is the exact call sequence of the transactions including any SQL statements
+  # issued.
+  transaction_tracer:
+  
+    # Transaction tracer is enabled by default. Set this to false to turn it off. This feature
+    # is only available at the Silver and above product levels.
+    enabled: true
+    
+    # Threshold in seconds for when to collect a transaction trace. When the response time of
+    # a controller action exceeds this threshold, a transaction trace will be recorded and sent
+    # to RPM. Valid values are any float value, or (default) "apdex_f", which will use the
+    # threshold for an dissatisfying Apdex controller action - four times the Apdex T value.
+    transaction_threshold: apdex_f
+ 
+    # When transaction tracer is on, SQL statements can optionally be recorded. The recorder
+    # has three modes, "off" which sends no SQL, "raw" which sends the SQL statement in its 
+    # original form, and "obfuscated", which strips out numeric and string literals
+    record_sql: obfuscated
+    
+    # Threshold in seconds for when to collect stack trace for a SQL call. In other words, 
+    # when SQL statements exceed this threshold, then capture and send to RPM the current
+    # stack trace. This is helpful for pinpointing where long SQL calls originate from  
+    stack_trace_threshold: 0.500
+  
+  # Error collector captures information about uncaught exceptions and sends them to RPM for
+  # viewing
+  error_collector:
+    
+    # Error collector is enabled by default. Set this to false to turn it off. This feature
+    # is only available at the Silver and above product levels
+    enabled: true
+    
+    # Tells error collector whether or not to capture a source snippet around the place of the
+    # error when errors are View related.
+    capture_source: true    
+    
+    # To stop specific errors from reporting to RPM, set this property to comma separated 
+    # values
+    #
+    #ignore_errors: ActionController::RoutingError, ...
+    ignore_errors: IgnoredError
+
+  # (Advanced) Uncomment this to ensure the cpu and memory samplers won't run.  Useful when you
+  # are using the agent to monitor an external resource
+  # disable_samplers: true
+
+# override default settings based on your application's environment
+
+# NOTE if your application has other named environments, you should
+# provide newrelic conifguration settings for these enviromnents here.
+
+development:
+  <<: *default_settings
+  # turn off communication to RPM service in development mode.
+  # NOTE: for initial evaluation purposes, you may want to temporarily turn
+  # the agent on in development mode.
+  enabled: true
+
+  # When running in Developer Mode, the New Relic Agent will present 
+  # performance information on the last 100 transactions you have 
+  # executed since starting the mongrel.  to view this data, go to 
+  # http://localhost:3000/newrelic
+  developer: true
+
+test:
+  <<: *default_settings
+  # it almost never makes sense to turn on the agent when running unit, functional or
+  # integration tests or the like.
+  enabled: true
+  log_level: debug
+
+# Turn on the agent in production for 24x7 monitoring.  NewRelic testing shows
+# an average performance impact of < 5 ms per transaction, you you can leave this on
+# all the time without incurring any user-visible performance degredation.
+production:
+  <<: *default_settings
+  enabled: true
+
+# many applications have a staging environment which behaves identically to production.
+# Support for that environment is provided here.  By default, the staging environment has
+# the agent turned on.
+staging:
+  <<: *default_settings
+  enabled: true
+  app_name: My Application (Staging)

data/test/multiverse/suites/rails_3_error_tracing/error_tracing_test.rb

@@ -0,0 +1,236 @@
+# https://newrelic.atlassian.net/browse/RUBY-747
+
+ENV['NEW_RELIC_DISPATCHER'] = 'test'
+
+require 'action_controller/railtie'
+require 'rails/test_unit/railtie'
+require 'rails/test_help'
+require 'test/unit'
+require 'new_relic/rack/error_collector'
+require 'fake_service'
+
+
+# BEGIN RAILS APP
+
+class MyApp < Rails::Application
+  # We need a secret token for session, cookies, etc.
+  config.active_support.deprecation = :log
+  config.secret_token = "49837489qkuweoiuoqwehisuakshdjksadhaisdy78o34y138974xyqp9rmye8yrpiokeuioqwzyoiuxftoyqiuxrhm3iou1hrzmjk"
+end
+MyApp.initialize!
+
+MyApp.routes.draw do
+  get('/bad_route' => 'Test#controller_error',
+      :constraints => lambda do |_|
+        raise ActionController::RoutingError.new('this is an uncaught routing error')
+      end)
+  match '/:controller(/:action(/:id))'
+end
+
+class ApplicationController < ActionController::Base; end
+
+# a basic active model compliant model we can render
+class Foo
+  extend ActiveModel::Naming
+  def to_model
+    self
+  end
+
+  def valid?()      true end
+  def new_record?() true end
+  def destroyed?()  true end
+
+  def raise_error
+    raise 'this is an uncaught model error'
+  end
+
+  def errors
+    obj = Object.new
+    def obj.[](key)         [] end
+    def obj.full_messages() [] end
+    obj
+  end
+end
+
+class TestController < ApplicationController
+  include Rails.application.routes.url_helpers
+  newrelic_ignore :only => :ignored_action
+
+  def controller_error
+    raise 'this is an uncaught controller error'
+  end
+
+  def view_error
+    render :inline => "<% raise 'this is an uncaught view error' %>"
+  end
+
+  def model_error
+    Foo.new.raise_error
+  end
+
+  def ignored_action
+    raise 'this error should not be noticed'
+  end
+
+  def ignored_error
+    raise IgnoredError.new('this error should not be noticed')
+  end
+
+  def server_ignored_error
+    raise ServerIgnoredError.new('this is a server ignored error')
+  end
+
+  def noticed_error
+    newrelic_notice_error(RuntimeError.new('this error should be noticed'))
+    render :text => "Shoulda noticed an error"
+  end
+end
+# END RAILS APP
+
+class IgnoredError < StandardError; end
+class ServerIgnoredError < StandardError; end
+
+class TestControllerTest < ActionDispatch::IntegrationTest
+  def setup
+    NewRelic::Agent::Agent.instance_variable_set(:@instance, NewRelic::Agent::Agent.new)
+
+    @service = NewRelic::FakeService.new
+    NewRelic::Agent::Agent.instance.service = @service
+
+    NewRelic::Agent.manual_start
+  end
+
+  def teardown
+    NewRelic::Agent::Agent.instance.shutdown if NewRelic::Agent::Agent.instance
+  end
+end
+
+class ErrorsWithoutSSCTest < TestControllerTest
+  def setup
+    super
+    reset_error_collector
+  end
+
+  def reset_error_collector
+    @error_collector = NewRelic::Agent.instance.error_collector
+    NewRelic::Agent.instance.error_collector \
+      .instance_variable_set(:@ignore_filter, nil)
+
+    # sanity checks
+    assert(@error_collector.enabled?,
+           'error collector should be enabled')
+    assert(!NewRelic::Agent.instance.error_collector.ignore_error_filter,
+           'no ignore error filter should be set')
+  end
+
+
+  def test_error_collector_should_be_enabled
+    assert NewRelic::Agent.config[:agent_enabled]
+    assert NewRelic::Agent.config[:'error_collector.enabled']
+    assert @error_collector.enabled?
+    assert Rails.application.config.middleware.include?(NewRelic::Rack::ErrorCollector)
+  end
+
+  def test_should_capture_error_raised_in_view
+    get '/test/view_error'
+    assert_error_reported_once('this is an uncaught view error')
+  end
+
+  def test_should_capture_error_raised_in_controller
+    get '/test/controller_error'
+    assert_error_reported_once('this is an uncaught controller error')
+  end
+
+  def test_should_capture_error_raised_in_model
+    get '/test/model_error'
+    assert_error_reported_once('this is an uncaught model error')
+  end
+
+  def test_should_capture_noticed_error_in_controller
+    get '/test/noticed_error'
+    assert_error_reported_once('this error should be noticed')
+  end
+
+  def test_should_capture_manually_noticed_error
+    NewRelic::Agent.notice_error(RuntimeError.new('this is a noticed error'))
+    assert_error_reported_once('this is a noticed error')
+  end
+
+  def test_should_capture_routing_error
+    get '/bad_route'
+    assert_error_reported_once('this is an uncaught routing error')
+  end
+
+  def test_should_capture_request_uri_and_params
+    get '/bad_route?eat=static'
+    assert_equal('/bad_route',
+                 @error_collector.errors[0].params[:request_uri])
+    assert_equal({'eat' => 'static'},
+                 @error_collector.errors[0].params[:request_params])
+  end
+
+  def test_should_not_notice_errors_from_ignored_action
+    get '/test/ignored_action'
+    assert(@error_collector.errors.empty?,
+           'Noticed an error that should have been ignored')
+  end
+
+  def test_should_not_notice_ignored_error_classes
+    get '/test/ignored_error'
+    assert(@error_collector.errors.empty?,
+           'Noticed an error that should have been ignored')
+  end
+
+  def test_should_not_notice_filtered_errors
+    NewRelic::Agent.instance.error_collector.ignore_error_filter do |error|
+      !error.kind_of?(RuntimeError)
+    end
+
+    get 'test/controller_error'
+    assert(NewRelic::Agent.instance.error_collector.errors.empty?,
+           'Noticed an error that should have been ignored')
+  end
+
+  def test_should_notice_server_ignored_error_if_no_server_side_config
+    get '/test/server_ignored_error'
+    assert_error_reported_once('this is a server ignored error')
+  end
+
+  protected
+
+  def assert_error_reported_once(message)
+    assert_equal(message,
+                 @error_collector.errors[0].message,
+                 'This error type was not detected')
+    assert_equal(1, @error_collector.errors.size,
+                 'Too many of this error type was detected')
+  end
+end
+
+class ErrorsWithSSCTest < ErrorsWithoutSSCTest
+  def setup
+    super
+    @service.mock['connect'] = {
+      "listen_to_server_config" => true,
+      "agent_run_id" => 1,
+      "error_collector.ignore_errors" => 'IgnoredError,ServerIgnoredError',
+      "error_collector.enabled" => true,
+      "error_collector.capture_source" => true,
+      "collect_errors" => true
+    }
+
+    # Force us to apply the mocked connect values to our configuration
+    NewRelic::Agent.instance.query_server_for_configuration
+  end
+
+  def test_should_notice_server_ignored_error_if_no_server_side_config
+    # Overrides test in base class, since doesn't apply
+  end
+
+  def test_should_ignore_server_ignored_errors
+    get '/test/server_ignored_error'
+    assert(@error_collector.errors.empty?,
+           'Noticed an error that should have been ignored')
+  end
+
+end