From f77474a91a46fdc23d1c6c937fa5bc30b0e9f520 Mon Sep 17 00:00:00 2001 From: Simone Basso Date: Mon, 12 Sep 2022 11:03:55 +0200 Subject: [PATCH] fix(webconnectivity@v0.5): avoid confusing log message (#954) It's confusing to see ``` measuring additional addrs from TH: [] ``` when actually nothing is going to be measured. So, instead, let us log about the additional addrs discovered by the TH instead, which is less confusing. Part of https://github.com/ooni/probe/issues/2237 --- internal/experiment/webconnectivity/control.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/experiment/webconnectivity/control.go b/internal/experiment/webconnectivity/control.go index 8824c96..3e58f42 100644 --- a/internal/experiment/webconnectivity/control.go +++ b/internal/experiment/webconnectivity/control.go @@ -164,7 +164,7 @@ func (c *Control) maybeStartExtraMeasurements(ctx context.Context, thAddrs []str thOnlyAddrs = append(thOnlyAddrs, addr) } - c.Logger.Infof("measuring additional addrs from TH: %+v", thOnlyAddrs) + c.Logger.Infof("additional addrs discovered by the TH: %+v", thOnlyAddrs) var thOnly []DNSEntry for _, addr := range thOnlyAddrs {