Opened 10 months ago

Closed 9 months ago

#23870 closed defect (fixed)

Authorities: document what happens when relays have misconfigured IPv6

Reported by: teor Owned by: teor
Priority: Medium Milestone: Tor: 0.3.3.x-final
Component: Core Tor/Tor Version: Tor: 0.2.4.1-alpha
Severity: Normal Keywords: ipv6, doc, review-group-25, review-group-26
Cc: Actual Points: 0.1
Parent ID: #23826 Points: 0.2
Reviewer: Sponsor:

Description (last modified by teor)

~Otherwise, we lose useful relays when authorities set AuthDirHasIPv6Connectivity.~

Child Tickets

Change History (9)

comment:1 Changed 10 months ago by teor

Description: modified (diff)
Keywords: doc added; needs-proposal regression removed
Summary: Authorities: When relays have misconfigured IPv6, mark them as running and IPv4 onlyAuthorities: document what happens when relays have misconfigured IPv6

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

comment:2 Changed 10 months ago by teor

Status: newneeds_review
Version: Tor: 0.2.4.1-alpha

This is implemented in my tor branch bug23826-23828 along with #23826 and #23828.

comment:3 Changed 10 months ago by teor

Parent ID: #20916#23826

comment:4 Changed 10 months ago by nickm

Owner: set to teor
Status: needs_reviewassigned

setting owner

comment:5 Changed 10 months ago by nickm

Status: assignedneeds_review

comment:6 Changed 9 months ago by nickm

Keywords: review-group-25 added

comment:7 Changed 9 months ago by nickm

Status: needs_reviewmerge_ready

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.

[*] https://lists.torproject.org/pipermail/tor-dev/2017-November/012543.html

comment:8 Changed 9 months ago by nickm

Keywords: review-group-26 added

Creating review-group-26.

comment:9 Changed 9 months ago by nickm

Resolution: fixed
Status: merge_readyclosed

Parent merged.

Note: See TracTickets for help on using tickets.