Ok turns out that DNS and Exit Policy settings are kind of linked together...
For instance, when building our routerinfo object, we append "reject *" if our DNS init failed but this patch is about trying to initializing the DNS by looking if we are configured as an Exit... so the approach gets a bit more complicated.
Furthermore, check_dns_honesty_event() can be launched through the dns_init() code path which means it somehow needs to be enabled which only gets called in the initialized event callback which is after the first dns_init() call in do_main_loop()....
Bottom line, there seems to be some complexity with this one so deferring to 035. Not too big of a deal if a non-Exit relay launches those events.
Trac: Milestone: Tor: 0.3.4.x-final to Tor: 0.3.5.x-final Reviewer: dgoulet toN/A
Deferring 51 tickets from 0.4.0.x-final. Tagging them with 040-deferred-20190220 for visibility. These are the tickets that did not get 040-must, 040-can, or tor-ci.