puma_metrics_engine 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 248a19d8e7fb9d4392b1f4c4bf3cfa13d8e4872426d843f171732264783eaf94
-  data.tar.gz: b09efaf7ede10a759b435b8be828f0dbebde7f54ece70efddb050180e474ac4a
+  metadata.gz: 1ab335fa7843d45934fd97c5442b9bfd3a87006066b6f14ef99123a9316ab165
+  data.tar.gz: 7bd9134d5a436aecebd2a3f2a4073a5b51c46f3fb3d4a8b2391464e108326503
 SHA512:
-  metadata.gz: 4bbfb1639f3157a75afc8bbc538eecfd52492c05240be8d96576884ea2448cd6d8598b16e0a87c8fd2a93153b0b051e186842083e18bb874b8c5bb14bb17813c
-  data.tar.gz: 006066e431c7c1d7fed9cfb6c19b17715e2e7437a09ef23b8962838e33e063299a62ce7b47d6c6d535ebe535032ed8abd840f3851cf7e0505fab11bc023a041e
+  metadata.gz: c9f71532254eed460c8169a894c41e0ac03c7a089c37a133fc097d7632a0ab2bd2e282bbe84b7cd3ea667ea1adfa9f5a9cb3afc4aaaeb8d470107e5f403ab016
+  data.tar.gz: b704e4c376bdbdad8324b004c71ed20d176dbdda1febaaa63c6c411da33beee40cdb98460671dd8c558feb085a491602f2babc220d3ede42f2d77e90594b00d7

data/README.md

@@ -25,6 +25,12 @@ end
 
 Access metrics at `/matrix` endpoint.
 
