puma_metrics_engine 1.1.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 248a19d8e7fb9d4392b1f4c4bf3cfa13d8e4872426d843f171732264783eaf94
-  data.tar.gz: b09efaf7ede10a759b435b8be828f0dbebde7f54ece70efddb050180e474ac4a
+  metadata.gz: 32e8e883e66e79ca9d1ed80ee21f5b0b880241f0157e325b21c7b679d9ffa56f
+  data.tar.gz: de06e4bed7924f401382ad5e852936cc16bbb699e7c98569957d8a7dae203cf9
 SHA512:
-  metadata.gz: 4bbfb1639f3157a75afc8bbc538eecfd52492c05240be8d96576884ea2448cd6d8598b16e0a87c8fd2a93153b0b051e186842083e18bb874b8c5bb14bb17813c
-  data.tar.gz: 006066e431c7c1d7fed9cfb6c19b17715e2e7437a09ef23b8962838e33e063299a62ce7b47d6c6d535ebe535032ed8abd840f3851cf7e0505fab11bc023a041e
+  metadata.gz: d51049bfb157f4641cf41499ddd3c1769f7ffa8e13b0dba731124e748cfa2b80cfefb445ca7ecdc1fe06ba00aa31a137f7bc7978d84f962cb4ac413515a5bb7f
+  data.tar.gz: d2d192c9ceda0d7eed4de38ec1a2cce70f36d468cf3dc9de345cb7b3929fd4cd93df1b8fbda1d7deeb11546af6dea4cf0910291238ccb98393d9227572065599

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]
@@ -47,7 +57,7 @@ module PumaMetricsEngine
 
       def extract_request_start_time(env)
         # Try X-Request-Start header (common in nginx, HAProxy, etc.)
-        # Format: "t=1234567890.123" or "1234567890.123" or Unix timestamp in microseconds
+        # Format: "t=1234567890.123" or "1234567890.123" or Unix timestamp in milliseconds/microseconds
         header_value = env["HTTP_X_REQUEST_START"] || env["X-Request-Start"]
 
         return nil unless header_value
@@ -65,10 +75,25 @@ module PumaMetricsEngine
 
         timestamp = timestamp_str.to_f
 
-        # If timestamp is in microseconds (common with nginx), convert to seconds
-        # Timestamps > year 2100 are likely in microseconds
+        # Detect timestamp format based on magnitude:
+        # - 10 digits or less: seconds (e.g., 1764410986)
+        # - 11-13 digits: milliseconds (e.g., 1764410986245)
+        # - 14+ digits: microseconds (e.g., 1764410986245000)
         if timestamp > 4_102_444_800 # Year 2100 in seconds
-          timestamp = timestamp / 1_000_000.0
+          # Count digits in integer part to determine format
+          integer_part = timestamp.to_i.to_s
+          digit_count = integer_part.length
+
+          if digit_count <= 10
+            # Already in seconds
+            timestamp
+          elsif digit_count <= 13
+            # Milliseconds - convert to seconds
+            timestamp = timestamp / 1_000.0
+          else
+            # Microseconds - convert to seconds
+            timestamp = timestamp / 1_000_000.0
+          end
         end
 
         timestamp
@@ -89,7 +114,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 +127,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.1"
 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.1
 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