Merge pull request #699 from stripe/brandur-check-request-id

Only send telemetry if request ID was present
stripe · Nov 15, 2018 · 10f2355 · 10f2355
2 parents 8c4813f + 78df532
commit 10f2355
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/stripe/stripe_client.rb b/lib/stripe/stripe_client.rb
@@ -225,7 +225,8 @@ def execute_request_with_rescues(api_base, context)
         resp = yield
         context = context.dup_from_response(resp)
         log_response(context, request_start, resp.status, resp.body)
-        if Stripe.enable_telemetry?
+
+        if Stripe.enable_telemetry? && context.request_id
           request_duration_ms = ((Time.now - request_start) * 1000).to_int
           @last_request_metrics = StripeRequestMetrics.new(context.request_id, request_duration_ms)
         end