appsignal 0.9.3 → 0.9.4

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 87281056af89689a3f66010e44c4f3a2791f137b
+  data.tar.gz: 2c0011b3c7b98bcf39ed208ad14036ed964606c4
+SHA512:
+  metadata.gz: 6c220755d2e9ea83500fa489db7d95bf446cfaf9b15d781f3043d3bbe6208041ef46a528a5d42fe55ccc3526e4acbaacd6783ece791f7b261f9b83cab8127b16
+  data.tar.gz: 78ca1bcc64361c136879c5f1ae7f136e8718f166abd39a453b90ca274b3c5722b582dd527d0dc5f642d1292beb50777aa20aae4abe91826d4055e10c902b74cf

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# 0.9.4
+* Log Rails and Sinatra version
+* Resubscribe to notifications after fork
+
 # 0.9.3
 * Log if appsignal is not active for an environment
 

data/lib/appsignal/agent.rb

@@ -67,6 +67,18 @@ module Appsignal
       end
     end
 
+    def resubscribe
+      Appsignal.logger.debug('Resubscribing to notifications')
+      unsubscribe
+      subscribe
+    end
+
+    def unsubscribe
+      Appsignal.logger.debug('Unsubscribing from notifications')
+      ActiveSupport::Notifications.unsubscribe(@subscriber)
+      @subscriber = nil
+    end
+
     def enqueue(transaction)
       forked! if @pid != Process.pid
       aggregator.add(transaction)
@@ -107,12 +119,13 @@ module Appsignal
       Thread.exclusive do
         @aggregator = Aggregator.new
       end
+      resubscribe
       restart_thread
     end
 
     def shutdown(send_current_queue=false, reason=nil)
       Appsignal.logger.info("Shutting down agent (#{reason})")
-      ActiveSupport::Notifications.unsubscribe(subscriber)
+      unsubscribe
       Thread.kill(thread) if thread
       send_queue if send_current_queue && @aggregator.has_transactions?
     end

data/lib/appsignal/integrations/rails.rb

@@ -1,5 +1,5 @@
 if defined?(::Rails)
-  Appsignal.logger.info('Loading Rails integration')
+  Appsignal.logger.info("Loading Rails (#{Rails.version}) integration")
 
   module Appsignal
     module Integrations

data/lib/appsignal/integrations/sinatra.rb

@@ -1,6 +1,6 @@
 require 'appsignal'
 
-Appsignal.logger.info('Loading Sinatra integration')
+Appsignal.logger.info("Loading Sinatra (#{Sinatra::VERSION}) integration")
 
 app_settings = ::Sinatra::Application.settings
 Appsignal.config = Appsignal::Config.new(

data/lib/appsignal/transaction.rb

@@ -136,7 +136,7 @@
           Appsignal.enqueue(self)
         end
       else
-        Appsignal.logger.debug("Not processing transaction: #{@request_id}")
+        Appsignal.logger.debug("Not processing transaction: #{@request_id} (#{events.length} events recorded)")
       end
     ensure
       Appsignal.transactions.delete(@request_id)

data/lib/appsignal/version.rb

@@ -1,3 +1,3 @@
 module Appsignal
-  VERSION = '0.9.3'
+  VERSION = '0.9.4'
 end

data/spec/lib/appsignal/agent_spec.rb

@@ -83,11 +83,8 @@ describe Appsignal::Agent do
   end
 
   describe "#restart_thread" do
-    it "should stop thread" do
+    it "should stop and start the thread" do
       subject.should_receive(:stop_thread)
-    end
-
-    it "should start a thread" do
       subject.should_receive(:start_thread)
     end
 
@@ -135,7 +132,7 @@ describe Appsignal::Agent do
     end
 
     context "handling events" do
-      before(:each) do
+      before :each do
         # Unsubscribe previous notification subscribers
         ActiveSupport::Notifications.notifier.instance_variable_get(:@subscribers).
           reject { |sub| sub.instance_variable_get(:@pattern).is_a? String }.
@@ -193,6 +190,31 @@ describe Appsignal::Agent do
         ActiveSupport::Notifications.instrument 'perform_job.processor'
       end
     end
+
+    describe "#unsubscribe" do
+      before :each do
+        Appsignal.agent.unsubscribe
+      end
+
+      it "should not have a subscriber" do
+        Appsignal.agent.subscriber.should be_nil
+      end
+
+      it "should add a normal event" do
+        Appsignal::Transaction.current.should_not_receive(:add_event)
+
+        ActiveSupport::Notifications.instrument 'moo'
+      end
+    end
+  end
+
+  describe "#resubscribe" do
+    it "should stop and start the thread" do
+      subject.should_receive(:unsubscribe)
+      subject.should_receive(:subscribe)
+    end
+
+    after { subject.resubscribe }
   end
 
   describe "#enqueue" do
@@ -272,6 +294,7 @@ describe Appsignal::Agent do
       subject.pid.should == master_pid
 
       Process.stub(:pid => 9000000001)
+      subject.should_receive(:resubscribe)
       subject.should_receive(:restart_thread)
       previous_aggregator = subject.aggregator
 

metadata

@@ -1,8 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 0.9.3
-  prerelease: 
+  version: 0.9.4
 platform: ruby
 authors:
 - Robert Beekman
@@ -13,134 +12,118 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-24 00:00:00.000000000 Z
+date: 2014-08-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
 - !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
@@ -151,8 +134,8 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .travis.yml
+- ".gitignore"
+- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE
@@ -251,27 +234,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