appsignal 1.1.5.beta.1 → 1.1.5.beta.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cfdc08fdd42809a6cffc841c28a20596d2db3f3d
-  data.tar.gz: 9f892def0286d4b5cbcb247c31132cee44b69def
+  metadata.gz: 4ce59be8afa0036d43feda6d0448b7f425331710
+  data.tar.gz: fa7a0f18be1dc0bff11b6361df21af99dabca3bd
 SHA512:
-  metadata.gz: fed1598d4d05ec88e9eeb400eb53b94af578b3c0d7184ecb3213d1c517250f53b44fe3cee4ef63c1b9a2a8d509f66439f10ba8c2c089da7cdee84922493d70d7
-  data.tar.gz: 1ea337a28fbc4817b820aa0cd78d9767f9f8a1cf77dccaa1ffd56cb8a4106a880b0c8ef05c5ae7a49f877869597205a1da44f41e98ad60f1757a690826d6f518
+  metadata.gz: 4223ca5924dbe22903844f8231e55cf9676565b4516d2a4760945cacfb32b16ea235eb00d9ce18e621cf930665bf0212da029068058eea15a672f4b9a4188398
+  data.tar.gz: 9a6c39eb3ca679bf9e3bda808dc67091b366f6c796a0fa7fc2c9d58265c34721e290e9453f94a26d72d14c1075250dc2f8810084676e0b59047fbfedd1c9d643

data/CHANGELOG.md

@@ -3,6 +3,7 @@
 * Add require to deploy.rb if present on installation
 * Warn when overwriting already existing transaction
 * Support for x86-linux
+* Some improvements in debug logging
 
 # 1.1.4
 * Better debug logging for agent issues

data/ext/agent.yml

@@ -1,19 +1,19 @@
 ---
-version: 1776f03
+version: '4201306'
 triples:
   x86_64-linux:
-    checksum: e7762cef292e9449e4d86f833ee7a80651845c3847122b4d4d995fb171462697
-    download_url: https://appsignal-agent-releases.global.ssl.fastly.net/1776f03/appsignal-agent-x86_64-linux-static.tar.gz
+    checksum: 29dc30be5de25655e2649f6f41266342483afca95a816d729da8be18c98341d7
+    download_url: https://appsignal-agent-releases.global.ssl.fastly.net/4201306/appsignal-agent-x86_64-linux-static.tar.gz
     lib_filename: libappsignal.a
   i686-linux:
-    checksum: 0407af36bc2d417a6fa5533f75f442489b71c4df7dc744421e199b2949d57d1f
-    download_url: https://appsignal-agent-releases.global.ssl.fastly.net/1776f03/appsignal-agent-i686-linux-static.tar.gz
+    checksum: 4563916bd95f8d6942c15926583a3ee3979d71d2c606788dcb41369e6f495205
+    download_url: https://appsignal-agent-releases.global.ssl.fastly.net/4201306/appsignal-agent-i686-linux-static.tar.gz
     lib_filename: libappsignal.a
   x86-linux:
-    checksum: 0407af36bc2d417a6fa5533f75f442489b71c4df7dc744421e199b2949d57d1f
-    download_url: https://appsignal-agent-releases.global.ssl.fastly.net/1776f03/appsignal-agent-i686-linux-static.tar.gz
+    checksum: 4563916bd95f8d6942c15926583a3ee3979d71d2c606788dcb41369e6f495205
+    download_url: https://appsignal-agent-releases.global.ssl.fastly.net/4201306/appsignal-agent-i686-linux-static.tar.gz
     lib_filename: libappsignal.a
   x86_64-darwin:
-    checksum: 5438333a6402ca07a606387086e4206a7fd7b5d884a291ccf44f9dffd39bd7c3
-    download_url: https://appsignal-agent-releases.global.ssl.fastly.net/1776f03/appsignal-agent-x86_64-darwin-static.tar.gz
+    checksum: 7ca9e9133d3c3876380ad1e682a8d0924c2857205cd4946012adf39dfd88a8bf
+    download_url: https://appsignal-agent-releases.global.ssl.fastly.net/4201306/appsignal-agent-x86_64-darwin-static.tar.gz
     lib_filename: libappsignal.a

