bug_bunny 4.1.0 → 4.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ef1416747a080c5d6af30ac97412688c0419e8b3635b410d93580ad869d78924
-  data.tar.gz: e362258f131a6fefdad0b835a410218e349d8d56966bfef67586dd573375ffcb
+  metadata.gz: 599004cdcb498ca746612f60202301ce9783a67ac37d14ecd4c29f51ccaf48bb
+  data.tar.gz: f748ba9a55e91f96a5571f671afa524f7e330cceede0f12c5c7cef6d25d2d2ff
 SHA512:
-  metadata.gz: e788b41069a507289245398fc56ab5829df1139665319a254b6136aa26eb44a607b0f7d0843b517f4ed3298095629bed97e64d9aaace5cc81313f4995392d12d
-  data.tar.gz: cfff2337042002bf850200a8b63739502e2c4607d2c26d661756f980cf7c8c8986a96545811c03aba7b8fc3c23fe929e62240f63dada14c55ddb02cd38feda30
+  metadata.gz: 7df1503a55cb62d9f87168a84b8e92fe773b4c9525b143d6f848227e66fe63e5ab75f6191a408b00bb9c8840b44c0f8fdbc36cd0a8d2eb56e02018d09654d269
+  data.tar.gz: d5433d839c45ad95a2d0399535ff629112458e90d0e4e1ec89f97afa450b139bcb7855c64c27042c22bce0bde87b2bfa7bd6f8bae5b879dd7eb0c71ff1d20d3c

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## [4.1.1] - 2026-03-22
+
+### 🐛 Bug Fixes
+* **Consumer:** Previene memory leak al detener el `TimerTask` de health check previo antes de realizar una reconexión.
+* **Controller:** Corrige la mutación accidental de \`log_tags\` globales al usar una lógica de herencia no destructiva en \`compute_tags\`.
+
+### ✨ Improvements
+* **Controller:** Ahora lanza una excepción \`BugBunny::BadRequest\` (400) si el cuerpo de la petición contiene un JSON inválido, mejorando la depuración en el cliente.
+* **Resource:** Se añadió una protección a \`.with\` (\`ScopeProxy\`) para asegurar que el contexto sea de un solo uso, evitando efectos secundarios en llamadas encadenadas.
+
 ## [4.1.0] - 2026-03-22
 
 ### 🚀 New Features & Improvements

data/lib/bug_bunny/consumer.rb

@@ -44,6 +44,7 @@ module BugBunny
     # @param connection [Bunny::Session] Conexión nativa de Bunny.
     def initialize(connection)
       @session = BugBunny::Session.new(connection)
+      @health_timer = nil
     end
 
     # Inicia la suscripción a la cola y comienza el bucle de procesamiento.
@@ -244,12 +245,16 @@ module BugBunny
     # @param q_name [String] Nombre de la cola a monitorear.
     # @return [void]
     def start_health_check(q_name)
+      # Detener el timer anterior antes de crear uno nuevo (evita leak en cada retry)
+      @health_timer&.shutdown
+      @health_timer = nil
+
       file_path = BugBunny.configuration.health_check_file
 
       # Toque inicial para indicar al orquestador que el worker arrancó correctamente
       touch_health_file(file_path) if file_path
 
-      Concurrent::TimerTask.new(execution_interval: BugBunny.configuration.health_check_interval) do
+      @health_timer = Concurrent::TimerTask.new(execution_interval: BugBunny.configuration.health_check_interval) do
         # 1. Verificamos la salud de RabbitMQ (si falla, levanta un error y corta la ejecución del bloque)
         session.channel.queue_declare(q_name, passive: true)
 
@@ -258,7 +263,8 @@ module BugBunny
       rescue StandardError => e
         BugBunny.configuration.logger.warn("[BugBunny::Consumer] ⚠️  Queue check failed: #{e.message}. Reconnecting session...")
         session.close
-      end.execute
+      end
+      @health_timer.execute
     end
 
     # Actualiza la fecha de modificación del archivo de health check (touchfile).

data/lib/bug_bunny/controller.rb

@@ -146,11 +146,6 @@ module BugBunny
     def process(body)
       prepare_params(body)
 
-      # Inyección de configuración global de logs si el controlador no define propios
-      if self.class.log_tags.empty? && BugBunny.configuration.log_tags.any?
-        self.class.log_tags = BugBunny.configuration.log_tags
-      end
-
       action_name = headers[:action].to_sym
       current_arounds = resolve_callbacks(self.class.around_actions, action_name)
 
@@ -245,12 +240,11 @@ module BugBunny
       if body.is_a?(Hash)
         params.merge!(body)
       elsif body.is_a?(String) && headers[:content_type].to_s.include?('json')
-        parsed = begin
-                   JSON.parse(body)
-                 rescue JSON::ParserError
-                   nil
-                 end
-        params.merge!(parsed) if parsed
+        begin
+          params.merge!(JSON.parse(body))
+        rescue JSON::ParserError => e
+          raise BugBunny::BadRequest, "Invalid JSON in request body: #{e.message}"
+        end
       else
         self.raw_string = body
       end
@@ -284,7 +278,8 @@ module BugBunny
     end
 
     def compute_tags
-      self.class.log_tags.map do |tag|
+      tags = self.class.log_tags.presence || BugBunny.configuration.log_tags
+      tags.map do |tag|
         case tag
         when Proc
           tag.call(self)

data/lib/bug_bunny/resource.rb

@@ -154,9 +154,24 @@ module BugBunny
       end
 
       # Proxy para el encadenamiento del método `.with`.
+      # Solo puede usarse para UNA llamada de método: el contexto se restaura al finalizar.
       class ScopeProxy < BasicObject
-        def initialize(target, keys, old_values); @target = target; @keys = keys; @old_values = old_values; end
-        def method_missing(method, *args, &block); @target.public_send(method, *args, &block); ensure; @keys.each { |k, v| ::Thread.current[v] = @old_values[k] }; end
+        def initialize(target, keys, old_values)
+          @target = target
+          @keys = keys
+          @old_values = old_values
+          @used = false
+        end
+
+        def method_missing(method, *args, &block)
+          if @used
+            ::Kernel.raise ::BugBunny::Error, "ScopeProxy is single-use. Call .with again for a new context."
+          end
+          @used = true
+          @target.public_send(method, *args, &block)
+        ensure
+          @keys.each { |k, v| ::Thread.current[v] = @old_values[k] }
+        end
       end
 
       # Calcula la routing key final.

data/lib/bug_bunny/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BugBunny
-  VERSION = "4.1.0"
+  VERSION = "4.1.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bug_bunny
 version: !ruby/object:Gem::Version
-  version: 4.1.0
+  version: 4.1.1
 platform: ruby
 authors:
 - gabix