appsignal 0.10.5 → 0.10.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b534937f18014e6942e96590872f412395b628c7
-  data.tar.gz: 59e4b4d4ad5bb0eec58a9814a26680cb055966e9
+  metadata.gz: 29ac953458214ed9268bd6d52afd40b5c45785cd
+  data.tar.gz: 858cfc04cd727f92934d36c6d1dbcda240404d85
 SHA512:
-  metadata.gz: 5345239997a144a05a572da7744cb3eb3c405931c3b3b37b0a4c1fcc1e49db7778abc22d39733ad711dd28ed1e535197ec9f0aa0dd1e5c346cd0db326a03bb0a
-  data.tar.gz: b144ad46eba9528a1d98c08dd6ba6fd701e443ae7425a8a34390bde2dfd20bf88204430d7d4c6a57ec98c6965e2dcc87cf04b847d82b65f686a877f5a9a3bbb9
+  metadata.gz: 8e70b79af43d968f5b18e5d526ffc79a4f31e1e7f147f828b47cd011dbcb4455b9ca3eeadfb9262628e1bc4be3a83cbd550a48c20f5ee245eea7dec264ab6132
+  data.tar.gz: 8e453b41199aef1fa956a701b3d8a929c36d6d9222e5f216106e929e15e76510b66d6be543943796c652ab6235b1aa27006216717ca32bccad548e493fffbc0c

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+# 0.10.6
+* Add config option to skip session data
+
 # 0.10.5
 * Don't shutdown in `at_exit`
 * Debug log about missing name in config

data/lib/appsignal/config.rb

@@ -7,12 +7,13 @@ module Appsignal
     include Appsignal::CarefulLogger
 
     DEFAULT_CONFIG = {
-      :ignore_exceptions => [],
-      :ignore_actions => [],
-      :send_params => true,
-      :endpoint => 'https://push.appsignal.com/1',
+      :ignore_exceptions      => [],
+      :ignore_actions         => [],
+      :send_params            => true,
+      :endpoint               => 'https://push.appsignal.com/1',
       :slow_request_threshold => 200,
-      :instrument_net_http => true
+      :instrument_net_http    => true,
+      :skip_session_data      => false
     }.freeze
 
     attr_reader :root_path, :env, :initial_config, :config_hash
@@ -45,8 +46,8 @@ module Appsignal
       end
       if config_hash && !config_hash[:name]
         @logger.debug(
-          "There's no name defined in the config, " \
-          "unless you're using the Heroku add-on you probably want to do this."
+          "There's no application name set in your config file. " \
+          "You should set one unless your app runs on Heroku."
         )
       end
     end

data/lib/appsignal/transaction.rb

@@ -206,7 +206,7 @@
     def sanitize_session_data!
       @sanitized_session_data = Appsignal::Transaction::ParamsSanitizer.sanitize(
         request.session.to_hash
-      )
+      ) if Appsignal.config[:skip_session_data] == false
       @fullpath = request.fullpath
     end
   end

data/lib/appsignal/version.rb

@@ -1,3 +1,3 @@
 module Appsignal
-  VERSION = '0.10.5'
+  VERSION = '0.10.6'
 end

data/spec/lib/appsignal/config_spec.rb

@@ -19,6 +19,7 @@ describe Appsignal::Config do
         :ignore_exceptions => [],
         :ignore_actions => [],
         :instrument_net_http => true,
+        :skip_session_data => false,
         :send_params => true,
         :endpoint => 'https://push.appsignal.com/1',
         :slow_request_threshold => 200,
@@ -122,7 +123,7 @@ describe Appsignal::Config do
 
       it "should merge with the default config, fill the config hash and log about the missing name" do
         Appsignal.logger.should_receive(:debug).with(
-          "There's no name defined in the config, unless you're using the Heroku add-on you probably want to do this."
+          "There's no application name set in your config file. You should set one unless your app runs on Heroku."
         )
 
         subject.config_hash.should == {
@@ -131,6 +132,7 @@ describe Appsignal::Config do
           :ignore_actions => [],
           :send_params => true,
           :instrument_net_http => true,
+          :skip_session_data => false,
           :endpoint => 'https://push.appsignal.com/1',
           :slow_request_threshold => 200,
           :active => true

data/spec/lib/appsignal/transaction_spec.rb

@@ -598,6 +598,18 @@ describe Appsignal::Transaction do
           end
         end
       end
+
+      context "when skipping session data" do
+        before do
+          Appsignal.config = {:skip_session_data => true}
+        end
+
+        it "does not pass the session data into the params sanitizer" do
+          Appsignal::ParamsSanitizer.should_not_receive(:sanitize)
+          subject
+          transaction.sanitized_session_data.should == {}
+        end
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 0.10.5
+  version: 0.10.6
 platform: ruby
 authors:
 - Robert Beekman
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-08 00:00:00.000000000 Z
+date: 2014-09-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack