appsignal 0.11.11 → 0.11.12

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 0ba213178a250bec23014f41561ccaa3c4ca2c95
+  data.tar.gz: f9648371cd893047a5dcf4db1467aeff67ba4f97
+SHA512:
+  metadata.gz: 0d53da1203003488e9213d6fe9c9afbe3608419c95ab8c17d829a402ee4d60ff922a445893e63db1da1b27fd6b783bfe29aa85e7eb708a37ddacd6a6dc944421
+  data.tar.gz: 1e86ea7f96f1b6fb437e100acb158e20edd8ca0ebbd963abc5a66235871453eca4e40c00ba4e4458dc5b47e97ad01f8f0665b0eaf9203b81f4a6d387ba042f04

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+# 0.11.12
+* Sanitizer will no longer inspect unknown objects, since implementations of inspect sometimes trigger unexpected behavior.
+
 # 0.11.11
 * Reliably get errors in production for Sinatra
 

data/lib/appsignal/params_sanitizer.rb

@@ -51,10 +51,7 @@ module Appsignal
       end
 
       def inspected(value)
-        value.inspect
-      rescue
-        # It turns out that sometimes inspect can fail
-        "#<#{value.class.to_s}/>"
+        "#<#{value.class.to_s}>"
       end
     end
   end

data/lib/appsignal/version.rb

@@ -1,3 +1,3 @@
 module Appsignal
-  VERSION = '0.11.11'
+  VERSION = '0.11.12'
 end

data/spec/lib/appsignal/params_sanitizer_spec.rb

@@ -6,6 +6,12 @@ class ErrorOnInspect
   end
 end
 
+class ClassWithInspect
+  def inspect
+    "#<ClassWithInspect foo=\"bar\"/>"
+  end
+end
+
 describe Appsignal::ParamsSanitizer do
   let(:klass) { Appsignal::ParamsSanitizer }
   let(:file) { uploaded_file }
@@ -23,13 +29,14 @@ describe Appsignal::ParamsSanitizer do
             :key => 'value',
             :file => file,
           },
-          ErrorOnInspect.new
+          ErrorOnInspect.new,
+          ClassWithInspect.new
         ]
       }
     }
   end
   let(:sanitized_params) { klass.sanitize(params) }
-  let(:scrubbed_params) { klass.scrub(params) }
+  let(:scrubbed_params)  { klass.scrub(params) }
 
   describe ".sanitize!" do
     subject { params }
@@ -54,7 +61,8 @@ describe Appsignal::ParamsSanitizer do
         its([1]) { should == 'else' }
         its([2]) { should be_instance_of String }
         its([2]) { should include '::UploadedFile' }
-        its([4]) { should == '#<ErrorOnInspect/>' }
+        its([4]) { should == '#<ErrorOnInspect>' }
+        its([5]) { should == '#<ClassWithInspect>' }
 
         context "nested hash" do
           subject { params[:hash][:nested_array][3] }

data/spec/lib/appsignal/transaction_spec.rb

@@ -365,7 +365,7 @@ describe Appsignal::Transaction do
 
           it "should convert all payloads to primitives" do
             should == {
-              :model => {:with => [:weird, weird_class.inspect]},
+              :model => {:with => [:weird, '#<Class>']},
             }
           end
         end
@@ -382,7 +382,7 @@ describe Appsignal::Transaction do
           end
 
           its([:weird]) { should be_a(String) }
-          its([:weird]) { should match(/#<Class:(.*)>/) }
+          its([:weird]) { should eql("#<Class>") }
           its([:smash]) { should == {:foo => 'bar'} }
         end
       end

metadata

@@ -1,8 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 0.11.11
-  prerelease: 
+  version: 0.11.12
 platform: ruby
 authors:
 - Robert Beekman
@@ -13,118 +12,104 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-25 00:00:00.000000000 Z
+date: 2015-06-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: thread_safe
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 2.14.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 2.14.1
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: The official appsignal.com gem
@@ -135,9 +120,9 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
-- .travis.yml
+- ".gitignore"
+- ".rspec"
+- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE
@@ -259,27 +244,26 @@ files:
 homepage: https://github.com/appsignal/appsignal
 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: '1.9'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.4.5
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Logs performance and exception data from your app to appsignal.com
 test_files:
 - spec/lib/appsignal/agent_spec.rb