diff options
| author | Will Hawkins <[email protected]> | 2022-05-12 15:45:40 -0400 |
|---|---|---|
| committer | Will Hawkins <[email protected]> | 2022-05-12 15:45:40 -0400 |
| commit | f0dcb55643612ab05137314b46ee66ccf9308a53 (patch) | |
| tree | 339af5680fdec05d1a5bcab27a1ec782459103a7 | |
| parent | e6cd1fedf2522e58a38a51eaa676f59094da8249 (diff) | |
[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.
| -rw-r--r-- | rpm/rpm.go | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -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) { |