Remove arbitrary upper bound on queue time

Only validate that queue time is positive - there's no good reason to silently
drop high queue time values as they represent legitimate observability data
during incidents.

Author: samuel-williams-shopify

lib/metrics/provider/async/http/server.rb

@@ -53,8 +53,8 @@ def calculate_queue_time(header_value)
 			current_time = Process.clock_gettime(Process::CLOCK_REALTIME)
 			queue_time = current_time - timestamp
 
-			# Sanity check: queue time should be positive and reasonable (< 1 hour)
-			if queue_time > 0 && queue_time < 3600
+			# Sanity check: queue time should be positive
+			if queue_time > 0
 				return queue_time
 			end
 		rescue ArgumentError, TypeError

test/metrics/provider/async/http/server.rb

@@ -129,26 +129,6 @@
 			server_task.stop
 		end
 
-		it "ignores unreasonable queue times" do
-			server_task = server.run
-			bound_endpoint = server_task.wait_until_ready
-			
-			# Timestamp from 2 hours ago (unreasonable)
-			request_start = Process.clock_gettime(Process::CLOCK_REALTIME) - 7200
-			
-			# Should not emit the queue time metric for unreasonable timestamp
-			expect(Async::HTTP::Server::ASYNC_HTTP_SERVER_REQUEST_QUEUE_TIME).not.to receive(:emit)
-			
-			client = Async::HTTP::Client.new(bound_endpoint)
-			headers = [["x-request-start", "t=#{request_start}"]]
-			response = client.get("/", headers)
-			
-			expect(response.status).to be == 200
-			response.finish
-			
-			client.close
-			server_task.stop
-		end
 	end
 end