closeyourit-ruby 0.2.1 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b015462d6b0e563ef202410b5f2bc33b11ee10f985d984ac84a874836f67c82e
-  data.tar.gz: 41f3e3ca878dbfbfd818cd4963b238d1d78400344e0e8bbb95cfd025853ebf82
+  metadata.gz: d98365053459eaded9732014e987b01e3fcf2ac60a21db83026243d076bc2dbf
+  data.tar.gz: 4cb2024f4681e57d415097b8af237156f3cc0c8b764a2d14ea958bb8824e1738
 SHA512:
-  metadata.gz: 16d70f8ef7436397d881a8f67664f24aaabfe3ae976d5b23276149242bf8f539d1c5dc1d27f9d7c8a2c72a080ab42b2b56c11ec8af1e0fae108e06f5bf8cfbdc
-  data.tar.gz: '02853ab18aad59772dd4aa0cc497c32983e4e49f2926d2b42d28654552dabad1c0f9dacd589fc0423c920cbb1ea903457a35767365130777dc4d0317282e7225'
+  metadata.gz: de2f19a59df8c78825e0070e5c48c97424a7481d314219da675543da1ab71171ecfe7d5cd09cc0d81d48323be6d50435027e8a6bf901f121c6f5f3435b419b0d
+  data.tar.gz: 75ab998c2d1049bdcc047a96a54a01fe92cb50b820e81d1eac3795f4695416d7076956aac3defd82c520daaffd2dccf3726bb585cc189fb76090601928b3447f

data/README.md

@@ -171,6 +171,23 @@ Privacy-by-default (`send_pii = false`). In sintesi:
 > Rischio residuo: `exception.value` e i nomi tabella/colonna nello SQL possono contenere dati di
 > dominio. Usa `before_send`/`scrub_message_patterns` per azzerarli. Dettaglio in [`PDR.md` §9](PDR.md).
 
