pigeon-http 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b49e953d40948596cfea066d3002efb0e612ece45ce6b8bcdd1f25772d329ed
-  data.tar.gz: fc38bffab847cd7c37b13e0009bd11a61e039ceaf87fd8495833ec2228861317
+  metadata.gz: 204c0c65ca18699400c662cdce4813337fa5f270a15012c63279ab03e1fb35c4
+  data.tar.gz: f2c2070ca2ab97182b0574ecf281759a14cee4b7d7d42f42ce23d97b8a8a0ace
 SHA512:
-  metadata.gz: 2296d57aabe77db0a39cf9ef06abdbff05160b8a72fd16b910407e3c42075d8132c289ab9568942ccee5c44af51a96a43e020b01186f49225121e4d9a4f9e852
-  data.tar.gz: ff378ba4eb6137df6e43bf6355bc77882d0d4520c9ec9132ec8b64d5742ab5ecde89faeaf6bc0a72f7651130a8e8fd424aa941af4eb3a8450146821926fe224a
+  metadata.gz: 2f66c5ce34abe98064138ade62435d182a0592ee147799b52c199e19893f5b5c66dc01d5a3d7755efc7b9c330fb95d74e539d8176637b8fbda03b6baa31fce4a
+  data.tar.gz: 72cc4bbac4814d99f6f3a79fdfe49a261cc65a7044706e8736469e61fb1b90ed5ca2edd07a47cc93f313c949bd7605b1f4982bfaf65990f08431b9a2b11dba58

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    pigeon-http (0.1.3)
+    pigeon-http (0.1.4)
       circuitbox
       ddtrace
       dogstatsd-ruby

data/lib/pigeon/core.rb

@@ -77,49 +77,43 @@ module Pigeon
     end
 
     def get url, args = {}
-      start = Time.now
       response = http(:get, url, args)
-      Pigeon::Statsd.new(@options[:request_name] + '_latency', tags: [url]).capture(action: :histogram, count: (Time.now - start)) unless @options[:monitoring]
-
-      response
     rescue => e
+      puts e
       @callbacks[:HttpError]&.call(e)
     end
 
     def post url, args = {}
-      start = Time.now
       response = http(:post, url, args)
-      Pigeon::Statsd.new(@options[:request_name] + '_latency', tags: [url]).capture(action: :histogram, count: (Time.now - start)) unless @options[:monitoring]
-
-      response
     rescue => e
       @callbacks[:HttpError]&.call(e)
     end
 
     def put url, args = {}
-      start = Time.now
       response = http(:put, url, args)
-      Pigeon::Statsd.new(@options[:request_name] + '_latency', tags: [url]).capture(action: :histogram, count: (Time.now - start)) unless @options[:monitoring]
-
-      response
     rescue => e
       @callbacks[:HttpError]&.call(e)
     end
 
     def delete url, args = {}
-      http(:delete, url, args)
+      response = http(:delete, url, args)
     rescue => e
       @callbacks[:HttpError]&.call(e)
     end
 
     def http method, url, args = {}
+      start = Time.now
       args.merge({
         read_timeout: @options[:request_timeout],
         open_timeout: @options[:request_open_timeout],
         ssl_verify:   @options[:ssl_verify]
       })
       response = Pigeon::Http::Request.new(method, url, args).execute
-      Pigeon::Statsd.new(@options[:request_name] + '_througput', tags: [url]).capture
+
+      uri = URI.parse(url)
+      Pigeon::Statsd.new(@options[:request_name] + '_latency', tags: [uri.host]).capture(action: :histogram, count: (Time.now - start))
+      Pigeon::Statsd.new(@options[:request_name] + '_througput', tags: [uri.host]).capture
+      Pigeon::Statsd.new(@options[:request_name] + '_status', tags: [response.code]).capture
 
       response
     end

data/lib/pigeon/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Pigeon
-  VERSION = '0.1.3'
+  VERSION = '0.1.4'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pigeon-http
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Fitra Aditya
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-09-12 00:00:00.000000000 Z
+date: 2023-09-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: circuitbox