appsignal 0.8.6 → 0.8.7.alpha.1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    MTdmNjgwM2IzMDBmNzE0ZGQxM2IwODRhMzdkM2RhZmVlOTIxNWEzMA==
+    MzE4ZGU1ZGJkN2Y2YmJiYTg4MjlkNjUwZDQ3Y2EzOTgyMzg2NDY0Zg==
   data.tar.gz: !binary |-
-    YTA1OWI5NjBmN2FmZmIwOGMxZjRkMmEwODkxODkwMDNiMDliMGI3YQ==
+    OGY1OTYxMzk4MTMyNjcyM2NhMWE0NDY3NDVjNjZhYWEwODU3ODM1Mw==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    ZWQ2ZTE0Y2I3NjM2MDljZGZmYmFlOWI3NmQ2MGNiNWJiODJhYWNjMWY3MjYx
-    MjQyN2ZlODQ2M2ZkZGI2MmMyZjIwYTcxNGFhZDk3OTFiZDg4NGVjNmFlMDdk
-    MGUxMzliYzdjNjA0YTQ4YWEwMDJjMTk2MzdiMDQ3NzQ4NzZiMzU=
+    ZGFmYzU4ZWUzNDg5NGMwNzU3ZmY3YWRmNTE0NTY0MGNkNWY3Yjk4MGI4MTYz
+    NDQ0MzdkNzdmOGFkNWNiYzUwYjlkNDE4NmFmY2VkNDIwNDkxZWFjNTNkYTU1
+    MTg4ZjUxZDdkMDI5NWI3ZDRmNWE4YzVjMWY0Y2IzMGY4OGJmM2I=
   data.tar.gz: !binary |-
-    Y2RjZmEzMDg0ZTJhYWFkMzMwYTQxZTE4ZGI0NzMzYzc5ODFjNjZlN2IzMDFi
-    NjljNTljNmYwMzU1NTFmNGU2ZTE4MWNhMzgzMjA1Y2M3ZGJlODg2NmQwMTAy
-    NzUzZTQ5ZGUzMGQ4ZjNiMDc5YTg4NjRiMWZkMjgwYzYxZjEzZWM=
+    NWNlNTIwNWQ1ZWQ3YWFiZjU1M2UzYzYzNTdjYmI4YTM3NzMyYjAyYjAxYTc3
+    NWM3N2ZiZmNhZDQwNjBhOTY3OGJiOTNhMzY4M2RmNjAxOTY5OWVjNWY3OWQ3
+    NWI3NjZiYjgzYzQ2NmViZDMzOWUyYjQ1OTBjZDhhMWViNWIxZGU=

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+# 0.8.7
+* Dup process action event to avoid threading issue
+
 # 0.8.6
 * Resque support (beta)
 * Support tags in Appsignal.send_exception

data/lib/appsignal/cli.rb

@@ -44,7 +44,7 @@ module Appsignal
         @config ||= Appsignal::Config.new(
           ENV['PWD'],
           options[:environment],
-          @initial_config,
+          initial_config,
           logger
         )
       end

data/lib/appsignal/transaction.rb

@@ -50,17 +50,17 @@
     end
 
     def set_process_action_event(event)
-      @process_action_event = event
       return unless event && event.payload
-      @action = "#{event.payload[:controller]}##{event.payload[:action]}"
+      @process_action_event = event.dup
+      @action = "#{@process_action_event.payload[:controller]}##{@process_action_event.payload[:action]}"
       @kind = 'http_request'
       set_http_queue_start
     end
 
     def set_perform_job_event(event)
-      @process_action_event = event
       return unless event && event.payload
-      @action = "#{event.payload[:class]}##{event.payload[:method]}"
+      @process_action_event = event.dup
+      @action = "#{@process_action_event.payload[:class]}##{@process_action_event.payload[:method]}"
       @kind = 'background_job'
       set_background_queue_start
     end
@@ -98,7 +98,9 @@
 
     def convert_values_to_primitives!
       Appsignal::Transaction::ParamsSanitizer.sanitize!(@process_action_event.payload) if @process_action_event
-      @events.each { |o| Appsignal::Transaction::ParamsSanitizer.sanitize!(o.payload) }
+      @events.map do |o|
+        Appsignal::Transaction::ParamsSanitizer.sanitize(o.payload)
+      end
       add_sanitized_context!
     end
 

data/lib/appsignal/version.rb

@@ -1,3 +1,3 @@
 module Appsignal
-  VERSION = '0.8.6'
+  VERSION = '0.8.7.alpha.1'
 end

data/spec/lib/appsignal/transaction_spec.rb

@@ -50,7 +50,8 @@ describe Appsignal::Transaction do
       let(:process_action_event) { notification_event }
 
       it 'should add a process action event' do
-        transaction.process_action_event.should == process_action_event
+        transaction.process_action_event.name.should == process_action_event.name
+        transaction.process_action_event.payload.should == process_action_event.payload
       end
 
       it "should set the action" do
@@ -77,8 +78,9 @@ describe Appsignal::Transaction do
         )
       end
 
-      it 'should add a process action event' do
-        transaction.process_action_event.should == perform_job_event
+      it 'should add a perform job event' do
+        transaction.process_action_event.name.should == perform_job_event.name
+        transaction.process_action_event.payload.should == perform_job_event.payload
       end
 
       it "should set the action" do
@@ -233,7 +235,7 @@ describe Appsignal::Transaction do
             transaction.convert_values_to_primitives!
           end
 
-          it { should == {:weird => weird_class.inspect} }
+          its([:weird]) { should be_a(Class) }
         end
       end
 

data/spec/lib/generators/appsignal/appsignal_generator_spec.rb

@@ -20,8 +20,15 @@ if rails_present?
     include GeneratorSpec::TestCase
     destination tmp_dir
 
+    let(:err_stream) { StringIO.new }
+
     before do
       FileUtils.rm_rf(tmp_dir)
+      @original_stderr = $stderr
+      $stderr = err_stream
+    end
+    after do
+      $stderr = @original_stderr
     end
 
     context "with key" do
@@ -109,6 +116,7 @@ if rails_present?
             no_file 'deploy.rb'
           end
         }
+        err_stream.string.should include "No value provided for required arguments 'push_api_key'"
       end
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 0.8.6
+  version: 0.8.7.alpha.1
 platform: ruby
 authors:
 - Robert Beekman
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-21 00:00:00.000000000 Z
+date: 2014-03-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -241,9 +241,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ! '>'
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.0.3