From 6d5b42784263e653a81296cf0ad5f06822676614 Mon Sep 17 00:00:00 2001 From: ain ghazal Date: Thu, 24 Oct 2024 01:31:22 +0200 Subject: [PATCH] add note about bogons and resolution --- internal/experiment/openvpn/targets.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/internal/experiment/openvpn/targets.go b/internal/experiment/openvpn/targets.go index f109d47f3..655a55fcb 100644 --- a/internal/experiment/openvpn/targets.go +++ b/internal/experiment/openvpn/targets.go @@ -41,6 +41,12 @@ func sampleN(a []string, n int) []string { // performing DNS resolution. The returned IP addreses exclude possible // bogons. func resolveOONIAddresses(logger model.Logger) ([]string, error) { + + // We explicitely resolve with BogonIsError set to false, and + // later remove bogons from the list. The reason is that in this way + // we are able to control the rate at which we run tests by adding bogon addresses to the + // domain records for the test. + resolver := netx.NewResolver(netx.Config{ BogonIsError: false, Logger: logger,