yabeda-rack-queue 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 74ed559c6678f3327af95b6d6eea03ecbe9de4bd6a77aa2a250bd17ce94125d3
-  data.tar.gz: 845745efa2a2ab7563574077844ee92a550f22a58fbec1bf4490b87552eb0faa
+  metadata.gz: a2460b8b476ceba45cccd0edd32320f133780bc01681d08bcf5fb162b7dd0cf4
+  data.tar.gz: 3b69c898a813804c2f8194fa9e5a84cf1508c1d3df2eb52096683e1b4612d09d
 SHA512:
-  metadata.gz: 8957a43169221a5bc4616b1906351c97c13044c87594f0231857f2f065dcddb59aa89bb508985620875b1ae1e58f5c05d7b14eb829acfb6d5ff3ecbab619b164
-  data.tar.gz: bb860cf5b7e30ee679bcaa46f77fca199eb324070873ab1c51533a3549c2065e97119c72bb84522cb11cea6c71e1327d42edfc4643ed85c985e435c164398bde
+  metadata.gz: 7b34b7083e9afb7c53825d8fd2645d74d36f4b6e9551885546de59f19d836199eb9cf3abc2a4a504d9844a52b468d157c1e6c1a3b7c1df62436b2286b1199271
+  data.tar.gz: c82d59eb210d12595a0c53a4330d004d0d7d4667fdb71101d0b2c2e4a017bc78b77f91ed312ceb6fae8bc68d2d28fb3ee6c7c761eb5020ffbedfc937af6b9788

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.2.1 - 2026-06-29
+
+### Fixed
+
+- Keep requests flowing if rack queue metric reporting fails, logging the error instead.
+- Load `yabeda/railtie` when Rails appears after Yabeda was already required.
+
 ## 0.2.0 - 2026-03-09
 
 ### Changed

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    yabeda-rack-queue (0.2.0)
+    yabeda-rack-queue (0.2.1)
       yabeda (>= 0.14, < 1.0)
 
 GEM
@@ -114,7 +114,7 @@ CHECKSUMS
   unicode-display_width (3.2.0) sha256=0cdd96b5681a5949cdbc2c55e7b420facae74c4aaf9a9815eee1087cb1853c42
   unicode-emoji (4.2.0) sha256=519e69150f75652e40bf736106cfbc8f0f73aa3fb6a65afe62fefa7f80b0f80f
   yabeda (0.14.0) sha256=bc517bf22d692ebd80a29fc9fd2246c257aaf92d10b2735a775e2419351a43bf
-  yabeda-rack-queue (0.2.0)
+  yabeda-rack-queue (0.2.1)
 
 BUNDLED WITH
   4.0.3

data/lib/yabeda/rack/queue/metric.rb

@@ -2,6 +2,10 @@
 
 require "yabeda"
 
+# Yabeda only loads its Railtie if Rails exists when Yabeda is first required.
+# Puma plugins can require Yabeda before Rails, so retry when this gem loads.
+require "yabeda/railtie" if defined?(::Rails::Railtie)
+
 Yabeda.configure do
   group :rack_queue do
     histogram :duration,

data/lib/yabeda/rack/queue/middleware.rb

@@ -32,6 +32,8 @@ module Yabeda
           start = @parser.parse(env["HTTP_X_REQUEST_START"], now: now) ||
             @parser.parse(env["HTTP_X_QUEUE_START"], now: now)
           report_queue_time(env, now, start) if start
+        rescue => error
+          @logger.warn("rack queue metric failed: #{error.class}: #{error.message}")
         end
 
         def report_queue_time(env, now, request_start)

data/lib/yabeda/rack/queue/version.rb

@@ -3,7 +3,7 @@
 module Yabeda
   module Rack
     module Queue
-      VERSION = "0.2.0"
+      VERSION = "0.2.1"
     end
   end
 end

data/test/yabeda/rack/queue/middleware_test.rb

@@ -14,6 +14,12 @@ class CapturingReporter
   end
 end
 
+class FailingReporter
+  def observe(_value)
+    raise NoMethodError, "undefined method `rack_queue' for Yabeda:Module"
+  end
+end
+
 class CapturingLogger
   attr_reader :warnings
 
@@ -144,6 +150,21 @@ class MiddlewareTest < Minitest::Test
     assert_empty @reporter.values
   end
 
+  def test_metric_reporting_errors_are_logged_and_do_not_break_request
+    middleware = Yabeda::Rack::Queue::Middleware.new(
+      @app,
+      reporter: FailingReporter.new,
+      clock: @clock,
+      logger: @logger
+    )
+
+    result = middleware.call("HTTP_X_REQUEST_START" => "t=1699999999.9")
+
+    assert_same @response, result
+    assert_equal 1, @app.called_count
+    assert_includes @logger.warnings.join("\n"), "rack queue metric failed: NoMethodError"
+  end
+
   def test_drops_negative_queue_times_and_logs_warning
     @middleware.call("HTTP_X_REQUEST_START" => "t=1700000000.1")
 

data/test/yabeda/rack/queue/railtie_test.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require "test_helper"
+require "open3"
+require "rbconfig"
+
+class RailtieTest < Minitest::Test
+  def test_loads_yabeda_railtie_when_rails_appears_after_yabeda
+    script = <<~RUBY
+      require "bundler/setup"
+      require "yabeda"
+
+      module Rails
+        class Railtie
+          def self.config
+            @config ||= Config.new
+          end
+
+          class Config
+            def after_initialize
+            end
+          end
+        end
+      end
+
+      require "yabeda/rack/queue"
+
+      abort "Yabeda::Rails::Railtie was not loaded" unless defined?(Yabeda::Rails::Railtie)
+    RUBY
+
+    stdout, stderr, status = Open3.capture3("bundle", "exec", RbConfig.ruby, "-Ilib", "-e", script)
+
+    assert status.success?, [stdout, stderr].join("\n")
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: yabeda-rack-queue
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Nate Berkopec
@@ -167,6 +167,7 @@ files:
 - test/yabeda/rack/queue/metric_test.rb
 - test/yabeda/rack/queue/middleware_performance_test.rb
 - test/yabeda/rack/queue/middleware_test.rb
+- test/yabeda/rack/queue/railtie_test.rb
 - yabeda-rack-queue.gemspec
 homepage: https://github.com/speedshop/yabeda-rack-queue
 licenses:
@@ -191,7 +192,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 4.0.10
 specification_version: 4
 summary: Yabeda middleware for HTTP request queue duration
 test_files: []