error_env_parser.rb in errdo-0.11.0

- old
+ new
@@ -1,59 +1,76 @@
 module Errdo
   module Models
     class ErrorEnvParser
 
-      attr_accessor :user
+      attr_accessor :user, :exception_class_name, :exception_message, :http_method,
+                    :host_name, :url, :backtrace, :ip, :user_agent, :referer,
+                    :query_string, :param_values, :cookie_values, :header_values,
+                    :experiencer_id, :experiencer_type
 
       def initialize(env, user_parser)
-        @env = env
-        @request = ActionDispatch::Request.new(env)
-        @controller = @env["action_controller.instance"]
-        @user = user_parser.user
+        set_accessible_params(
+          env,
+          ActionDispatch::Request.new(env),
+          env["action_controller.instance"],
+          user_parser.user
+        )
       end
 
       def error_hash
         {
-          exception_class_name: @env["action_dispatch.exception"].class.to_s,
-          exception_message:    @env["action_dispatch.exception"].try(:message),
-          http_method:          @request.try(:request_method),
-          host_name:            @request.try(:server_name),
-          url:                  @request.try(:original_url),
-          backtrace:            prepare_backtrace(@env)
+          exception_class_name:   @exception_class_name,
+          exception_message:      @exception_message,
+          http_method:            @http_method,
+          host_name:              @host_name,
+          url:                    @url,
+          backtrace:              @backtrace,
+          last_experiencer_id:    @experiencer_id,
+          last_experiencer_type:  @experiencer_type
         }
       end
 
       def error_occurrence_hash
         {
-          ip:                   @request.try(:ip),
-          user_agent:           @request.try(:user_agent),
-          referer:              @request.try(:referer),
-          query_string:         @request.try(:query_string),
-          param_values:         scrubbed_params(@request),
-          cookie_values:        @request.try(:cookies),
-          header_values:        @controller.try(:headers),
-          experiencer_id:       @user.try(:id),
-          experiencer_type:     @user.try(:class).try(:name)
+          ip:                   @ip,
+          user_agent:           @user_agent,
+          referer:              @referer,
+          query_string:         @query_string,
+          param_values:         @param_values,
+          cookie_values:        @cookie_values,
+          header_values:        @header_values,
+          experiencer_id:       @experiencer_id,
+          experiencer_type:     @experiencer_type
         }
       end
 
       def exception_name
-        @env["action_dispatch.exception"].class.to_s
+        @exception_class_name
       end
 
-      def exception_message
-        @env["action_dispatch.exception"].try(:message)
-      end
-
-      def backtrace
-        prepare_backtrace(@env)
-      end
-
       def short_backtrace
-        backtrace.first if backtrace.respond_to?(:first)
+        @backtrace.first if @backtrace.respond_to?(:first)
       end
 
       private
+
+      def set_accessible_params(env, request, controller, user)
+        @exception_class_name = env["action_dispatch.exception"].class.to_s
+        @exception_message =    env["action_dispatch.exception"].try(:message)
+        @http_method =          request.try(:request_method)
+        @host_name =            request.try(:server_name)
+        @url =                  request.try(:original_url)
+        @backtrace =            prepare_backtrace(env)
+        @ip =                   request.try(:ip)
+        @user_agent =           request.try(:user_agent)
+        @referer =              request.try(:referer)
+        @query_string =         request.try(:query_string)
+        @param_values =         scrubbed_params(request)
+        @cookie_values =        request.try(:cookies)
+        @header_values =        controller.try(:headers)
+        @experiencer_id =       user.try(:id)
+        @experiencer_type =     user.try(:class).try(:name)
+      end
 
       def prepare_backtrace(env)
         env["action_dispatch.exception"].try(:backtrace)
       end
 