Skip to content

Commit

Permalink
x
Browse files Browse the repository at this point in the history
  • Loading branch information
bassosimone committed Jan 24, 2024
1 parent 60ebc7a commit d7df251
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion internal/experiment/webconnectivitylte/analysisclassic.go
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,7 @@ func analysisClassicComputeBlockingAccessible(woa *minipipeline.WebAnalysis, tk

// 5.3. When the probe says dns_no_answer the control would otherwise say that
// we have resolved zero IP addresses for historical reasons. In such a case,
// let's pretend also the control returned dns_no_answer.
// let's pretend that also the control returned dns_no_answer.
if entry.Failure.Unwrap() == netxlite.FailureDNSNoAnswer &&
!entry.ControlDNSResolvedAddrs.IsNone() &&
entry.ControlDNSResolvedAddrs.Unwrap().Len() <= 0 {
Expand Down
2 changes: 1 addition & 1 deletion internal/minipipeline/analysis.go
Original file line number Diff line number Diff line change
Expand Up @@ -453,7 +453,7 @@ func (wa *WebAnalysis) dnsComputeFailureMetrics(c *WebObservationsContainer) {
if failure := obs.DNSLookupFailure.Unwrap(); failure != "" {
// When the probe says dns_no_answer the control would otherwise say that
// we have resolved zero IP addresses for historical reasons. In such a case,
// let's pretend also the control returned dns_no_answer.
// let's pretend that also the control returned dns_no_answer.
if failure == netxlite.FailureDNSNoAnswer {
if !obs.ControlDNSResolvedAddrs.IsNone() && obs.ControlDNSResolvedAddrs.Unwrap().Len() <= 0 {
wa.DNSLookupExpectedFailure.Add(obs.DNSTransactionID.Unwrap())
Expand Down

0 comments on commit d7df251

Please sign in to comment.