appsignal 0.8.6.beta.0 → 0.8.6.beta.1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YTFjZWJlMmZlZDk5YzFiZGZmN2E4ODZiYWNlNDM4ZWM0MGUzODBjMQ==
+    NzlhYWQ4OGJmZTVkOWRhYzIzNGQzNmU5YzJlZGRiNTEwMGMzMTZjNw==
   data.tar.gz: !binary |-
-    ZDBlMDAxZjMwNDdiN2VlMjBiM2Y5ZTUxMjQ0ZjFhY2M0ZGQ2MWEwMg==
+    NjY2MTUyOWFjMjUyMGRiNWZjYjNhYTc1MDQ1OTk2MjI1NDA4ZmI5OQ==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    NTM0NGE3Y2JlNTQ1NzI3YTQ3MzIwYjRmYTkxZmVmZWFiY2YxODMyYmNjMmYw
-    OTM1YmYzODQzMGJiNWNhZWQ0OTMyZTRkNDkwOWNjNTRkMjczNGNkYWNmMTY4
-    NWUyYjU4YWNkMzU1NjYwOGNiNjA1OGM5YmZjN2ViMzM4ODQyMTM=
+    YmMwOTdkMjBmM2RjY2M0Y2ViYmFhNzc3NGEwY2IyNzU5YzAyMmMzZWNkNDI2
+    ZTA0MGQzYTk0YzM0YWVjYTJmN2Q3NDMxNmYyNTgwMTVkYWM1ZmYzMWU3MDc2
+    ZmIxYTU1MmZkNTA4NjBiM2M5ZjkxZjcyYTYwYTI3MGUyZWVhNTE=
   data.tar.gz: !binary |-
-    MzE1MDg5ODE2YjJmMDkzMGNjY2I0NTk1ODhhNzBkYzZjMzZkZmE2NzZmMGM1
-    NTgyZTZhNjk4MTZjOTA3NzU1NGI1MmMzMDBkMzUwNjIxOThjYTBkNjhhZTUy
-    YTA3NGE2OTAyN2FjYTcxNDgxZjRmMTA0MzhiODhkOWFlMTE1NzA=
+    NGM3MWMwOTRmN2MxYmRiMWE5NmUyMzE5ODU1MDllOWQxMGZjZWU5YTMzMjcx
+    ZWIzMWNjYTQzZjdiMTg2MmVmNjZmNGMzZDFjNTc5ZWFjNWRkOGM1OTQ4Mzdh
+    ZTQxMmZkNGVjYzgyOWMwZjJiNjViMzY1OTRhZmY4NDdhZWU5NWQ=

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # 0.8.6
 * Resque support
+* Support tags in Appsignal.send_exception
+* Alias tag_request to tag_job, for background jobs
+* Skip sanitization of env if env is nil
+* Small bugfix in forking logic
 
 # 0.8.5
 * Don't require revision in CLI notify_of_deploy

data/lib/appsignal.rb

@@ -61,10 +61,11 @@ module Appsignal
       raise exception
     end
 
-    def send_exception(exception)
+    def send_exception(exception, tags=nil)
       return if is_ignored_exception?(exception)
       transaction = Appsignal::Transaction.create(SecureRandom.uuid, ENV.to_hash)
       transaction.add_exception(exception)
+      transaction.set_tags(tags) if tags
       transaction.complete!
       Appsignal.agent.send_queue
     end
@@ -81,6 +82,7 @@ module Appsignal
       return false unless transaction
       transaction.set_tags(params)
     end
+    alias :tag_job :tag_request
 
     def transactions
       @transactions ||= {}

data/lib/appsignal/agent.rb

@@ -62,9 +62,9 @@ module Appsignal
     end
 
     def enqueue(transaction)
+      forked! if @pid != Process.pid
       Appsignal.logger.debug('Enqueueing transaction')
       aggregator.add(transaction)
-      forked! if @pid != Process.pid
     end
 
     def send_queue

data/lib/appsignal/transaction.rb

@@ -167,6 +167,7 @@
     end
 
     def sanitize_environment!
+      return unless env
       env.each do |key, value|
         sanitized_environment[key] = value if ENV_METHODS.include?(key)
       end

data/lib/appsignal/version.rb

@@ -1,3 +1,3 @@
 module Appsignal
-  VERSION = '0.8.6.beta.0'
+  VERSION = '0.8.6.beta.1'
 end

data/spec/lib/appsignal/transaction_spec.rb

@@ -453,6 +453,12 @@ describe Appsignal::Transaction do
       before { transaction.send(:sanitize_environment!) }
 
       its(:keys) { should =~ whitelisted_keys }
+
+      context "when env is nil" do
+        let(:env) { nil }
+
+        it { should be_empty }
+      end
     end
 
     describe '#sanitize_tags!' do

data/spec/lib/appsignal_spec.rb

@@ -132,6 +132,10 @@ describe Appsignal do
           Appsignal.tag_request.should be_false
         end
       end
+
+      it "should also listen to tag_job" do
+        Appsignal.should respond_to(:tag_job)
+      end
     end
 
     describe ".transactions" do
@@ -253,6 +257,7 @@ describe Appsignal do
 
     describe ".send_exception" do
       before { Appsignal::Pipe.stub(:current => false) }
+      let(:tags) { nil }
 
       it "should send the exception to AppSignal" do
         agent = double(:shutdown => true)
@@ -263,6 +268,16 @@ describe Appsignal do
         Appsignal::Transaction.should_receive(:create).and_call_original
       end
 
+      context "with tags" do
+        let(:tags) { {:a => 'a', :b => 'b'} }
+
+        it "should tag the request before sending" do
+          transaction = Appsignal::Transaction.create(SecureRandom.uuid, {})
+          Appsignal::Transaction.stub(:create => transaction)
+          transaction.should_receive(:set_tags).with(tags)
+        end
+      end
+
       it "should not send the exception if it's in the ignored list" do
         Appsignal.stub(:is_ignored_exception? => true)
         Appsignal::Transaction.should_not_receive(:create)
@@ -272,7 +287,7 @@ describe Appsignal do
         begin
           raise "I am an exception"
         rescue Exception => e
-          Appsignal.send_exception(e)
+          Appsignal.send_exception(e, tags)
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 0.8.6.beta.0
+  version: 0.8.6.beta.1
 platform: ruby
 authors:
 - Robert Beekman
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-14 00:00:00.000000000 Z
+date: 2014-02-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport