rollbar 0.10.11 → 0.10.12

checksums.yaml

@@ -0,0 +1,15 @@
+---
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    NWUyNTdhMGRhMTM0YWJkNjgyOGRlMmVmMTczY2U4ZmJmZTFjZjE1Zg==
+  data.tar.gz: !binary |-
+    M2M0YjEwNTJmYTYzMDczZjkxOTU0YThkZWUxZWVmOGQxYzE3N2YwMQ==
+!binary "U0hBNTEy":
+  metadata.gz: !binary |-
+    ZDBhYTFiZGNhOWE0YjVjZDI3NjJjMTFiZmViMzNhYjJmMzk4NTY1Njk1N2Vh
+    NmJmZjk0NmZkNGJkNTA0OTU0NGIwMGVmM2NmNTY1MThhZmYzZDA3MmM4NTAw
+    YjhlNzA5MjdjZjcyYTg5NTA3MDJkMThmYmQ1YWMyMGNjZGU3ZDM=
+  data.tar.gz: !binary |-
+    M2UwY2Y1OTc4ZjFkYzEzN2FjZDBlMjgyYWRiMGVlMjg4Zjk0YmM5YzEwMzEz
+    NWM5ODE5ZGQyZWFiY2UxZDExZTk3ODNhYWQ2NWE1MGNhM2Q2ZjIwODNkNWFi
+    OTRhMzUwM2ZkZTFiZjMwNDQ5MTJlNmUyMzljNjg2M2FiNDEwNGQ=

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Change Log
 
+**0.10.12**
+- Exclude HTTP_COOKIE header (since cookies are already included in parsed form)
+
 **0.10.11**
 - Fix usage of custom Sidekiq options
 

data/README.md

@@ -178,7 +178,7 @@ Rollbar.silenced {
 
 ## Asynchronous reporting
 
-By default, all messages are reported synchronously. You can enable asynchronous reporting with [girl_friday](https://github.com/mperham/girl_friday) or [Sidekiq](https://github.com/mperham/sidekiq).
+By default, all messages are reported synchronously. You can enable asynchronous reporting with [girl_friday](https://github.com/mperham/girl_friday) or [sucker_punch](https://github.com/brandonhilkert/sucker_punch) or [Sidekiq](https://github.com/mperham/sidekiq).
 
 ### Using girl_friday
 
@@ -190,6 +190,14 @@ config.use_async = true
 
 Asynchronous reporting falls back to Threading if girl_friday is not installed.
 
+### Using sucker_punch
+
+Add the following in ```config/initializers/rollbar.rb```:
+
+```ruby
+config.use_sucker_punch = true
+```
+
 ### Using Sidekiq
 
 Add the following in ```config/initializers/rollbar.rb```:

data/lib/generators/rollbar/templates/initializer.rb

@@ -37,6 +37,9 @@ Rollbar.configure do |config|
   #  Thread.new { Rollbar.process_payload(payload) }
   # }
 
+  # Enable asynchronous reporting (using sucker_punch)
+  # config.use_sucker_punch = true
+
   # Enable delayed reporting (using Sidekiq)
   # config.use_sidekiq = true
   # You can supply custom Sidekiq options:

data/lib/rollbar/request_data_extractor.rb

@@ -42,7 +42,12 @@ module Rollbar
     def rollbar_headers(env)
       env.keys.grep(/^HTTP_/).map do |header|
         name = header.gsub(/^HTTP_/, '').split('_').map(&:capitalize).join('-')
-        { name => env[header] }
+        # exclude cookies - we already include a parsed version as request_data[:cookies]
+        if name == 'Cookie'
+          {}
+        else
+          { name => env[header] }
+        end
       end.inject(:merge)
     end
 

data/lib/rollbar/version.rb

@@ -1,3 +1,3 @@
 module Rollbar
-  VERSION = "0.10.11"
+  VERSION = "0.10.12"
 end

data/spec/rollbar_spec.rb

@@ -648,6 +648,24 @@ describe Rollbar do
     end
   end
 
+  context "request_data_extractor" do
+    before(:each) do
+      class DummyClass
+      end
+      @dummy_class = DummyClass.new
+      @dummy_class.extend(Rollbar::RequestDataExtractor)
+    end
+    
+    context "rollbar_headers" do
+      it "should not include cookies" do
+        env = {"HTTP_USER_AGENT" => "test", "HTTP_COOKIE" => "cookie"}
+        headers = @dummy_class.send(:rollbar_headers, env)
+        headers.should have_key "User-Agent"
+        headers.should_not have_key "Cookie"
+      end
+    end
+  end
+
   # configure with some basic params
   def configure
     Rollbar.reconfigure do |config|

metadata

@@ -1,20 +1,18 @@
 --- !ruby/object:Gem::Specification
 name: rollbar
 version: !ruby/object:Gem::Version
-  version: 0.10.11
-  prerelease: 
+  version: 0.10.12
 platform: ruby
 authors:
 - Brian Rue
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-20 00:00:00.000000000 Z
+date: 2013-08-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -22,7 +20,6 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -30,7 +27,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -38,7 +34,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -46,7 +41,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -54,7 +48,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -62,7 +55,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -70,7 +62,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -78,7 +69,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: girl_friday
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -86,7 +76,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -94,7 +83,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: sucker_punch
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -102,7 +90,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -110,7 +97,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -118,7 +104,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -126,7 +111,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: genspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -134,7 +118,6 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -244,27 +227,26 @@ files:
 homepage: https://github.com/rollbar/rollbar-gem
 licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 2.0.7
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Reports exceptions to Rollbar
 test_files:
 - spec/controllers/home_controller_spec.rb