rollbar 0.12.9 → 0.12.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8af9c1cd4fe9adc621acb5a44240c0b61a2a2177
-  data.tar.gz: 57a3478b9a9299769e04de690e43ece7c81ab836
+  metadata.gz: 88ac00441d336613c80d821e682e4e8c5250cbd4
+  data.tar.gz: d6441c75660074ac8aa6b2108cd2a6f20bff35d8
 SHA512:
-  metadata.gz: 3f77c6c4fae2a8d82ee381762d14fcceedb9ad8c764100a37dcec330858e304a21938b27869c53f07d070dbdc30bb9d766034756ba5477f2b6fb6b9295b36e14
-  data.tar.gz: 82112483b0e51c31c6646de253ad349e7246b4d3420ad58e5d1cb928a87e6f3395a601afccd9cf531f0b29c3097c1974c8f3107a1a042961de2f89d04d197323
+  metadata.gz: 68bbecf026c56a3320a84cf58af48a5ef9b5d42b7c13e10fe5c0ceba680419d6fb61ce5babcffb51b92e17449d8eb6c1db65eab86b6a4d263393f4d178938ab9
+  data.tar.gz: c5798afb37d2f45b8de7b05c6052d97742bc5a31ecd78e850bcdb5d1326426a26e0f2c7c392d9d450c0ddfdadc651a4597e054a90409c4b9d17a8195e22b6549

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change Log
 
+**0.12.10**
+- Log payloads that are too large to be sent to Rollbar
+- Don't record controller context if request route info isn't readily available (ex. non-Rails)
+
 **0.12.9**
 - Fixed delayed job regression introduced in 0.12.5 by re-raising caught exceptions
 - Removed Active Support call introduced in 0.12.6 to remove rails dependency in `report_exception`

data/LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2013 Rollbar, Inc.
+Copyright (c) 2014 Rollbar, Inc.
 
 MIT License
 

data/lib/rollbar.rb

@@ -88,7 +88,12 @@ module Rollbar
       data = exception_data(exception, level ? level : filtered_level(exception))
       if request_data
         if request_data[:route]
-          data[:context] = "#{request_data[:route][:controller]}" + '#' + "#{request_data[:route][:action]}"
+          route = request_data[:route]
+          
+          # make sure route is a hash built by RequestDataExtractor in rails apps
+          if route.is_a?(Hash) and not route.empty?
+            data[:context] = "#{request_data[:route][:controller]}" + '#' + "#{request_data[:route][:action]}"
+          end
         end
         
         request_data[:env].reject!{|k, v| v.is_a?(IO) } if request_data[:env]
@@ -368,6 +373,7 @@ module Rollbar
             break
           elsif i == thresholds.length - 1
             send_failsafe('Could not send payload due to it being too large after truncating attempts', nil)
+            log_error "[Rollbar] Payload too large to be sent: #{MultiJson.dump(payload)}"
             return
           end
         end

data/lib/rollbar/request_data_extractor.rb

@@ -99,12 +99,16 @@ module Rollbar
     end
     
     def rollbar_route_params(env)
-      route = ::Rails.application.routes.recognize_path(env['PATH_INFO']) rescue {}
-      {
-        :controller => route[:controller],
-        :action => route[:action],
-        :format => route[:format]
-      }
+      begin
+        route = ::Rails.application.routes.recognize_path(env['PATH_INFO'])
+        {
+          :controller => route[:controller],
+          :action => route[:action],
+          :format => route[:format]
+        }
+      rescue
+        {}
+      end
     end
     
     def rollbar_request_cookies(rack_req)

data/lib/rollbar/version.rb

@@ -1,3 +1,3 @@
 module Rollbar
-  VERSION = "0.12.9"
+  VERSION = "0.12.10"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rollbar
 version: !ruby/object:Gem::Version
-  version: 0.12.9
+  version: 0.12.10
 platform: ruby
 authors:
 - Brian Rue
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-20 00:00:00.000000000 Z
+date: 2014-02-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json