exception_notification-td 0.0.1.pre → 0.0.1.pre2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c29372e0364d9254aedfe11dd57cf4ea5b2539a2
-  data.tar.gz: 403dab98cd39902459865cd1e084cdce6fb71f27
+  metadata.gz: 592a064dbe40de85210df6e3c7c7312c73339be5
+  data.tar.gz: 2198d8c11f95c1d1585ac547ccfe97f0514d150f
 SHA512:
-  metadata.gz: 169a4a6927ddce4363086c09618345c441ef3870b14cc5b2a702bfc3e7c952c51abb8a328e925658201bc008e691698fa2df728af92f65f2d560ddbbe81d4f52
-  data.tar.gz: 46a6ca7e67186e9ee2afc61a0e9dec018ea0064569fdbe2d78d39d3dc5ca93d76ccb6e03a9523ca495d701cd1eaea360a44dc512210df7536be0b7c6c8784025
+  metadata.gz: 3585e51101db121217774782867a7b9ad4709e3237a40a900effb0189843975c8e0ed66d49625f1ad5f96c337620783f18cc74990fa6cb415115aca421b1033c
+  data.tar.gz: 81934771ad7de9632f0da2e61c4c38039eed11f9f6f0982d78885ee86ad5577819f93abc60bc21463ba156f5f037bcdade8c328e7d61a2ee0b147c4db6eb27cf

data/lib/exception_notification/td/version.rb

@@ -1,5 +1,5 @@
 module ExceptionNotification
   module Td
-    VERSION = "0.0.1.pre"
+    VERSION = "0.0.1.pre2"
   end
 end

data/lib/exception_notifier/td_notifier.rb

@@ -4,28 +4,62 @@ require "td-logger"
 
 module ExceptionNotifier
   class TdNotifier
+    BACKTRACE_LIMIT_DEFAULT = 10
+
     def initialize(options)
-      @database   = options.delete(:database)
       @table_name = options.delete(:table_name)
-      raise "Please set database and table_name. options: #{options.inspect}" if !@database or !@table_name
+      raise "Please set table_name. options: #{options.inspect}" unless @table_name
+
+      @backtrace_limit = options.delete(:backtrace_limit) || BACKTRACE_LIMIT_DEFAULT
+      @custom_param_proc = options.delete(:custom_param_proc)
 
-      TreasureData::Logger.open(@database, options)
+      unless defined? TreasureData::Logger::Agent::Rails
+        @database = options.delete(:database)
+        raise "Please set database. options: #{options.inspect}" unless @database
+        TreasureData::Logger.open(@database, options)
+      end
     end
-    attr_accessor :td
 
     def call(exception, options = {})
       TD.event.post(@table_name, exception_to_td_data(exception, options))
     end
 
+    private
+    def request_klass
+      @request_klass ||= if defined?(ActionDispatch::Request)
+                           ActionDispatch::Request
+                         else
+                           require 'rack/request'
+                           Rack::Request
+                         end
+    rescue LoadError, NameError
+      warn "ExceptionNotification::Td is designed to be used with Rack-based apps. Skip some of features."
+      nil
+    end
+
     def exception_to_td_data(exception, options)
+      backtrace = exception.backtrace ? exception.backtrace[0, @backtrace_limit] : []
       params = {
-        class: exception.class,
+        class: exception.class.to_s,
         message: exception.message,
-        backtrace: exception.backtrace,
+        backtrace: backtrace,
         hostname: (Socket.gethostname rescue nil),
         environment: Rails.env,
       }
-      params.merge!(request_env: options[:env]) if options[:env]
+      if request_klass && options[:env]
+        request = request_klass.new(options[:env])
+        params.merge!(
+          method: request.request_method,
+          request_url: request.url,
+          cookies: request.cookies,
+          referer: request.referer,
+        )
+        params[:post_body] = request.body unless request.get?
+
+      end
+      if @custom_param_proc
+        @custom_param_proc.call(params, exception, request)
+      end
       params
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: exception_notification-td
 version: !ruby/object:Gem::Version
-  version: 0.0.1.pre
+  version: 0.0.1.pre2
 platform: ruby
 authors:
 - Uchio KONDO