appsignal 0.6.3.beta.3 → 0.6.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    MjhhMGUxYTk4NTgyYmUyZTlhNmU5OTQ5YTA5YjI1MjI1NmRiZjFkZA==
+    YWQxNjliNTEwYjc3ODI4ZjBkZGNjYzFhZjEwZDJlM2QzMDc0OWEyNQ==
   data.tar.gz: !binary |-
-    MzdkMmI2NDJhNTU4NDliZWJjYmQwNTQ3NGZkYWRkZTFhZmIzYjdiMQ==
+    NjVmNDU2YjI3YTlkOGNkY2MyYzAwYmMyMGM4ZjU5MDczZTI1OTg0ZQ==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    YzRkMWNhMjFlNzEzNjZlZTFjYTU5YWI0NmZhM2ViMDllZjg2Y2NkOWMwZmFm
-    M2VlYTExYjJjY2E3ZTFkYjJjMjg5ZjMyMGQ4NGIwZWYwNjliNTE0MTNjZGI2
-    NDE2OTRmMDE2ZTBlOGQ4MWIyNjgyMWM0MzVlMjgzNzI5MGI4ZDg=
+    NzI5NzJhY2RhYjE2NjdlMDZlNmNkNDRlMzdjNThlZmRkMGRjNjZjOTQ4YWRk
+    ZjJmYTc1MDhkMjE3N2UwODVkZmMwNWFkMmJmOThiMGYzMmZmMjRkOGIzMTZl
+    MjBkMWQyODAwNDY2YTE2YTA4YWVlMWE3ZGQyYmNhODQyMjkyMDA=
   data.tar.gz: !binary |-
-    ZDJkZWM1MTc5OTA3MjZhZjFjZjMzOTMxZWM2NTIyYWI1NDAyYzZlODZiMzJh
-    ZjY0ODg1MDEwZTllYzA2MzFjZTg1NGUwMTUyOGQxNTNiNDEyMmE5MTNkNzQw
-    MDk3MWEwN2Y1ZjNlMjZjMGM3MjQ1ZTg1YWMwNGNlOGE5MTEwNWE=
+    NzEwNTMwOTBlYzU3NWFhZTA5OTFkNTRiMzVmODFjMjA3NDM1MTcyNDQ5YjNm
+    ZWFlNDBhNTJiZDNjNjI1ZGIwNDhiM2Q0ZDUwMWM3NmI2YjYyY2I2YTQ5Y2Q1
+    MmViNjU1ODE1ODZiNTAwNGMyYzk0MzM4NWM5MmQ4OWVhMzJiZGU=

data/CHANGELOG.md

@@ -2,6 +2,7 @@
 * Use a mutex around access to the aggregator
 * Bugfix for accessing connection config in Rails 3.0
 * Add Appsignal.tag_request
+* Only warn if there are duplicate push keys
 
 # 0.6.2
 * Bugfix in backtrace cleaner usage for Rails 4

data/lib/appsignal/agent.rb

@@ -22,7 +22,7 @@ module Appsignal
         ACTION,
         Appsignal.config.fetch(:api_key)
       )
-      Appsignal.logger.info("Started the Appsignal agent")
+      Appsignal.logger.info("Started Appsignal agent")
     end
 
     def enqueue(transaction)
@@ -93,6 +93,5 @@ module Appsignal
           "'#{code}'"
       end
     end
-
   end
 end

data/lib/appsignal/config.rb

@@ -24,8 +24,8 @@ module Appsignal
 
     def load
       return unless load_configurations
-      return unless used_unique_api_keys
       return unless current_environment_present
+      warn_if_duplicate_api_keys
 
       if Appsignal.respond_to?(:logger) && @logger == Appsignal.logger
         @logger.level = Logger::DEBUG if configurations[env][:debug]
@@ -35,7 +35,7 @@ module Appsignal
 
     def load_all
       return unless load_configurations
-      return unless used_unique_api_keys
+      warn_if_duplicate_api_keys
 
       {}.tap do |result|
         configurations.each do |env, config|
@@ -81,13 +81,10 @@ module Appsignal
       end
     end
 
-    def used_unique_api_keys
+    def warn_if_duplicate_api_keys
       keys = configurations.each_value.map { |config| config[:api_key] }.compact
       if keys.uniq.count < keys.count
-        carefully_log_error('Duplicate API keys found in appsignal.yml')
-        false
-      else
-        true
+        carefully_log_error('Warning: Duplicate API keys found in appsignal.yml')
       end
     end
 

data/lib/appsignal/version.rb

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

data/spec/appsignal/config_spec.rb

@@ -48,12 +48,6 @@ describe Appsignal::Config do
 
       it { should be_nil }
     end
-
-    context "when an api key is used for more then one environment" do
-      before { config.stub(:used_unique_api_keys => false) }
-
-      it { should be_nil }
-    end
   end
 
   # protected
@@ -136,28 +130,26 @@ describe Appsignal::Config do
     end
   end
 
-  describe "#used_unique_api_keys" do
+  describe "#warn_if_duplicate_api_keys" do
     let(:env) { {:api_key => :foo} }
-    subject { config.send(:used_unique_api_keys) }
+    subject { config.send(:warn_if_duplicate_api_keys) }
 
     context "when using all unique keys" do
       before do
         config.should_not_receive(:carefully_log_error)
         config.stub(:configurations => {1 => env})
       end
-
-      it { should be_true }
     end
 
     context "when using non-unique keys" do
       before do
         config.should_receive(:carefully_log_error).
-          with("Duplicate API keys found in appsignal.yml")
+          with("Warning: Duplicate API keys found in appsignal.yml")
         config.stub(:configurations => {:production => env, :staging => env})
       end
-
-      it { should be_false }
     end
+
+    after { subject }
   end
 
   describe "#current_environment_present" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 0.6.3.beta.3
+  version: 0.6.3
 platform: ruby
 authors:
 - Robert Beekman
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-23 00:00:00.000000000 Z
+date: 2013-09-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -205,9 +205,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>'
+  - - ! '>='
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.0.3