bugsnag 5.3.2 → 5.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ef6284041ebdca482f26c88e3c7567c66202c257
-  data.tar.gz: 08db78478dce28720b87e375bb4442ef67a59d13
+  metadata.gz: f0f59a73d8f4b8c5594828664573e892ecf8bd49
+  data.tar.gz: ec14896b7774fac186b1db8e42c5e41b233e4555
 SHA512:
-  metadata.gz: 8176288fe2c5a68ec57f47fc8b6ae8d654eba29753e2da0bc397a16709ce2b004f715d14ac25850b53215c20460bcdde805d0667e64df75a4b8c8018b4e5d543
-  data.tar.gz: ae3709d8217bcc5b8fa6654cbf8ca39d2cbb6585689a4ffd5dfa7e11ae782244680630dd307497e766b40361ba31150dfc00b9484f45538eb8d035ba39456f42
+  metadata.gz: 4a20576b59c6a146d2290a6a65a8ac13d30c75aaa76f0b2cfd216d0c7505b36af8a10b4f0269bc869e926b50c844cf435bd10634fc5c118e251f85b9d8bd3d59
+  data.tar.gz: 52fbb7da5f06de0f178c3c0d3b37c84cdbdcd79421923764a26dd6753b400fd509df3efe62862d51fe55a888d2b5db39eb711812e5470df82588d49b1bb03d05

data/CHANGELOG.md

@@ -1,6 +1,11 @@
 Changelog
 =========
 
+## 5.3.3 (16 June 2017)
+
+* [Rails] Fix failure to report when encountering objects which throw in `to_s`
+  [#361](https://github.com/bugsnag/bugsnag-ruby/pull/361)
+
 ## 5.3.2 (27 April 2017)
 
 ### Bug fixes

data/VERSION

@@ -1 +1 @@
-5.3.2
+5.3.3

data/lib/bugsnag/cleaner.rb

@@ -6,6 +6,7 @@ module Bugsnag
     FILTERED = '[FILTERED]'.freeze
     RECURSION = '[RECURSION]'.freeze
     OBJECT = '[OBJECT]'.freeze
+    RAISED = '[RAISED]'.freeze
 
     def initialize(filters)
       @filters = Array(filters)
@@ -43,7 +44,7 @@ module Bugsnag
       when String
         clean_string(obj)
       else
-        str = obj.to_s
+        str = obj.to_s rescue RAISED
         # avoid leaking potentially sensitive data from objects' #inspect output
         if str =~ /#<.*>/
           OBJECT

data/lib/bugsnag/middleware/rack_request.rb

@@ -1,5 +1,7 @@
 module Bugsnag::Middleware
   class RackRequest
+    SPOOF = "[SPOOF]".freeze
+
     def initialize(bugsnag)
       @bugsnag = bugsnag
     end
@@ -11,7 +13,7 @@ module Bugsnag::Middleware
         request = ::Rack::Request.new(env)
 
         params = request.params rescue {}
-
+        client_ip = request.ip.to_s rescue SPOOF
         session = env["rack.session"]
 
         # Set the context
@@ -51,7 +53,7 @@ module Bugsnag::Middleware
           :httpMethod => request.request_method,
           :params => params.to_hash,
           :referer => request.referer,
-          :clientIp => request.ip,
+          :clientIp => client_ip,
           :headers => headers
         })
 

data/lib/bugsnag/middleware/rails3_request.rb

@@ -1,5 +1,7 @@
 module Bugsnag::Middleware
   class Rails3Request
+    SPOOF = "[SPOOF]".freeze
+
     def initialize(bugsnag)
       @bugsnag = bugsnag
     end
@@ -8,6 +10,7 @@ module Bugsnag::Middleware
       if notification.request_data[:rack_env]
         env = notification.request_data[:rack_env]
         params = env["action_dispatch.request.parameters"]
+        client_ip = env["action_dispatch.remote_ip"].to_s rescue SPOOF
 
         if params
           # Set the context
