#23826 closed enhancement (implemented)

Authorities: Put Relay IPv6 addresses in the microdesc consensus

Reported by: teor Owned by: teor
Priority: Medium Milestone: Tor: 0.3.3.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: ipv6, prop283, review-group-25, review-group-26
Cc: Actual Points: 0.2
Parent ID: #20916 Points: 0.5
Reviewer: Sponsor: SponsorV-can

Description

Authority Implementation for #20916

Child Tickets

TicketTypeStatusOwnerSummary
#23870defectclosedteorAuthorities: document what happens when relays have misconfigured IPv6

Change History (12)

comment:1 Changed 22 months ago by teor

Actual Points: 0.2
Points: 0.5
Sponsor: SponsorV-can

comment:2 Changed 22 months ago by teor

This is implemented in my tor branch bug23826-23828. The corresponding torspec entry is in my torspec branch bug23826-23828.

comment:3 Changed 22 months ago by teor

Status: newneeds_review

That branch also has a commit for #23828. The corresponding torspec entry is in my torspec branch bug23826-23828, and is tracked in ticket #23898.

comment:4 Changed 22 months ago by nickm

Keywords: prop283 added

I've added the proposal as prop283.

comment:5 Changed 22 months ago by teor

Owner: set to teor
Status: needs_reviewassigned

These are all mine

comment:6 Changed 22 months ago by teor

Status: assignedneeds_review

These have code and a proposal in progress on tor-dev@

comment:7 Changed 22 months ago by nickm

Keywords: review-group-25 added

comment:8 Changed 21 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:9 Changed 21 months ago by teor

I pushed a fixup to this branch to improve the AuthDirHasIPv6Connectivity man page documentation.

comment:10 Changed 21 months ago by nickm

Keywords: review-group-26 added

Creating review-group-26.

comment:11 Changed 21 months ago by nickm

Still looks good: merged to master.

comment:12 Changed 21 months ago by nickm

Resolution: implemented
Status: merge_readyclosed
Note: See TracTickets for help on using tickets.