appsignal 1.2.1 → 1.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b40b132199c6068e639dd806e587066b942e4e24
-  data.tar.gz: 3a0fe572f3d6d32ac2c88862fb2bc5ea98661cd3
+  metadata.gz: 4d5f6164fcbb45bfc381a58c68140a97c1c94393
+  data.tar.gz: f88bea811683f981f8e88d1cf7de344917a34499
 SHA512:
-  metadata.gz: ea977bb6a35a3f94e3bdd46018af4f81c7801c804c2cf485371b7216afa8bf7c2ed4f2444503ac393ceeb2ca34cd7f4e37b90fe75c949b37f67696adfe372c6c
-  data.tar.gz: b3d4df756299586f67e77899569c6951167ff72aeb3ec2783ac4461f046bf571e7a98fc0f6bc125da85d054900dc53b527d7f4da5d5f3609f38614b463043b7c
+  metadata.gz: 7b51229db4c5a815eaea0459078d4bcc3b99b252de412b74d1d7e020ce0bc27f64cf0a10154acf1175cc1cda0aec2af2223f9790d2eca45a2334fa5a396a8d96
+  data.tar.gz: 477227b0dd58d3276d1ec5b1fb56a9b0f0499fc1e9fccbc20cfab7b9172ce6c90fd02695fd3743a7f332bec6b3a1b07669d34c0382da1d1f432af8e4f8d36fca

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+# 1.2.2
+* Handle out of range numbers in queue lenght and metrics api
+
 # 1.2.1
 * Use Dir.pwd in CLI install wizard
 * Support bignums when setting queue length

data/lib/appsignal.rb

@@ -179,22 +179,32 @@ module Appsignal
 
     def set_gauge(key, value)
       Appsignal::Extension.set_gauge(key, value.to_f)
+    rescue RangeError
+      Appsignal.logger.warn("Gauge value #{value} for key '#{key}' is too big")
     end
 
     def set_host_gauge(key, value)
       Appsignal::Extension.set_host_gauge(key, value.to_f)
+    rescue RangeError
+      Appsignal.logger.warn("Host gauge value #{value} for key '#{key}' is too big")
     end
 
     def set_process_gauge(key, value)
       Appsignal::Extension.set_process_gauge(key, value.to_f)
+    rescue RangeError
+      Appsignal.logger.warn("Process gauge value #{value} for key '#{key}' is too big")
     end
 
     def increment_counter(key, value=1)
       Appsignal::Extension.increment_counter(key, value)
+    rescue RangeError
+      Appsignal.logger.warn("Counter value #{value} for key '#{key}' is too big")
     end
 
     def add_distribution_value(key, value)
       Appsignal::Extension.add_distribution_value(key, value.to_f)
+    rescue RangeError
+      Appsignal.logger.warn("Distribution value #{value} for key '#{key}' is too big")
     end
 
     def logger

data/lib/appsignal/transaction.rb

@@ -117,6 +117,8 @@ module Appsignal
     def set_queue_start(start)
       return unless start
       @ext.set_queue_start(start)
+    rescue RangeError
+      Appsignal.logger.warn("Queue start value #{start} is too big")
     end
 
     def set_http_or_background_queue_start

data/lib/appsignal/version.rb

@@ -1,5 +1,5 @@
 require 'yaml'
 
 module Appsignal
-  VERSION = '1.2.1'
+  VERSION = '1.2.2'
 end

data/spec/lib/appsignal/transaction_spec.rb

@@ -269,6 +269,16 @@ describe Appsignal::Transaction do
 
         transaction.set_queue_start(nil)
       end
+
+      it "should not raise an error when the queue start is too big" do
+        transaction.ext.should_receive(:set_queue_start).and_raise(RangeError)
+
+        Appsignal.logger.should_receive(:warn).with("Queue start value 10 is too big")
+
+        lambda {
+          transaction.set_queue_start(10)
+        }.should_not raise_error
+      end
     end
 
     describe "#set_http_or_background_queue_start" do

data/spec/lib/appsignal_spec.rb

@@ -402,6 +402,14 @@ describe Appsignal do
           Appsignal::Extension.should_receive(:set_gauge).with('key', 1.0)
           Appsignal.set_gauge('key', 1)
         end
+
+        it "should not raise an exception when out of range" do
+          Appsignal::Extension.should_receive(:set_gauge).with('key', 10).and_raise(RangeError)
+          Appsignal.logger.should_receive(:warn).with("Gauge value 10 for key 'key' is too big")
+          lambda {
+            Appsignal.set_gauge('key', 10)
+          }.should_not raise_error
+        end
       end
 
       describe ".set_host_gauge" do
@@ -414,6 +422,14 @@ describe Appsignal do
           Appsignal::Extension.should_receive(:set_host_gauge).with('key', 1.0)
           Appsignal.set_host_gauge('key', 1)
         end
+
+        it "should not raise an exception when out of range" do
+          Appsignal::Extension.should_receive(:set_host_gauge).with('key', 10).and_raise(RangeError)
+          Appsignal.logger.should_receive(:warn).with("Host gauge value 10 for key 'key' is too big")
+          lambda {
+            Appsignal.set_host_gauge('key', 10)
+          }.should_not raise_error
+        end
       end
 
       describe ".set_process_gauge" do
@@ -426,6 +442,14 @@ describe Appsignal do
           Appsignal::Extension.should_receive(:set_process_gauge).with('key', 1.0)
           Appsignal.set_process_gauge('key', 1)
         end
+
+        it "should not raise an exception when out of range" do
+          Appsignal::Extension.should_receive(:set_process_gauge).with('key', 10).and_raise(RangeError)
+          Appsignal.logger.should_receive(:warn).with("Process gauge value 10 for key 'key' is too big")
+          lambda {
+            Appsignal.set_process_gauge('key', 10)
+          }.should_not raise_error
+        end
       end
 
       describe ".increment_counter" do
@@ -438,6 +462,14 @@ describe Appsignal do
           Appsignal::Extension.should_receive(:increment_counter).with('key', 5)
           Appsignal.increment_counter('key', 5)
         end
+
+        it "should not raise an exception when out of range" do
+          Appsignal::Extension.should_receive(:increment_counter).with('key', 10).and_raise(RangeError)
+          Appsignal.logger.should_receive(:warn).with("Counter value 10 for key 'key' is too big")
+          lambda {
+            Appsignal.increment_counter('key', 10)
+          }.should_not raise_error
+        end
       end
 
       describe ".add_distribution_value" do
@@ -450,6 +482,14 @@ describe Appsignal do
           Appsignal::Extension.should_receive(:add_distribution_value).with('key', 1.0)
           Appsignal.add_distribution_value('key', 1)
         end
+
+        it "should not raise an exception when out of range" do
+          Appsignal::Extension.should_receive(:add_distribution_value).with('key', 10).and_raise(RangeError)
+          Appsignal.logger.should_receive(:warn).with("Distribution value 10 for key 'key' is too big")
+          lambda {
+            Appsignal.add_distribution_value('key', 10)
+          }.should_not raise_error
+        end
       end
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.2
 platform: ruby
 authors:
 - Robert Beekman
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-29 00:00:00.000000000 Z
+date: 2016-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack