mrhenry-failtale-reporter 0.0.5 → 0.1.0

data/LICENSE.txt

@@ -0,0 +1,22 @@
+Copyright (c) 2009 Mr.Henry
+
+Permission is hereby granted, free of charge, to any person
+obtaining a copy of this software and associated documentation
+files (the "Software"), to deal in the Software without
+restriction, including without limitation the rights to use,
+copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the
+Software is furnished to do so, subject to the following
+conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+OTHER DEALINGS IN THE SOFTWARE.

data/lib/failtale_reporter.rb

@@ -1,6 +1,6 @@
 
+require 'digest/sha1'
 begin
-  require 'digest/sha1'
   httparty = Dir.glob(File.dirname(__FILE__)+'/../vendor/httparty-*/lib/httparty.rb').last
   httparty ||= 'httparty'
   require httparty
@@ -34,15 +34,85 @@ module FailtaleReporter
       @default_reporter = reporter if reporter
       @default_reporter
     end
+    def application_root(path=nil)
+      @application_root = backtrace_cleaner_regexp(path) if path
+      @application_root
+    end
     def configure
       yield self
     end
+    def clean_backtrace(backtrace)
+      backtrace.collect do |line|
+        path = File.expand_path(line.split(':').first)
+        if File.exist?(path)
+          line = File.expand_path(line)
+          cleaned_line = nil
+          backtrace_cleaners.each do |proc|
+            cleaned_line = proc.call(line)
+            break if cleaned_line
+          end
+          cleaned_line || line
+        else
+          line
+        end
+      end
+    end
+    def backtrace_cleaners
+      @backtrace_cleaners ||= []
+    end
+    def backtrace_cleaner(&block)
+      backtrace_cleaners.push(block)
+    end
+    def backtrace_cleaner_regexp(path)
+      Regexp.new("^#{Regexp.escape(File.expand_path(path))}")
+    end
+    def collect_information(error, ctxs)
+      information_collectors.each do |proc|
+        proc.call(error, *ctxs)
+      end
+    end
+    def information_collectors
+      @information_collectors ||= []
+    end
+    def information_collector(&block)
+      information_collectors.push(block)
+    end
   end
   
   default_reporter 'ruby'
   
-  def self.report(error=nil, &block)
-    Client.new.report(error, &block)
+  backtrace_cleaner do |line|
+    if FailtaleReporter.application_root
+      line.sub! FailtaleReporter.application_root, "[APP]"
+    end
+  end
+  
+  if defined?(Gem)
+    class << self
+      def gem_backtrace_cleaner(spec)
+        @gem_backtrace_cleaner ||= {}
+        unless @gem_backtrace_cleaner[spec.full_name]
+          @gem_backtrace_cleaner[spec.full_name] = {
+            :regexp => backtrace_cleaner_regexp(spec.full_gem_path),
+            :label  => "[GEM: #{spec.name} @#{spec.version.to_s}]"
+          }
+        end
+        @gem_backtrace_cleaner[spec.full_name]
+      end
+    end
+    backtrace_cleaner do |line|
+      cleaned_line = nil
+      Gem.loaded_specs.values.each do |spec|
+        options = FailtaleReporter.gem_backtrace_cleaner(spec)
+        cleaned_line = line.sub!(options[:regexp], options[:label])
+        break if cleaned_line
+      end
+      cleaned_line
+    end
+  end
+  
+  def self.report(error=nil, *ctxs, &block)
+    Client.new.report(error, *ctxs, &block)
   end
   
 end

data/lib/failtale_reporter/adapters/rails.rb

@@ -18,11 +18,20 @@ module FailtaleReporter
         FailtaleReporter.configure do |config|
           config.ignored_exceptions IGNORED_EXCEPTIONS
           config.default_reporter "rails"
+          config.application_root RAILS_ROOT
+          config.information_collector do |error, controller|
+            env = error.environment
+            env = env.merge(controller.request.env)
+            
+            env.delete('action_controller.rescue.response')
+            env.delete('action_controller.rescue.request')
+            error.environment = env
+          end
         end
       end
       
       def rescue_action_in_public_with_failtale(exception)
-        FailtaleReporter.report(exception) unless is_private?
+        FailtaleReporter.report(exception, self) unless is_private?
         rescue_action_in_public_without_failtale(exception)
       end
       

data/lib/failtale_reporter/client.rb

@@ -2,11 +2,11 @@
 module FailtaleReporter
   class Client
     
-    def report(error=nil)
-      error = handle_exception(error)
+    def report(error=nil, *ctxs)
+      error = handle_exception(error, ctxs)
       yield if block_given? and error.nil?
     rescue Exception => exception
-      error = handle_exception(exception)
+      error = handle_exception(exception, ctxs)
     ensure
       post_report(error) unless error.nil?
       raise exception unless exception.nil?
@@ -14,12 +14,15 @@ module FailtaleReporter
     
     private
     
-    def handle_exception(exception)
+    def handle_exception(exception, ctxs)
       return exception if exception.is_a? FailtaleReporter::Error
       return nil if     exception.nil?
       return nil unless FailtaleReporter.reportable_exceptions.any? {|c| exception.is_a? c }
       return nil if     FailtaleReporter.ignored_exceptions.any?    {|c| exception.is_a? c }
-      FailtaleReporter::Error.new(exception)
+      FailtaleReporter::Error.new(exception, ctxs)
+    rescue Exception => e
+      puts "#{e.class}: #{e.message}"
+      puts e.backtrace
     end
     
     def post_report(error)

data/lib/failtale_reporter/error.rb

@@ -11,17 +11,26 @@ module FailtaleReporter
     attr_accessor :backtrace
     attr_accessor :environment
     
-    def initialize(exception)
+    def initialize(exception, ctxs=[])
       self.api_token = FailtaleReporter.api_token
       self.reporter  = FailtaleReporter.default_reporter
       
       self.name = "#{exception.class} #{exception.message}"
       self.description = exception.message
-      self.backtrace = exception.backtrace.join("\n")
+      self.backtrace = FailtaleReporter.clean_backtrace(exception.backtrace).join("\n")
       self.environment = ENV.to_hash
       self.hash_string = Digest::SHA1.hexdigest(
         [exception.class, exception.backtrace.first].join('--')
       )
+      FailtaleReporter.collect_information(self, ctxs)
+      
+      self.backtrace = self.backtrace.inspect unless self.backtrace.is_a? String 
+      self.environment = self.environment.inject({}) do |m,(k,v)|
+        k = k.inspect unless k.is_a? String
+        v = v.inspect unless v.is_a? String
+        m[k] = v
+        m
+      end
     end
     
     def to_param

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: mrhenry-failtale-reporter
 version: !ruby/object:Gem::Version 
-  version: 0.0.5
+  version: 0.1.0
 platform: ruby
 authors: 
 - Simon Menke
@@ -9,11 +9,12 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-02-10 00:00:00 -08:00
+date: 2009-02-18 00:00:00 -08:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: httparty
+  type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
@@ -23,6 +24,7 @@ dependencies:
     version: 
 - !ruby/object:Gem::Dependency 
   name: httparty
+  type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
@@ -49,6 +51,7 @@ files:
 - tasks/failtale.rake
 - spec/error_reporter_spec.rb
 - spec/error_spec.rb
+- LICENSE.txt
 - README.textile
 has_rdoc: false
 homepage: http://github.com/mrhenry