+## Diagnostica
+
+Il trasporto è fire-and-forget: non solleva mai e non blocca la request. Per non lasciare
+fallimenti silenziosi, ogni risposta HTTP non-2xx (es. `401` token errato, `404` progetto
+inesistente) viene loggata a `warn`, così come gli eventi scartati a coda piena. I contatori
+sono ispezionabili a runtime:
+
+```ruby
+CloseYourIt.stats.to_h
+# => { enqueued: 128, dropped: 0, sent: 126, failed: 2 }
+```
+
+- `enqueued` — eventi accettati per l'invio
+- `dropped` — scartati perché la coda async era piena (mai backpressure)
+- `sent` — risposta HTTP 2xx
+- `failed` — errore di rete o status non-2xx (vedi i log a `warn`)
+
 ## Sviluppo
 
 ```bash

data/lib/closeyourit/background_worker.rb

@@ -13,13 +13,22 @@ module CloseYourIt
       @executor = build_executor(threads.to_i, max_queue)
     end
 
+    # Ritorna true se l'evento è stato accettato (o eseguito sincrono), false se scartato
+    # perché la coda era piena (`fallback_policy: :discard`). Mai backpressure sulla request.
     def perform(&block)
-      @executor.post do
+      accepted = @executor.post do
         block.call
       rescue Exception => e # rubocop:disable Lint/RescueException
         # Mai propagare: la telemetria non deve poter crashare l'app ospite.
         CloseYourIt.logger.error("CloseYourIt background worker: #{e.class}: #{e.message}")
       end
+
+      unless accepted
+        CloseYourIt.stats.increment(:dropped)
+        CloseYourIt.logger.warn("CloseYourIt background worker: coda piena, evento scartato")
+      end
+
+      accepted
     end
 
     def shutdown(timeout = 1)

data/lib/closeyourit/client.rb

@@ -19,7 +19,8 @@ module CloseYourIt
       return nil if payload.nil?
 
       path = event.ingest_path(@configuration.project_id)
-      @worker.perform { @transport.send_event(payload, path: path) }
+      accepted = @worker.perform { @transport.send_event(payload, path: path) }
+      CloseYourIt.stats.increment(:enqueued) if accepted
       payload
     end
 

data/lib/closeyourit/configuration.rb

@@ -95,9 +95,12 @@ module CloseYourIt
       true
     end
 
-    # Logga i warning di configurazione (es. endpoint http://). Chiamata da `CloseYourIt.init`.
+    # Logga i warning di configurazione (es. endpoint http://, project_id/endpoint malformati).
+    # Non solleva mai: coerente con la filosofia no-op del client. Chiamata da `CloseYourIt.init`.
     def validate!
       CloseYourIt.logger.warn(insecure_endpoint_message) if insecure_endpoint?
+      CloseYourIt.logger.warn(malformed_project_id_message) if malformed_project_id?
+      CloseYourIt.logger.warn(malformed_endpoint_message) if malformed_endpoint?
       self
     end
 
@@ -133,11 +136,35 @@ module CloseYourIt
       nil
     end
 
+    UUID_FORMAT = /\A[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}\z/i
+
     def insecure_endpoint?
       uri = parsed_endpoint
       !uri.nil? && uri.scheme != "https"
     end
 
+    # Avvisa se il project_id è valorizzato ma non sembra uno UUID (l'errore tipico è incollare
+    # uno slug/nome al posto dell'id). Non blocca: il server è l'autorità sulla validità.
+    def malformed_project_id?
+      !blank?(project_id) && !UUID_FORMAT.match?(project_id.to_s)
+    end
+
+    def malformed_project_id_message
+      "CloseYourIt: project_id (#{project_id}) non ha forma UUID — verifica di aver copiato l'id corretto."
+    end
+
+    # Avvisa se endpoint_url è valorizzato ma non parsabile o privo di host.
+    def malformed_endpoint?
+      return false if blank?(endpoint_url)
+
+      uri = parsed_endpoint
+      uri.nil? || blank?(uri.host)
+    end
+
+    def malformed_endpoint_message
+      "CloseYourIt: endpoint_url (#{endpoint_url}) non è un URL valido (host mancante)."
+    end
+
     def insecure_endpoint_message
       tail = production? ? "Telemetria DISABILITATA in production." : "Consentito solo in sviluppo."
       "CloseYourIt: endpoint_url usa http:// non sicuro (#{endpoint_url}) — il token viaggerebbe in chiaro. #{tail}"

data/lib/closeyourit/stats.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+require "concurrent"
+
+module CloseYourIt
+  # Contatori diagnostici thread-safe del client: quanti eventi sono stati accodati,
+  # scartati (coda piena), spediti con successo o falliti (rete o status non-2xx).
+  # Servono a rendere visibili i fallimenti silenziosi del trasporto fire-and-forget.
+  #
+  #   CloseYourIt.stats.to_h # => { enqueued: 12, dropped: 0, sent: 11, failed: 1 }
+  class Stats
+    COUNTERS = %i[enqueued dropped sent failed].freeze
+
+    def initialize
+      @counters = COUNTERS.to_h { |name| [ name, Concurrent::AtomicFixnum.new(0) ] }
+    end
+
+    def increment(name)
+      counter = @counters.fetch(name)
+      counter.increment
+      counter.value
+    end
+
+    def [](name)
+      @counters.fetch(name).value
+    end
+
+    def to_h
+      @counters.transform_values(&:value)
+    end
+
+    def reset!
+      @counters.each_value { |counter| counter.value = 0 }
+      self
+    end
+  end
+end

data/lib/closeyourit/transport.rb

@@ -10,8 +10,8 @@ module CloseYourIt
   class Transport
     OPEN_TIMEOUT = 2
     READ_TIMEOUT = 3
-    # Net::HTTP non segue i redirect da solo. L'host canonico può rispondere 301 (es. apex → www):
-    # ri-POSTiamo a Location preservando metodo + body, così l'evento non si perde in silenzio.
+    # Net::HTTP non segue i redirect: l'host canonico può rispondere 301 (es. apex → www).
+    # Ri-POSTiamo a Location preservando metodo + body, così l'evento non si perde in silenzio.
     MAX_REDIRECTS = 2
 
     def initialize(configuration)
@@ -19,8 +19,16 @@ module CloseYourIt
     end
 
     def send_event(payload, path:)
-      post(payload, path)
+      response = post(payload, path)
+      if response.is_a?(Net::HTTPSuccess)
+        CloseYourIt.stats.increment(:sent)
+      else
+        CloseYourIt.stats.increment(:failed)
+        CloseYourIt.logger.warn("CloseYourIt transport: HTTP #{response.code} su #{path}")
+      end
+      response
     rescue StandardError => e
+      CloseYourIt.stats.increment(:failed)
       CloseYourIt.logger.error("CloseYourIt transport: #{e.class}: #{e.message}")
       nil
     end

data/lib/closeyourit/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CloseYourIt
-  VERSION = "0.2.1"
+  VERSION = "0.3.1"
 end

data/lib/closeyourit-ruby.rb

@@ -7,6 +7,7 @@ require_relative "closeyourit/configuration"
 require_relative "closeyourit/breadcrumb"
 require_relative "closeyourit/scope"
 require_relative "closeyourit/scrubber"
+require_relative "closeyourit/stats"
 require_relative "closeyourit/background_worker"
 require_relative "closeyourit/transport"
 require_relative "closeyourit/event"
@@ -144,6 +145,12 @@ module CloseYourIt
 
     attr_writer :logger
 
+    # Contatori diagnostici del client (accodati/scartati/spediti/falliti).
+    #   CloseYourIt.stats.to_h # => { enqueued: …, dropped: …, sent: …, failed: … }
+    def stats
+      @stats ||= Stats.new
+    end
+
     private
 
     def client

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: closeyourit-ruby
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.1
 platform: ruby
 authors:
 - Alessio Bussolari
@@ -55,6 +55,7 @@ files:
 - lib/closeyourit/scope.rb
 - lib/closeyourit/scrubber.rb
 - lib/closeyourit/sidekiq/error_handler.rb
+- lib/closeyourit/stats.rb
 - lib/closeyourit/subscribers/slow_query.rb
 - lib/closeyourit/transport.rb
 - lib/closeyourit/version.rb