+For debugging production issues, use `/debug` endpoint to check:
+- Middleware registration status
+- Redis connectivity
+- Header presence
+- Current data in Redis
+
 ## Response Format
 
 ```json

data/app/controllers/puma_metrics_engine/debug_controller.rb

@@ -0,0 +1,87 @@
+# frozen_string_literal: true
+
+module PumaMetricsEngine
+  class DebugController < ActionController::Base
+    skip_before_action :verify_authenticity_token
+
+    def show
+      render json: {
+        timestamp: Time.current,
+        middleware: middleware_status,
+        redis: redis_status,
+        headers: sample_headers,
+        data: redis_data_summary,
+        environment: environment_info
+      }
+    end
+
+    private
+
+      def middleware_status
+        middleware_stack = Rails.application.middleware.to_a.map(&:klass).map(&:name)
+        is_registered = middleware_stack.include?("PumaMetricsEngine::QueueTimeTracker")
+
+        {
+          registered: is_registered,
+          middleware_stack: middleware_stack,
+          note: is_registered ? "Middleware is registered" : "Middleware NOT found in stack"
+        }
+      rescue StandardError => e
+        { error: e.message }
+      end
+
+      def redis_status
+        client = redis
+        client.ping
+        {
+          connected: true,
+          url: ENV.fetch("REDIS_URL") { "redis://localhost:6379/1" },
+          keys: {
+            queue_times: client.exists?(MatrixController::QUEUE_TIMES_KEY),
+            request_timestamps: client.exists?(MatrixController::REQUESTS_KEY)
+          }
+        }
+      rescue StandardError => e
+        {
+          connected: false,
+          error: e.message,
+          url: ENV.fetch("REDIS_URL") { "redis://localhost:6379/1" }
+        }
+      end
+
+      def redis_data_summary
+        client = redis
+        {
+          queue_times_count: client.zcard(MatrixController::QUEUE_TIMES_KEY),
+          request_timestamps_count: client.zcard(MatrixController::REQUESTS_KEY),
+          recent_queue_times: client.zrange(MatrixController::QUEUE_TIMES_KEY, -10, -1),
+          recent_timestamps: client.zrange(MatrixController::REQUESTS_KEY, -10, -1)
+        }
+      rescue StandardError => e
+        { error: e.message }
+      end
+
+      def sample_headers
+        # Show what headers we're looking for
+        {
+          x_request_start: request.headers["X-Request-Start"],
+          http_x_request_start: request.headers["HTTP_X_REQUEST_START"],
+          all_x_headers: request.headers.select { |k, _| k.to_s.upcase.include?("X-REQUEST") },
+          note: "Check if X-Request-Start header is being sent by your load balancer"
+        }
+      end
+
+      def environment_info
+        {
+          rails_env: Rails.env,
+          redis_url_set: ENV.key?("REDIS_URL"),
+          puma_defined: defined?(Puma)
+        }
+      end
+
+      def redis
+        @redis ||= Redis.new(url: ENV.fetch("REDIS_URL") { "redis://localhost:6379/1" })
+      end
+  end
+end
+

data/config/routes.rb

@@ -2,4 +2,5 @@
 
 PumaMetricsEngine::Engine.routes.draw do
   get "matrix", to: "matrix#show"
+  get "debug", to: "debug#show"
 end

data/lib/puma_metrics_engine/queue_time_tracker.rb

@@ -14,6 +14,13 @@ module PumaMetricsEngine
       request_start_time = extract_request_start_time(env)
       process_start_time = Time.now.to_f
 
+      # Log header presence for debugging
+      if defined?(Rails)
+        header_value = env["HTTP_X_REQUEST_START"] || env["X-Request-Start"]
+        Rails.logger.debug("[QueueTimeTracker] X-Request-Start header: #{header_value.inspect}") if header_value
+        Rails.logger.debug("[QueueTimeTracker] No X-Request-Start header found") unless header_value
+      end
+
       status, headers, response = @app.call(env)
 
       # Calculate queue time if we have request start time
@@ -27,8 +34,10 @@ module PumaMetricsEngine
             timestamp = process_start_time
             # Store in Redis asynchronously to avoid blocking the request
             store_metrics_async(timestamp, queue_time_ms)
+            Rails.logger.debug("[QueueTimeTracker] Stored queue time: #{queue_time_ms}ms") if defined?(Rails)
           else
             # Still track request timestamp even if queue time is invalid
+            Rails.logger.warn("[QueueTimeTracker] Invalid queue time: #{queue_time_ms}ms (rejected)") if defined?(Rails)
             store_request_timestamp_async(process_start_time)
           end
         else
@@ -37,7 +46,8 @@ module PumaMetricsEngine
         end
       rescue StandardError => e
         # Don't let tracking errors break the request
-        Rails.logger.error("QueueTimeTracker error: #{e.message}") if defined?(Rails)
+        Rails.logger.error("[QueueTimeTracker] Error: #{e.message}") if defined?(Rails)
+        Rails.logger.error("[QueueTimeTracker] Backtrace: #{e.backtrace.first(5).join("\n")}") if defined?(Rails)
       end
 
       [status, headers, response]
@@ -89,7 +99,8 @@ module PumaMetricsEngine
             # Cleanup old data (older than TTL)
             cleanup_old_data(redis_client)
           rescue StandardError => e
-            Rails.logger.error("Failed to store queue time metrics: #{e.message}") if defined?(Rails)
+            Rails.logger.error("[QueueTimeTracker] Failed to store metrics: #{e.message}") if defined?(Rails)
+            Rails.logger.error("[QueueTimeTracker] Redis error backtrace: #{e.backtrace.first(3).join("\n")}") if defined?(Rails)
           end
         end
       end
@@ -101,7 +112,7 @@ module PumaMetricsEngine
             redis_client.zadd(REQUESTS_KEY, timestamp, timestamp)
             cleanup_old_data(redis_client)
           rescue StandardError => e
-            Rails.logger.error("Failed to store request timestamp: #{e.message}") if defined?(Rails)
+            Rails.logger.error("[QueueTimeTracker] Failed to store request timestamp: #{e.message}") if defined?(Rails)
           end
         end
       end

data/lib/puma_metrics_engine/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PumaMetricsEngine
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puma_metrics_engine
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Neeto
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-11-28 00:00:00.000000000 Z
+date: 2025-11-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -146,6 +146,7 @@ extra_rdoc_files: []
 files:
 - LICENSE.txt
 - README.md
+- app/controllers/puma_metrics_engine/debug_controller.rb
 - app/controllers/puma_metrics_engine/matrix_controller.rb
 - config/routes.rb
 - lib/puma_metrics_engine.rb