rack-stats 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 57478c413f1cd1b052070b10d22cc7efa2bc3ca5
-  data.tar.gz: db6b9b28a2cd768f3c387fd738d1697c25e6668e
+  metadata.gz: f929b49c78e4e0f740861cfefe7e0c9ac7f8578d
+  data.tar.gz: b7c518fb2cec24740373cbabcefe94d8c3eb240f
 SHA512:
-  metadata.gz: 09de449d84aad70ff1b0d3415f53d9f48c0f16cfe4606f38ec6c98b4874428764b1549ffb634a470912956ea468acd38dcdb519b6382078773ed71a479292c6a
-  data.tar.gz: f60e6c7904216a117ad9f041d7a50408e7994e3827f7362270055a9544a4eab340936e660e8d9f4f976abe2dacf47407b173a73d87610bc2d4ae15080091906d
+  metadata.gz: 0136a39a4e69d328cb302f16a0ccad319b5c903e1bf9e0fa5c24e8de8ebde50d64b0fc577f4f45d3c35c481f65d9b74f3b11eb891d8874c882615df8c8ff4ada
+  data.tar.gz: b1742b17aaab56ebccef81d977c5c54286f6295fdd9905c71ed727ba86e29a9cea24c2c11b3b2ff8c206ab28eb42f848b88eac1dda9819af4c923a2c5ff7e55c

data/lib/rack/stats/runtime.rb

@@ -6,12 +6,12 @@ require 'rack/stats/stat'
 module Rack
   class Stats
     class Runtime
-      def initialize(statsd, app, env, stats)
+      def initialize(statsd, app, env, stats, namespace)
         @batch = Statsd::Batch.new statsd
         @app, @env, @timer = app, env, Timer.new
         @stats = stats.map do |stat|
           klass = stat[:type] == :increment ? Stat::Increment : Stat::Timing
-          klass.new stat[:name], stat[:value], stat[:condition]
+          klass.new stat[:name], stat[:value], stat[:condition], namespace
         end
       end
 

data/lib/rack/stats/stat.rb

@@ -2,13 +2,15 @@ module Rack
   class Stats
     module Stat
       class Base
-        def initialize(compute_name, compute_value = nil, condition = nil)
-          @compute_name, @compute_value, @condition = \
-            compute_name, compute_value, condition
+        def initialize(compute_name, compute_value = nil,
+                       condition = nil, namespace = nil)
+          @compute_name, @compute_value, @condition, @namespace = \
+            compute_name, compute_value, condition, namespace
         end
 
         def execute(batch, request, duration, response)
-          if @condition.nil? || @condition.call(request, duration, response)
+          if duration.ms > 0 && \
+            (@condition.nil? || @condition.call(request, duration, response))
             send_data(batch, request, duration, response)
           end
         end
@@ -18,7 +20,10 @@ module Rack
         def send_data(batch, request, duration, response)
           batch.send(
             :timing,
-            [@compute_name.call(request, duration, response)].compact.join('.'),
+            [
+              @namespace,
+              @compute_name.call(request, duration, response)
+            ].flatten.compact.join('.'),
             @compute_value.call(request, duration, response)
           )
         end
@@ -28,7 +33,10 @@ module Rack
         def send_data(batch, request, duration, response)
           batch.send(
             :increment,
-            [@compute_name.call(request, duration, response)].compact.join('.')
+            [
+              @namespace,
+              @compute_name.call(request, duration, response)
+            ].flatten.compact.join('.')
           )
         end
       end

data/lib/rack/stats/version.rb

@@ -1,5 +1,5 @@
 module Rack
   class Stats
-    VERSION = "0.0.2"
+    VERSION = "0.0.3"
   end
 end

data/lib/rack/stats.rb

@@ -36,12 +36,11 @@ module Rack
       @app = app
       @namespace = args.fetch(:namespace, ENV['RACK_STATS_NAMESPACE'])
       @statsd = Statsd.new(*args.fetch(:statsd, '127.0.0.1:8125').split(':'))
-      @statsd.tap { |sd| sd.namespace = @namespace } if @namespace
       @stats = args.fetch(:stats, DEFAULT_STATS)
     end
 
     def call(env)
-      Runtime.new(@statsd, @app, env, @stats).execute
+      Runtime.new(@statsd, @app, env, @stats, @namespace).execute
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rack-stats
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Alexis Montagne