From 63d2a69b7d32482385f2fcf03c84539a245c6862 Mon Sep 17 00:00:00 2001 From: Mathias Meyer Date: Thu, 9 May 2013 14:18:59 +0200 Subject: [PATCH] Only track endpoint if an endpoint is set. For compatibility with the V1 API layer. --- lib/travis/api/app/middleware/metriks.rb | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/travis/api/app/middleware/metriks.rb b/lib/travis/api/app/middleware/metriks.rb index 5c1849ba..ecfea648 100644 --- a/lib/travis/api/app/middleware/metriks.rb +++ b/lib/travis/api/app/middleware/metriks.rb @@ -12,9 +12,11 @@ class Travis::Api::App after do if response.status < 400 time = Time.now.utc - env['metriks.request.start'] - pattern = headers['X-Pattern'].gsub(/[:\/]/, ".") - metric = "api.request.endpoint.#{pattern}" - ::Metriks.timer(metric).update(time) + if headers['X-Pattern'] + pattern = headers['X-Pattern'].gsub(/[:\/]/, ".") + metric = "api.request.endpoint.#{pattern}" + ::Metriks.timer(metric).update(time) + end ::Metriks.meter("api.request.#{request.request_method.downcase}").mark end ::Metriks.meter("api.request.status.#{response.status.to_s[0]}").mark