I spoke with the directory authority operators about this, and they are happy with the existing behaviour. But we need to document it:
if no voting authorities set AuthDirHasIPv6Connectivity, there will be no IPv6 addresses in the consensus
if a minority of voting authorities set AuthDirHasIPv6Connectivity, relays with unreachable IPv6 addresses will have those addresses removed from the consensus
if a majority of voting authorities set AuthDirHasIPv6Connectivity, relays with unreachable IPv6 addresses will not be listed as running
Trac: Description: Otherwise, we lose useful relays when authorities set AuthDirHasIPv6Connectivity.
to
**Summary**: Authorities: When relays have misconfigured IPv6, mark them as running and IPv4 only **to** Authorities: document what happens when relays have misconfigured IPv6 **Keywords**: needs-proposal, regression **deleted**, doc **added**
This code looks good to me, modulo open questions on proposal 283 [*]. After we discuss them, we should either merge this, or put it into needs_revision, depending on the outcome.