From 2a4b016dd40e77d0b4578ac70a4c005050f3ce4f Mon Sep 17 00:00:00 2001 From: Abhijeet Prasad Date: Thu, 22 Sep 2022 13:50:47 +0200 Subject: [PATCH] fix(tracing): Remove connection.downlink measurement As per RFC: https://github.com/getsentry/rfcs/blob/main/text/0003-browser-js-built-in-metrics.md we have decided to remove `connection.downlink` from as a recorded performance metrics in the BrowserTracing integration --- packages/tracing/src/browser/metrics/index.ts | 4 ---- 1 file changed, 4 deletions(-) diff --git a/packages/tracing/src/browser/metrics/index.ts b/packages/tracing/src/browser/metrics/index.ts index 4e09e4a72bd1..d77b05fed837 100644 --- a/packages/tracing/src/browser/metrics/index.ts +++ b/packages/tracing/src/browser/metrics/index.ts @@ -395,10 +395,6 @@ function _trackNavigator(transaction: Transaction): void { if (isMeasurementValue(connection.rtt)) { _measurements['connection.rtt'] = { value: connection.rtt, unit: 'millisecond' }; } - - if (isMeasurementValue(connection.downlink)) { - _measurements['connection.downlink'] = { value: connection.downlink, unit: '' }; // unit is empty string for now, while relay doesn't support download speed units - } } if (isMeasurementValue(navigator.deviceMemory)) {