appsignal 0.10.1 → 0.10.2

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 859c45e8e3c4c18607db69172155bcba95894ff6
+  data.tar.gz: 2ce273fbee2435ff234f84caec78b8ff9f980940
+SHA512:
+  metadata.gz: 4fb5d774dfbd2dc99f8a2ac4a380dc22641eb7747797c3855311b88e40ea3ce60536456966a1da46f48453d6c33bc2ed21f24c960dda3d51f96779b9e57eb7b4
+  data.tar.gz: 01c652ae81255dca073ca584d3702fcfbc0642c8431d6efe6e3835a8217530020f35ebe55535a0f42881d73f886e75931f76b5184f0ba29054d65f616050b213

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+# 0.10.2
+* Bugfix in event payload sanitization
+
 # 0.10.1
 * Bugfix in event payload sanitization
 

data/lib/appsignal/transaction.rb

@@ -116,7 +116,9 @@
       return if have_values_been_converted_to_primitives?
       Appsignal::Transaction::ParamsSanitizer.sanitize!(@process_action_event.payload) if @process_action_event
       @events.map do |o|
-        Appsignal::Transaction::ParamsSanitizer.sanitize!(o.payload)
+        o.payload.each do |key, value|
+          o.payload[key] = Appsignal::Transaction::ParamsSanitizer.sanitize(value)
+        end
       end
       add_sanitized_context!
       @have_values_been_converted_to_primitives = true

data/lib/appsignal/version.rb

@@ -1,3 +1,3 @@
 module Appsignal
-  VERSION = '0.10.1'
+  VERSION = '0.10.2'
 end

data/spec/lib/appsignal/transaction_spec.rb

@@ -1,5 +1,11 @@
 require 'spec_helper'
 
+class Smash < Hash
+  def []=(key, val)
+    raise 'the roof'
+  end
+end
+
 describe Appsignal::Transaction do
   before :all do
     start_agent
@@ -251,30 +257,40 @@ describe Appsignal::Transaction do
       let(:action_event_payload) { transaction.process_action_event.payload }
       let(:event_payload) { transaction.events.first.payload }
       let(:weird_class) { Class.new }
+      let(:smash) { Smash.new.merge!(:foo => 'bar') }
 
       context "with values that need to be converted" do
         context "process action event payload" do
           subject { action_event_payload }
           before do
             action_event_payload.clear
-            action_event_payload.
-              merge!(:model => {:with => [:weird, weird_class]})
+            action_event_payload.merge!(
+              :model => {:with => [:weird, weird_class]},
+            )
             transaction.convert_values_to_primitives!
           end
 
-          it { should == {:model => {:with => [:weird, weird_class.inspect]}} }
+          it "should convert all payloads to primitives" do
+            should == {
+              :model => {:with => [:weird, weird_class.inspect]},
+            }
+          end
         end
 
         context "payload of events" do
           subject { event_payload }
           before do
             event_payload.clear
-            event_payload.merge!(:weird => weird_class)
+            event_payload.merge!(
+              :weird => weird_class,
+              :smash => smash
+              )
             transaction.convert_values_to_primitives!
           end
 
           its([:weird]) { should be_a(String) }
           its([:weird]) { should match(/#<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.10.1
-  prerelease: 
+  version: 0.10.2
 platform: ruby
 authors:
 - Robert Beekman
@@ -13,118 +12,104 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-14 00:00:00.000000000 Z
+date: 2014-08-15 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,8 +120,8 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .travis.yml
+- ".gitignore"
+- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE
@@ -239,27 +224,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.3
 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.2.2
 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