Skip to content
This repository has been archived by the owner on Mar 6, 2020. It is now read-only.

Commit

Permalink
nervousresolver: use cloudflare IP address (#94)
Browse files Browse the repository at this point in the history
We should probably introduce a longer chain of fallbacks in the
nervous resolver, but, for now, using the IP address is good enough
because it can protect us against the whole DNS being down.

This happened to me in Italy, when "Vodafone rete sicura" had
several hours of outage and the default DNS was not working as
intended, while there was IP connectivity.

Reference issue: ooni/probe-engine#88
  • Loading branch information
bassosimone authored Oct 30, 2019
1 parent 54fdd26 commit 1ec4f0b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion x/nervousresolver/nervousresolver.go
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ func init() {
// this when integrating in probe-engine.
overhttps, err := internal.NewResolver(
time.Time{}, handlers.NoHandler,
"doh", "https://cloudflare-dns.com/dns-query",
"doh", "https://1.1.1.1/dns-query",
)
rtx.PanicOnError(err, "internal.NewResolver #2 failed")
Default = New(system, overhttps)
Expand Down

0 comments on commit 1ec4f0b

Please sign in to comment.