Skip to content

Commit

Permalink
add note about bogons and resolution
Browse files Browse the repository at this point in the history
  • Loading branch information
ainghazal committed Oct 23, 2024
1 parent 2acc5df commit 6d5b427
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions internal/experiment/openvpn/targets.go
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down

0 comments on commit 6d5b427

Please sign in to comment.