data/lib/appsignal.rb

@@ -29,7 +29,12 @@ module Appsignal
     end
 
     def start
-      return unless extension_loaded?
+      unless extension_loaded?
+        logger.info('Not starting appsignal, extension is not loaded')
+        return
+      else
+        logger.debug('Starting appsignal')
+      end
 
       unless @config
         @config = Config.new(
@@ -62,7 +67,12 @@ module Appsignal
       end
     end
 
-    def stop
+    def stop(called_by=nil)
+      if called_by
+        logger.debug("Stopping appsignal (#{called_by})")
+      else
+        logger.debug('Stopping appsignal')
+      end
       Appsignal::Extension.stop
     end
 

data/lib/appsignal/hooks/celluloid.rb

@@ -18,7 +18,7 @@ module Appsignal
             alias shutdown_without_appsignal shutdown
 
             def shutdown
-              Appsignal.stop
+              Appsignal.stop('celluloid')
               shutdown_without_appsignal
             end
           end

data/lib/appsignal/hooks/passenger.rb

@@ -13,7 +13,7 @@ module Appsignal
         end
 
         ::PhusionPassenger.on_event(:stopping_worker_process) do
-          Appsignal.stop
+          Appsignal.stop('passenger')
         end
       end
     end

data/lib/appsignal/hooks/puma.rb

@@ -17,14 +17,14 @@ module Appsignal
 
         ::Puma.cli_config.options[:before_worker_shutdown] ||= []
         ::Puma.cli_config.options[:before_worker_shutdown] << Proc.new do |id|
-          Appsignal.stop
+          Appsignal.stop('puma before_worker_shutdown')
         end
 
         ::Puma::Cluster.class_eval do
           alias stop_workers_without_appsignal stop_workers
 
           def stop_workers
-            Appsignal.stop
+            Appsignal.stop('puma cluster')
             stop_workers_without_appsignal
           end
         end

data/lib/appsignal/hooks/rake.rb

@@ -24,7 +24,7 @@ module Appsignal
             transaction.set_action(name)
             transaction.set_error(error)
             transaction.complete
-            Appsignal.stop
+            Appsignal.stop('rake')
             raise error
           end
         end

data/lib/appsignal/hooks/unicorn.rb

@@ -29,7 +29,7 @@ module Appsignal
           alias close_without_appsignal close
 
           def close
-            Appsignal.stop
+            Appsignal.stop('unicorn')
             close_without_appsignal
           end
         end

data/lib/appsignal/integrations/delayed_job_plugin.rb

@@ -9,7 +9,7 @@ module Appsignal
         end
 
         lifecycle.after(:execute) do |execute|
-          Appsignal.stop
+          Appsignal.stop('delayed job')
         end
       end
 

data/lib/appsignal/transaction.rb

@@ -148,7 +148,6 @@ module Appsignal
       return unless Appsignal.active?
       return if Appsignal.is_ignored_error?(error)
 
-      Appsignal.logger.debug("Adding #{error.class.name} to transaction: #{transaction_id}")
       backtrace = cleaned_backtrace(error.backtrace)
       Appsignal::Extension.set_transaction_error(
         transaction_index,

data/lib/appsignal/version.rb

@@ -1,5 +1,5 @@
 require 'yaml'
 
 module Appsignal
-  VERSION = '1.1.5.beta.1'
+  VERSION = '1.1.5.beta.2'
 end

data/spec/lib/appsignal_spec.rb

@@ -153,10 +153,20 @@ describe Appsignal do
 
   describe ".stop" do
     it "should call stop on the extension" do
+      Appsignal.logger.should_receive(:debug).with('Stopping appsignal')
       Appsignal::Extension.should_receive(:stop)
       Appsignal.stop
       Appsignal.active?.should be_false
     end
+
+    context "with context specified" do
+      it "should log the context" do
+        Appsignal.logger.should_receive(:debug).with('Stopping appsignal (something)')
+        Appsignal::Extension.should_receive(:stop)
+        Appsignal.stop('something')
+        Appsignal.active?.should be_false
+      end
+    end
   end
 
   describe '.active?' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 1.1.5.beta.1
+  version: 1.1.5.beta.2
 platform: ruby
 authors:
 - Robert Beekman