From f0dcb55643612ab05137314b46ee66ccf9308a53 Mon Sep 17 00:00:00 2001 From: Will Hawkins Date: Thu, 12 May 2022 15:45:40 -0400 Subject: [Functionality] Match Apple's Client RPM Calcuation -- Remove DNS Per a (to-be-released) update from the spec, the time for a DNS lookup is no longer considered to make up a portion of the RPM. --- rpm/rpm.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'rpm') diff --git a/rpm/rpm.go b/rpm/rpm.go index f41ad92..a349cee 100644 --- a/rpm/rpm.go +++ b/rpm/rpm.go @@ -303,9 +303,8 @@ func getLatency(ctx context.Context, probe *Probe, url string, debugLevel debug. tlsAndHttpHeaderDelta := probe.GetTLSAndHttpHeaderDelta() httpDownloadDelta := probe.GetHttpDownloadDelta(time_after_probe) // Combined with above, constitutes 2 time measurements, per the Spec. - dnsDelta := probe.GetDnsDelta() // Constitutes 1 time measurement, per the Spec. tcpDelta := probe.GetTCPDelta() // Constitutes 1 time measurement, per the Spec. - totalDelay := tlsAndHttpHeaderDelta + httpDownloadDelta + dnsDelta + tcpDelta + totalDelay := tlsAndHttpHeaderDelta + httpDownloadDelta + tcpDelta // By default, assume that there was a reused connection which // means that we only made 1 time measurement. @@ -313,7 +312,7 @@ func getLatency(ctx context.Context, probe *Probe, url string, debugLevel debug. if !probe.stats.ConnectionReused { // If we did not reuse the connection, then we made three additional time measurements. // See above for details on that calculation. - measurementCount = 4 + measurementCount = 3 } if debug.IsDebug(debugLevel) { -- cgit v1.2.3