whoops_rails_logger 0.1.0 → 0.1.1

data/lib/whoops_rails_logger.rb

@@ -1,7 +1,5 @@
 require "digest"
 require "whoops_logger"
-require "whoops_rails_logger/action_controller_catcher"
-require "whoops_rails_logger/controller_methods"
 require "whoops_rails_logger/exception_strategy"
 require "whoops_rails_logger/rack"
 require "whoops_rails_logger/railtie"

data/lib/whoops_rails_logger/exception_strategy.rb

@@ -27,8 +27,16 @@ module WhoopsRailsLogger
         details[:backtrace] = exception.backtrace.collect{ |line|
           line.sub(/^#{ENV['GEM_HOME']}/, '$GEM_HOME').sub(/^#{Rails.root}/, '$Rails.root')
         }
-        details[:params]    = rack_env["action_dispatch.request.parameters"]
-        message.details      = details
+
+        details[:http_host]      = rack_env["HTTP_HOST"]        
+        details[:params]         = rack_env["action_dispatch.request.parameters"]
+        details[:query_string]   = rack_env["QUERY_STRING"]
+        details[:remote_addr]    = rack_env["REMOTE_ADDR"]
+        details[:request_method] = rack_env["REQUEST_METHOD"]
+        details[:server_name]    = rack_env["SERVER_NAME"]
+        details[:session]        = rack_env["rack.session"]
+        details[:env]            = ENV
+        message.details          = details
       end
 
       self.add_message_builder(:create_event_group_identifier) do |message, raw_data|

data/lib/whoops_rails_logger/rack.rb

@@ -14,7 +14,7 @@ module WhoopsRailsLogger
           :exception => raised,
           :rack_env  => env
         }
-        WhoopsLogger.notify(:rails_exception, raw_data)
+        WhoopsLogger.log(:rails_exception, raw_data)
         raise
       end
 

data/lib/whoops_rails_logger/railtie.rb

@@ -5,7 +5,6 @@ module WhoopsRailsLogger
     
     config.after_initialize do
       WhoopsRailsLogger.initialize
-      ::ActionController::Base.send(:include, WhoopsRailsLogger::ControllerMethods)
     end
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: whoops_rails_logger
 version: !ruby/object:Gem::Version 
-  hash: 27
+  hash: 25
   prerelease: false
   segments: 
   - 0
   - 1
-  - 0
-  version: 0.1.0
+  - 1
+  version: 0.1.1
 platform: ruby
 authors: 
 - Daniel Higginbotham
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-07-12 00:00:00 -04:00
+date: 2011-07-21 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -26,12 +26,12 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 27
+        hash: 31
         segments: 
         - 0
         - 1
-        - 0
-        version: 0.1.0
+        - 2
+        version: 0.1.2
   type: :runtime
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
@@ -145,8 +145,6 @@ extensions: []
 extra_rdoc_files: []
 
 files: 
-- lib/whoops_rails_logger/action_controller_catcher.rb
-- lib/whoops_rails_logger/controller_methods.rb
 - lib/whoops_rails_logger/evidence.rb
 - lib/whoops_rails_logger/exception_strategy.rb
 - lib/whoops_rails_logger/rack.rb

data/lib/whoops_rails_logger/action_controller_catcher.rb

@@ -1,19 +0,0 @@
-module WhoopsRailsLogger
-  module ActionControllerCatcher
-    # Sets up an alias chain to catch exceptions when Rails does
-    def self.included(base) #:nodoc:
-      # base.send(:alias_method, :rescue_action_in_public_without_whoops, :rescue_action_in_public)
-      # base.send(:alias_method, :rescue_action_in_public, :rescue_action_in_public_with_whoops)
-    end
-
-    private
-
-    # Overrides the rescue_action method in ActionController::Base, but does not inhibit
-    # any custom processing that is defined with Rails 2's exception helpers.
-    def rescue_action_in_public_with_whoops(exception)        
-      raw_data = {:exception => exception}.merge(whoops_request_data)
-       WhoopsRailsLogger.notify(raw_data)
-      rescue_action_in_public_without_whoops(exception)
-    end
-  end
-end

data/lib/whoops_rails_logger/controller_methods.rb

@@ -1,50 +0,0 @@
-module WhoopsRailsLogger
-  module ControllerMethods
-    private
-
-    # This method should be used for sending manual notifications while you are still
-    # inside the controller. Otherwise it works like WhoopsLoggerRails.notify.
-    def notify_whoops(exception)
-      raw_data = {:exception => exception}.merge(whoops_request_data)
-      WhoopsRailsLogger.notify(:rails, raw_data)
-    end
-
-    def whoops_request_data
-      { :parameters       => whoops_filter_if_filtering(params.to_hash),
-        :session_data     => whoops_filter_if_filtering(whoops_session_data),
-        :controller       => params[:controller],
-        :action           => params[:action],
-        :url              => whoops_request_url,
-        :cgi_data         => whoops_filter_if_filtering(request.env) }
-    end
-
-    def whoops_filter_if_filtering(hash)
-      return hash if ! hash.is_a?(Hash)
-
-      if respond_to?(:filter_parameters)
-        filter_parameters(hash) rescue hash
-      else
-        hash
-      end
-    end
-
-    def whoops_session_data
-      if session.respond_to?(:to_hash)
-        session.to_hash
-      else
-        session.data
-      end
-    end
-
-    def whoops_request_url
-      url = "#{request.protocol}#{request.host}"
-
-      unless [80, 443].include?(request.port)
-        url << ":#{request.port}"
-      end
-
-      url << request.request_uri
-      url
-    end
-  end
-end