@@ -22,11 +25,11 @@ module Bugsnag::Middleware
 
         # Use action_dispatch.remote_ip for IP address fields and send request id
         notification.add_tab(:request, {
-          :clientIp => env["action_dispatch.remote_ip"],
+          :clientIp => client_ip,
           :requestId => env["action_dispatch.request_id"]
         })
 
-        notification.user_id = env["action_dispatch.remote_ip"]
+        notification.user_id = client_ip
 
         # Add the rails version
         if notification.configuration.send_environment

data/spec/cleaner_spec.rb

@@ -82,6 +82,15 @@ describe Bugsnag::Cleaner do
       params = {:request => {:params => {:foo => {:bar => "baz"}}}}
       expect(described_class.new([/^foo\.bar/]).clean_object(params)).to eq({:request => {:params => {:foo => {:bar => '[FILTERED]'}}}})
     end
+
+    it "filters objects which can't be stringified" do
+      class StringRaiser
+        def to_s
+          raise 'Oh no you do not!'
+        end
+      end
+      expect(subject.clean_object({ :foo => StringRaiser.new })).to eq({ :foo => '[RAISED]' })
+    end
   end
 
   describe "#clean_url" do

data/spec/helper_spec.rb

@@ -39,6 +39,19 @@ describe Bugsnag::Helpers do
       expect(value[4]).to be_a(String)
     end
 
+    context "an object will throw if `to_s` is called" do
+      class StringRaiser
+        def to_s
+          raise 'Oh no you do not!'
+        end
+      end
+
+      it "uses the string '[RAISED]' instead" do
+        value = Bugsnag::Helpers.trim_if_needed([1, 3, StringRaiser.new])
+        expect(value[2]).to eq "[RAISED]"
+      end
+    end
+
     context "payload length is less than allowed" do
 
       it "does not change strings" do

data/spec/rails3_request_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe Bugsnag::Middleware::Rails3Request do
+  before(:each) do
+    Bugsnag.configuration.middleware.use(described_class)
+  end
+
+  describe "#call" do
+    it "sets request metadata" do
+      Bugsnag.configuration.set_request_data(:rack_env, {
+        "action_dispatch.remote_ip" => "10.2.2.224",
+        "action_dispatch.request_id" => "5",
+      })
+      Bugsnag.notify(BugsnagTestException.new('Grimbles'))
+
+      expect(Bugsnag).to have_sent_notification { |payload|
+        event = get_event_from_payload(payload)
+        puts event["metaData"].inspect
+        expect(event["metaData"]["request"]).to eq({
+          "clientIp" => "10.2.2.224",
+          "requestId" => "5"
+        })
+      }
+    end
+
+    context "the Remote IP will throw when serialized" do
+
+      it "sets the client IP metdata to [SPOOF]" do
+        class SpecialIP
+          def to_s
+            raise BugsnagTestException.new('oh no')
+          end
+        end
+        Bugsnag.configuration.set_request_data(:rack_env, {
+          "action_dispatch.remote_ip" => SpecialIP.new,
+          "action_dispatch.request_id" => "5",
+        })
+
+        Bugsnag.notify(BugsnagTestException.new('Grimbles'))
+
+        expect(Bugsnag).to have_sent_notification { |payload|
+          event = get_event_from_payload(payload)
+          puts event["metaData"].inspect
+          expect(event["metaData"]["request"]).to eq({
+            "clientIp" => "[SPOOF]",
+            "requestId" => "5"
+          })
+        }
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bugsnag
 version: !ruby/object:Gem::Version
-  version: 5.3.2
+  version: 5.3.3
 platform: ruby
 authors:
 - James Smith
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-27 00:00:00.000000000 Z
+date: 2017-06-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -171,6 +171,7 @@ files:
 - spec/middleware_spec.rb
 - spec/notification_spec.rb
 - spec/rack_spec.rb
+- spec/rails3_request_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/bugsnag/bugsnag-ruby
 licenses: