dolores_rpm 3.2.0.2

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

@@ -0,0 +1,424 @@
+require 'new_relic/agent/instrumentation/metric_frame'
+require 'new_relic/agent/instrumentation/queue_time'
+module NewRelic
+  module Agent
+    module Instrumentation
+      # == NewRelic instrumentation for controller actions and tasks
+      #
+      # This instrumentation is applied to the action controller to collect
+      # metrics for every web request.
+      #
+      # It can also be used to capture performance information for
+      # background tasks and other non-web transactions, including
+      # detailed transaction traces and traced errors.
+      #
+      # For details on how to instrument background tasks see
+      # ClassMethods#add_transaction_tracer and
+      # #perform_action_with_newrelic_trace
+      #
+      module ControllerInstrumentation
+
+        def self.included(clazz) # :nodoc:
+          clazz.extend(ClassMethods)
+        end
+
+        # This module is for importing stubs when the agent is disabled
+        module ClassMethodsShim # :nodoc:
+          def newrelic_ignore(*args); end
+          def newrelic_ignore_apdex(*args); end
+        end
+
+        module Shim # :nodoc:
+          def self.included(clazz)
+            clazz.extend(ClassMethodsShim)
+          end
+          def newrelic_notice_error(*args); end
+          def new_relic_trace_controller_action(*args); yield; end
+          def newrelic_metric_path; end
+          def perform_action_with_newrelic_trace(*args); yield; end
+        end
+
+        module ClassMethods
+          # Have NewRelic ignore actions in this controller.  Specify the actions as hash options
+          # using :except and :only.  If no actions are specified, all actions are ignored.
+          def newrelic_ignore(specifiers={})
+            newrelic_ignore_aspect('do_not_trace', specifiers)
+          end
+          # Have NewRelic omit apdex measurements on the given actions.  Typically used for
+          # actions that are not user facing or that skew your overall apdex measurement.
+          # Accepts :except and :only options, as with #newrelic_ignore.
+          def newrelic_ignore_apdex(specifiers={})
+            newrelic_ignore_aspect('ignore_apdex', specifiers)
+          end
+
+          def newrelic_ignore_aspect(property, specifiers={}) # :nodoc:
+            if specifiers.empty?
+              self.newrelic_write_attr property, true
+            elsif ! (Hash === specifiers)
+              logger.error "newrelic_#{property} takes an optional hash with :only and :except lists of actions (illegal argument type '#{specifiers.class}')"
+            else
+              self.newrelic_write_attr property, specifiers
+            end
+          end
+
+          # Should be monkey patched into the controller class implemented
+          # with the inheritable attribute mechanism.
+          def newrelic_write_attr(attr_name, value) # :nodoc:
+            instance_variable_set "@#{attr_name}", value
+          end
+          def newrelic_read_attr(attr_name) # :nodoc:
+            instance_variable_get "@#{attr_name}"
+          end
+
+          # Add transaction tracing to the given method.  This will treat
+          # the given method as a main entrypoint for instrumentation, just
+          # like controller actions are treated by default.  Useful especially
+          # for background tasks.
+          #
+          # Example for background job:
+          #   class Job
+          #     include NewRelic::Agent::Instrumentation::ControllerInstrumentation
+          #     def run(task)
+          #        ...
+          #     end
+          #     # Instrument run so tasks show up under task.name.  Note single
+          #     # quoting to defer eval to runtime.
+          #     add_transaction_tracer :run, :name => '#{args[0].name}'
+          #   end
+          #
+          # Here's an example of a controller that uses a dispatcher
+          # action to invoke operations which you want treated as top
+          # level actions, so they aren't all lumped into the invoker
+          # action.
+          #
+          #   MyController < ActionController::Base
+          #     include NewRelic::Agent::Instrumentation::ControllerInstrumentation
+          #     # dispatch the given op to the method given by the service parameter.
+          #     def invoke_operation
+          #       op = params['operation']
+          #       send op
+          #     end
+          #     # Ignore the invoker to avoid double counting
+          #     newrelic_ignore :only => 'invoke_operation'
+          #     # Instrument the operations:
+          #     add_transaction_tracer :print
+          #     add_transaction_tracer :show
+          #     add_transaction_tracer :forward
+          #   end
+          #
+          # Here's an example of how to pass contextual information into the transaction
+          # so it will appear in transaction traces:
+          #
+          #   class Job
+          #     include NewRelic::Agent::Instrumentation::ControllerInstrumentation
+          #     def process(account)
+          #        ...
+          #     end
+          #     # Include the account name in the transaction details.  Note the single
+          #     # quotes to defer eval until call time.
+          #     add_transaction_tracer :process, :params => '{ :account_name => args[0].name }'
+          #   end
+          #
+          # See NewRelic::Agent::Instrumentation::ControllerInstrumentation#perform_action_with_newrelic_trace
+          # for the full list of available options.
+          #
+          def add_transaction_tracer(method, options={})
+            # The metric path:
+            options[:name] ||= method.to_s
+            # create the argument list:
+            options_arg = []
+            options.each do |key, value|
+              valuestr = case
+                         when value.is_a?(Symbol)
+                           value.inspect
+                         when key == :params
+                           value.to_s
+                         else
+                           %Q["#{value.to_s}"]
+                         end
+              options_arg << %Q[:#{key} => #{valuestr}]
+            end
+            class_eval <<-EOC
+              def #{method.to_s}_with_newrelic_transaction_trace(*args, &block)
+                perform_action_with_newrelic_trace(#{options_arg.join(',')}) do
+                  #{method.to_s}_without_newrelic_transaction_trace(*args, &block)
+                 end
+              end
+            EOC
+            alias_method "#{method.to_s}_without_newrelic_transaction_trace", method.to_s
+            alias_method method.to_s, "#{method.to_s}_with_newrelic_transaction_trace"
+            NewRelic::Control.instance.log.debug("Traced transaction: class = #{self.name}, method = #{method.to_s}, options = #{options.inspect}")
+          end
+        end
+
+        # Must be implemented in the controller class:
+        # Determine the path that is used in the metric name for
+        # the called controller action.  Of the form controller_path/action_name
+        #
+        def newrelic_metric_path(action_name_override = nil) # :nodoc:
+          raise "Not implemented!"
+        end
+
+        # Yield to the given block with NewRelic tracing.  Used by
+        # default instrumentation on controller actions in Rails and Merb.
+        # But it can also be used in custom instrumentation of controller
+        # methods and background tasks.
+        #
+        # This is the method invoked by instrumentation added by the
+        # <tt>ClassMethods#add_transaction_tracer</tt>.
+        #
+        # Here's a more verbose version of the example shown in
+        # <tt>ClassMethods#add_transaction_tracer</tt> using this method instead of
+        # #add_transaction_tracer.
+        #
+        # Below is a controller with an +invoke_operation+ action which
+        # dispatches to more specific operation methods based on a
+        # parameter (very dangerous, btw!).  With this instrumentation,
+        # the +invoke_operation+ action is ignored but the operation
+        # methods show up in New Relic as if they were first class controller
+        # actions
+        #
+        #   MyController < ActionController::Base
+        #     include NewRelic::Agent::Instrumentation::ControllerInstrumentation
+        #     # dispatch the given op to the method given by the service parameter.
+        #     def invoke_operation
+        #       op = params['operation']
+        #       perform_action_with_newrelic_trace(:name => op) do
+        #         send op, params['message']
+        #       end
+        #     end
+        #     # Ignore the invoker to avoid double counting
+        #     newrelic_ignore :only => 'invoke_operation'
+        #   end
+        #
+        #
+        # When invoking this method explicitly as in the example above, pass in a
+        # block to measure with some combination of options:
+        #
+        # * <tt>:category => :controller</tt> indicates that this is a
+        #   controller action and will appear with all the other actions.  This
+        #   is the default.
+        # * <tt>:category => :task</tt> indicates that this is a
+        #   background task and will show up in New Relic with other background
+        #   tasks instead of in the controllers list
+        # * <tt>:category => :rack</tt> if you are instrumenting a rack
+        #   middleware call.  The <tt>:name</tt> is optional, useful if you
+        #   have more than one potential transaction in the #call.
+        # * <tt>:category => :uri</tt> indicates that this is a
+        #   web transaction whose name is a normalized URI, where  'normalized'
+        #   means the URI does not have any elements with data in them such
+        #   as in many REST URIs.
+        # * <tt>:name => action_name</tt> is used to specify the action
+        #   name used as part of the metric name
+        # * <tt>:params => {...}</tt> to provide information about the context
+        #   of the call, used in transaction trace display, for example:
+        #   <tt>:params => { :account => @account.name, :file => file.name }</tt>
+        #   These are treated similarly to request parameters in web transactions.
+        #
+        # Seldomly used options:
+        #
+        # * <tt>:force => true</tt> indicates you should capture all
+        #   metrics even if the #newrelic_ignore directive was specified
+        # * <tt>:class_name => aClass.name</tt> is used to override the name
+        #   of the class when used inside the metric name.  Default is the
+        #   current class.
+        # * <tt>:path => metric_path</tt> is *deprecated* in the public API.  It
+        #   allows you to set the entire metric after the category part.  Overrides
+        #   all the other options.
+        # * <tt>:request => Rack::Request#new(env)</tt> is used to pass in a
+        #   request object that may respond to uri and referer.
+        #
+        # If a single argument is passed in, it is treated as a metric
+        # path.  This form is deprecated.
+        def perform_action_with_newrelic_trace(*args, &block)
+
+          # Skip instrumentation based on the value of 'do_not_trace' and if
+          # we aren't calling directly with a block.
+          if !block_given? && do_not_trace?
+            # Also ignore all instrumentation in the call sequence
+            NewRelic::Agent.disable_all_tracing do
+              return perform_action_without_newrelic_trace(*args)
+            end
+          end
+
+          return perform_action_with_newrelic_profile(args, &block) if NewRelic::Control.instance.profiling?
+
+          frame_data = _push_metric_frame(block_given? ? args : [])
+          begin
+            NewRelic::Agent.trace_execution_scoped frame_data.recorded_metrics, :force => frame_data.force_flag do
+              frame_data.start_transaction
+              begin
+                NewRelic::Agent::BusyCalculator.dispatcher_start frame_data.start
+                if block_given?
+                  yield
+                else
+                  perform_action_without_newrelic_trace(*args)
+                end
+              rescue Exception => e
+                frame_data.notice_error(e)
+                raise
+              end
+            end
+          ensure
+            NewRelic::Agent::BusyCalculator.dispatcher_finish
+            # Look for a metric frame in the thread local and process it.
+            # Clear the thread local when finished to ensure it only gets called once.
+            frame_data.record_apdex unless ignore_apdex?
+
+            frame_data.pop
+          end
+        end
+
+        protected
+        # Should be implemented in the dispatcher class
+        def newrelic_response_code; end
+
+        def newrelic_request_headers
+          self.respond_to?(:request) && self.request.respond_to?(:headers) && self.request.headers
+        end
+        
+        # overrideable method to determine whether to trace an action
+        # or not - you may override this in your controller and supply
+        # your own logic for ignoring transactions.
+        def do_not_trace?
+          _is_filtered?('do_not_trace')
+        end
+        
+        # overrideable method to determine whether to trace an action
+        # for purposes of apdex measurement - you can use this to
+        # ignore things like api calls or other fast non-user-facing
+        # actions
+        def ignore_apdex?
+          _is_filtered?('ignore_apdex')
+        end
+
+        private
+
+        # Profile the instrumented call.  Dev mode only.  Experimental
+        # - should definitely not be used on production applications
+        def perform_action_with_newrelic_profile(args)
+          frame_data = _push_metric_frame(block_given? ? args : [])
+          val = nil
+          NewRelic::Agent.trace_execution_scoped frame_data.metric_name do
+            MetricFrame.current(true).start_transaction
+            NewRelic::Agent.disable_all_tracing do
+              # turn on profiling
+              profile = RubyProf.profile do
+                if block_given?
+                  val = yield
+                else
+                  val = perform_action_without_newrelic_trace(*args)
+                end
+              end
+              NewRelic::Agent.instance.transaction_sampler.notice_profile profile
+            end
+          end
+          return val
+        ensure
+          frame_data.pop
+        end
+
+        # Write a metric frame onto a thread local if there isn't already one there.
+        # If there is one, just update it.
+        def _push_metric_frame(args) # :nodoc:
+          frame_data = NewRelic::Agent::Instrumentation::MetricFrame.current(true)
+
+          frame_data.apdex_start ||= _detect_upstream_wait(frame_data.start)
+          _record_queue_length
+          # If a block was passed in, then the arguments represent options for the instrumentation,
+          # not app method arguments.
+          if args.any?
+            if args.last.is_a?(Hash)
+              options = args.last
+              frame_data.force_flag = options[:force]
+              frame_data.request = options[:request] if options[:request]
+            end
+            category, path, available_params = _convert_args_to_path(args)
+          else
+            category = 'Controller'
+            path = newrelic_metric_path
+            available_params = self.respond_to?(:params) ? self.params : {}
+          end
+          frame_data.request ||= self.request if self.respond_to? :request
+          frame_data.push(category + '/'+ path)
+          frame_data.filtered_params = (respond_to? :filter_parameters) ? filter_parameters(available_params) : available_params
+          frame_data
+        end
+
+        def _convert_args_to_path(args)
+          options =  args.last.is_a?(Hash) ? args.pop : {}
+          params = options[:params] || {}
+          category = case options[:category]
+                     when :controller, nil then 'Controller'
+                     when :task then 'OtherTransaction/Background' # 'Task'
+                     when :rack then 'Controller/Rack' #'WebTransaction/Rack'
+                     when :uri then 'Controller' #'WebTransaction/Uri'
+                     when :sinatra then 'Controller/Sinatra' #'WebTransaction/Uri'
+                       # for internal use only
+                     else options[:category].to_s
+                     end
+          unless path = options[:path]
+            action = options[:name] || args.first
+            metric_class = options[:class_name] || (self.is_a?(Class) ? self.name : self.class.name)
+            path = metric_class
+            path += ('/' + action) if action
+          end
+          [category, path, params]
+        end
+
+        # Filter out a request if it matches one of our parameters for
+        # ignoring it - the key is either 'do_not_trace' or 'ignore_apdex'
+        def _is_filtered?(key)
+          ignore_actions = self.class.newrelic_read_attr(key) if self.class.respond_to? :newrelic_read_attr
+          case ignore_actions
+          when nil; false
+          when Hash
+            only_actions = Array(ignore_actions[:only])
+            except_actions = Array(ignore_actions[:except])
+            only_actions.include?(action_name.to_sym) || (except_actions.any? && !except_actions.include?(action_name.to_sym))
+          else
+            true
+          end
+        end
+        # Take a guess at a measure representing the number of requests waiting in mongrel
+        # or heroku.
+        def _record_queue_length
+          if newrelic_request_headers
+            if queue_depth = newrelic_request_headers['HTTP_X_HEROKU_QUEUE_DEPTH']
+              queue_depth = queue_depth.to_i rescue nil
+            elsif mongrel = NewRelic::Control.instance.local_env.mongrel
+              # Always subtrace 1 for the active mongrel
+              queue_depth = [mongrel.workers.list.length.to_i - 1, 0].max rescue nil
+            end
+            NewRelic::Agent.agent.stats_engine.get_stats_no_scope('Mongrel/Queue Length').trace_call(queue_depth) if queue_depth
+          end
+        end
+
+        include NewRelic::Agent::Instrumentation::QueueTime
+
+        # Return a Time instance representing the upstream start time.
+        # now is a Time instance to fall back on if no other candidate
+        # for the start time is found.
+        def _detect_upstream_wait(now)
+          queue_start = nil
+          if newrelic_request_headers
+            queue_start = parse_frontend_headers(newrelic_request_headers)
+            Thread.current[:newrelic_queue_time] = (now.to_f - queue_start.to_f) if queue_start
+          end
+          queue_start || now
+        rescue Exception => e
+          NewRelic::Control.instance.log.error("Error detecting upstream wait time: #{e}")
+          NewRelic::Control.instance.log.debug("#{e.backtrace[0..20]}")
+          now
+        end
+        
+        # returns the NewRelic::MethodTraceStats object associated
+        # with the dispatcher time measurement
+        def _dispatch_stat
+          NewRelic::Agent.agent.stats_engine.get_stats_no_scope 'HttpDispatcher'
+        end
+
+      end
+    end
+  end
+end

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

@@ -0,0 +1,57 @@
+# NewRelic instrumentation for DataMapper
+# For now, we have to refer to all db metrics as "ActiveRecord"
+if defined? DataMapper
+  
+  DataMapper::Model.class_eval do
+    add_method_tracer :get, 'ActiveRecord/#{self.name}/find'
+    add_method_tracer :first, 'ActiveRecord/#{self.name}/find'
+    add_method_tracer :first_or_create, 'ActiveRecord/#{self.name}/find'
+    add_method_tracer :all, 'ActiveRecord/#{self.name}/find_all'
+  end
+  DataMapper::Resource.class_eval do
+
+    @@my_sql_defined = defined? ActiveRecord::ConnectionAdapters::MysqlAdapter
+    @@postgres_defined = defined? ActiveRecord::ConnectionAdapters::PostgreSQLAdapter
+
+    for method in [:query] do
+      add_method_tracer method, 'ActiveRecord/#{self.class.name[/[^:]*$/]}/execute'
+      add_method_tracer method, 'ActiveRecord/all', :push_scope => false
+    end
+    for method in [:update, :save] do
+      add_method_tracer method, 'ActiveRecord/#{self.class.name[/[^:]*$/]}/save'
+      add_method_tracer method, 'ActiveRecord/save', :push_scope => false
+    end
+    add_method_tracer :destroy, 'ActiveRecord/#{self.class.name[/[^:]*$/]}/destroy'
+    add_method_tracer :destroy, 'ActiveRecord/destroy', :push_scope => false
+
+    def log_with_newrelic_instrumentation(sql, name, &block)
+      # if we aren't in a blamed context, then add one so that we can
+      # see that controllers are calling SQL directly we check
+      # scope_depth vs 2 since the controller is 1
+      if NewRelic::Agent.instance.transaction_sampler.scope_depth < 2
+        self.class.trace_method_execution "Database/DirectSQL", true, true do
+          log_with_capture_sql(sql, name, &block)
+        end
+      else
+        log_with_capture_sql(sql, name, &block)
+      end
+    end
+    
+    def log_with_capture_sql(sql, name, &block)
+      if @@my_sql_defined && self.is_a?(ActiveRecord::ConnectionAdapters::MysqlAdapter)
+        config = @config
+      elsif @@postgres_defined && self.is_a?(ActiveRecord::ConnectionAdapters::PostgreSQLAdapter)
+        config = @config
+      else
+        config = nil
+      end
+      
+      t0 = Time.now
+      result = log_without_newrelic_instrumentation(sql, name, &block)
+      
+      NewRelic::Agent.instance.transaction_sampler.notice_sql(sql, config, (Time.now - t0).to_f)
+      
+      result
+    end
+  end
+end

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

@@ -0,0 +1,52 @@
+require 'new_relic/agent/instrumentation/controller_instrumentation'
+
+DependencyDetection.defer do
+  @name = :delayed_job
+  
+  depends_on do
+    !NewRelic::Control.instance['disable_dj']
+  end
+
+  depends_on do
+    # double check because of old JRuby bug 
+    defined?(::Delayed) && defined?(::Delayed::Job) &&
+      Delayed::Job.method_defined?(:invoke_job)
+  end
+  
+  executes do
+    NewRelic::Agent.logger.debug 'Installing DelayedJob instrumentation'
+  end
+  
+  executes do
+    Delayed::Job.class_eval do
+      include NewRelic::Agent::Instrumentation::ControllerInstrumentation
+      if self.instance_methods.include?('name') || self.instance_methods.include?(:name)
+        add_transaction_tracer "invoke_job", :category => 'OtherTransaction/DelayedJob', :path => '#{self.name}'
+      else
+        add_transaction_tracer "invoke_job", :category => 'OtherTransaction/DelayedJob'
+      end
+    end
+  end
+
+  executes do
+    Delayed::Job.instance_eval do
+      if self.respond_to?('after_fork')
+        if method_defined?(:after_fork)
+          def after_fork_with_newrelic
+            NewRelic::Agent.after_fork(:force_reconnect => true)
+            after_fork_without_newrelic
+          end
+
+          alias_method :after_fork_without_newrelic, :after_fork
+          alias_method :after_fork, :after_fork_with_newrelic
+        else
+          def after_fork
+            NewRelic::Agent.after_fork(:force_reconnect => true)
+            super
+          end
+        end
+      end
+    end
+  end
+end
+

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

@@ -0,0 +1,80 @@
+# NOTE there are multiple implementations of the MemCache client in Ruby,
+# each with slightly different API's and semantics.
+# See:
+#     http://www.deveiate.org/code/Ruby-MemCache/ (Gem: Ruby-MemCache)
+#     http://seattlerb.rubyforge.org/memcache-client/ (Gem: memcache-client)
+#     http://github.com/mperham/dalli (Gem: dalli)
+
+module NewRelic
+  module Agent
+    module Instrumentation
+      module Memcache
+        module_function
+        def instrument_methods(the_class, method_names)
+          method_names.each do |method_name|
+            next unless the_class.method_defined? method_name.to_sym
+            the_class.class_eval <<-EOD
+              def #{method_name}_with_newrelic_trace(*args, &block)
+                metrics = ["Memcache/#{method_name}",
+                           (NewRelic::Agent::Instrumentation::MetricFrame.recording_web_transaction? ? 'Memcache/allWeb' : 'Memcache/allOther')]
+                self.class.trace_execution_scoped(metrics) do
+                  t0 = Time.now
+                  begin
+                    #{method_name}_without_newrelic_trace(*args, &block)
+                  ensure
+                    #{memcache_key_snippet(method_name)}
+                  end
+                end
+              end
+              alias #{method_name}_without_newrelic_trace #{method_name}
+              alias #{method_name} #{method_name}_with_newrelic_trace
+            EOD
+         end      
+        end
+        def memcache_key_snippet(method_name)
+          return "" unless NewRelic::Control.instance['capture_memcache_keys']
+          "NewRelic::Agent.instance.transaction_sampler.notice_nosql(args.first.inspect, (Time.now - t0).to_f) rescue nil"
+        end
+      end
+    end
+  end
+end
+
+DependencyDetection.defer do
+  @name = :memcache
+  
+  depends_on do
+    !NewRelic::Control.instance['disable_memcache_instrumentation']
+  end
+
+  depends_on do
+    defined?(::MemCache) || defined?(::Memcached) ||
+      defined?(::Dalli::Client) || defined?(::Spymemcached)
+  end
+    
+  executes do
+    commands = %w[get get_multi set add incr decr delete replace append prepend]
+    if defined? ::MemCache
+      NewRelic::Agent::Instrumentation::Memcache.instrument_methods(::MemCache,
+                                                                    commands)
+      NewRelic::Agent.logger.debug 'Installing MemCache instrumentation'
+    end
+    if defined? ::Memcached
+      commands << 'cas'
+      NewRelic::Agent::Instrumentation::Memcache.instrument_methods(::Memcached,
+                                                                    commands)
+      NewRelic::Agent.logger.debug 'Installing Memcached instrumentation'
+    end
+    if defined? ::Dalli::Client
+      NewRelic::Agent::Instrumentation::Memcache.instrument_methods(::Dalli::Client,
+                                                                    commands)
+      NewRelic::Agent.logger.debug 'Installing Dalli Memcache instrumentation'
+    end
+    if defined? ::Spymemcached
+      commands << 'multiget'
+      NewRelic::Agent::Instrumentation::Memcache.instrument_methods(::Spymemcached,
+                                                                    commands)
+      NewRelic::Agent.logger.debug 'Installing Spymemcached instrumentation'
+    end
+  end
+end

data/lib/new_relic/agent/instrumentation/merb/controller.rb

@@ -0,0 +1,41 @@
+require 'set'
+
+DependencyDetection.defer do
+  @name = :merb_controller
+  
+  depends_on do
+    defined?(Merb) && defined?(Merb::Controller)
+  end
+  
+  executes do
+    NewRelic::Agent.logger.debug 'Installing Merb Controller instrumentation'
+  end
+  
+  executes do
+    require 'merb-core/controller/merb_controller'
+
+    Merb::Controller.class_eval do
+      include NewRelic::Agent::Instrumentation::ControllerInstrumentation
+
+      class_inheritable_accessor :do_not_trace
+      class_inheritable_accessor :ignore_apdex
+
+      def self.newrelic_write_attr(attr_name, value) # :nodoc:
+        self.send "#{attr_name}=", attr_name, value
+      end
+
+      def self.newrelic_read_attr(attr_name) # :nodoc:
+        self.send attr_name
+      end
+
+      protected
+      # determine the path that is used in the metric name for
+      # the called controller action
+      def newrelic_metric_path
+        "#{controller_name}/#{action_name}"
+      end
+      alias_method :perform_action_without_newrelic_trace, :_dispatch
+      alias_method :_dispatch, :perform_action_with_newrelic_trace
+    end
+  end
+end

data/lib/new_relic/agent/instrumentation/merb/errors.rb

@@ -0,0 +1,29 @@
+DependencyDetection.defer do
+  @name = :merb_error
+
+  depends_on do
+    defined?(Merb) && defined?(Merb::Dispatcher) && defined?(Merb::Dispatcher::DefaultException)
+  end
+
+  depends_on do
+    Merb::Dispatcher::DefaultException.respond_to?(:before)
+  end
+  
+  executes do
+    NewRelic::Agent.logger.debug 'Installing Merb Errors instrumentation'
+  end
+  
+  executes do
+
+    # Hook in the notification to merb
+    error_notifier = Proc.new {
+      if request.exceptions #check that there's actually an exception
+        # Note, this assumes we have already captured the other information such as uri and params in the MetricFrame.
+        NewRelic::Agent::Instrumentation::MetricFrame.notice_error(request.exceptions.first)
+      end
+    }
+    Merb::Dispatcher::DefaultException.before error_notifier
+    Exceptions.before error_notifier
+
+  end
+end